Bug 9574 - Drosera should show inline scripts within the original HTML
Summary: Drosera should show inline scripts within the original HTML
Status: CLOSED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 420+
Hardware: Mac OS X 10.4
: P2 Normal
Assignee: Timothy Hatcher
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2006-06-25 01:17 PDT by Timothy Hatcher
Modified: 2008-05-17 09:55 PDT (History)
0 users

See Also:

Attachments
Implements this (40.19 KB, patch)
2006-06-25 01:38 PDT, Timothy Hatcher 		darin: review+
Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Timothy Hatcher 2006-06-25 01:17:22 PDT 
Currently Drosera shows each script as a separate file in the popup list, even for <script> tags. Drosera needs work with inline scripts (event attributes and script tags) in the context of the original HTMl source.

* Only show one entry in the file popup for the main HTML file.
* Ability to set breakpoints in the HTML file to catch event attributes (onclick, etc).
* Ability to set breakpoints in the HTML file for scripts in <script> tags.

Right now Yahoo.com has around 13 inline scripts and they pollute the file popup. This will reduce to around 5 items once this is fixed.

This will require additional work in WebKit to provide Drosera the information it needs.
Comment 1 Timothy Hatcher 2006-06-25 01:38:13 PDT 
Created attachment 9014 [details]
Implements this
Comment 2 Darin Adler 2006-06-25 10:03:04 PDT 
Comment on attachment 9014 [details]
Implements this

r=me
Comment 3 Timothy Hatcher 2006-06-25 10:47:52 PDT 
Landed in r15026.
Comment 4 Timothy Hatcher 2008-05-17 09:55:51 PDT 
Closing since Drosera has been replaced by the new Web Inspector debugger. Moving to the New Bugs component so the Drosera component can be closed and removed.