Bug 9574

Summary: Drosera should show inline scripts within the original HTML
Product: WebKit Reporter: Timothy Hatcher <timothy>
Component: New BugsAssignee: Timothy Hatcher <timothy>
Status: CLOSED FIXED    
Severity: Normal    
Priority: P2    
Version: 420+   
Hardware: Mac   
OS: OS X 10.4   
Attachments:
Description Flags
Implements this darin: review+

Timothy Hatcher
Reported 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.
Attachments
Implements this (40.19 KB, patch)
2006-06-25 01:38 PDT, Timothy Hatcher 		darin: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Timothy Hatcher
Comment 1 2006-06-25 01:38:13 PDT
Created attachment 9014 [details] Implements this
Darin Adler
Comment 2 2006-06-25 10:03:04 PDT
Comment on attachment 9014 [details] Implements this r=me
Timothy Hatcher
Comment 3 2006-06-25 10:47:52 PDT
Landed in r15026.
Timothy Hatcher
Comment 4 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.
Note You need to log in before you can comment on or make changes to this bug.