[Webkit-unassigned] [Bug 71017] New: Web Inspector: introduce script origin and push it into the inspector.

Thu Oct 27 07:42:26 PDT 2011

https://bugs.webkit.org/show_bug.cgi?id=71017

           Summary: Web Inspector: introduce script origin and push it
                    into the inspector.
           Product: WebKit
           Version: 528+ (Nightly build)
          Platform: All
        OS/Version: All
            Status: UNCONFIRMED
          Severity: Normal
          Priority: P2
         Component: Web Inspector
        AssignedTo: webkit-unassigned at lists.webkit.org
        ReportedBy: pfeldman at chromium.org
                CC: timothy at apple.com, rik at webkit.org, keishi at webkit.org,
                    pmuellr at yahoo.com, joepeck at webkit.org,
                    pfeldman at chromium.org, yurys at chromium.org,
                    bweinstein at apple.com, apavlov at chromium.org,
                    loislo at chromium.org

We need to distinguish scripts generated with document.write from the ones generated by the parser of the original resource. Otherwise we bind them to the resource and hence lose them from the debugging UI.

Consider the following example:

1: <html>
2: <script>function aa() {}</script>
3: <script>function bb() {}</script>
4: <script>document.write("<scrip" + "t>\n\n\n\n\nfunction cc() {}</sc" + "ript>");</script>
5: <script>function dd() {}</script>
6: </html>

As a result, following scripts are reported to be parsed (all sharing the main resource url):

line 2-2
line 3-3
line 4-10
line 5-5

As a result of this patch, scripts will be annotated:

line 2-2 (parser)
line 3-3 (parser)
line 4-10 (document.write)
line 5-5 (parser)

As a drive by, I annotate more script origins in this patch. Note that it will need additional support from the JSC to work properly.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.