[Webkit-unassigned] [Bug 104384] Web Inspector: Pass the script url to the script-preprocessor script

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Tue Dec 11 09:28:13 PST 2012


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





--- Comment #10 from johnjbarton <johnjbarton at chromium.org>  2012-12-11 09:30:36 PST ---
(In reply to comment #9)
> (From update of attachment 178585 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=178585&action=review
> 
> > LayoutTests/inspector/debugger/debugger-script-preprocessor-expected.txt:5
> > +inspector-test.js.js
> 
> It should be inspector-test.js (not .js.js)
> 
> > LayoutTests/inspector/debugger/debugger-script-preprocessor-expected.txt:6
> > +debugger-test.js.js
> 
> ditto

Note: this part of the test was added for bug 104384, beyond bug 80992

I added the ".js" to be certain that the output resulted from the preprocessor call and thus to be certain that the test did indeed use the |name| argument. Would it be clearer if I used ".verified" or some such? 

> 
> > LayoutTests/inspector/debugger/debugger-script-preprocessor-expected.txt:7
> > +debugger-script-preprocessor.html
> 
> This should occur only once

I guess one comes from the <script> tag and one from the 
   onload="load()"
but in any case how can I prevent this second occurrence?

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


More information about the webkit-unassigned mailing list