Uploaded image for project: 'The Virtual Brain'
  1. The Virtual Brain
  2. TVB-690 Code Review - Unit test
  3. TVB-675

Lots of external JavaScript ressources get requested multiple times per page

    XMLWordPrintable

Details

    • Sub-task
    • Status: Closed
    • Minor
    • Resolution: Fixed
    • None
    • 1.0.1
    • None
    • None

    Description

      A casual glance at several page's HTML source code reveals that <script> elements do request the same JavaScript resources multiple times (e.g. jQuery and its modules/minified versions). This phenomenon also covers (to a lesser extent) CSS ressources.

      This is a consequence of TVB's adapter architecture IMO. To see this in action, try e.g. the Burst or the Surface Stimulus page.

      The culprits are the pseudo header sequences from each adapter. I understand they want to be independent, so they each request their own resources in order to survive in different coding contexts.

      Regarding redundant CSS rssources, that's not such a big problem as multiple requests to the same CSS file hit the browser cache.

      For JS resources, however, I'm not so sure:
      I don't think that multiple requests for the same .js file hit the browser cache. And these .js files tend to be large!

      Gliffy Diagrams

        Attachments

          Activity

            People

              ionel.ortelecan Ionel Ortelecan
              michael.burgstahler Michael Burgstahler
              Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Time Tracking

                  Estimated:
                  Original Estimate - 5 hours Original Estimate - 5 hours
                  5h
                  Remaining:
                  Remaining Estimate - 0 minutes
                  0m
                  Logged:
                  Time Spent - 3 days, 5 hours, 30 minutes
                  3d 5h 30m