Bug 11246

Summary: Minor Drosera code cleanup
Product: WebKit Reporter: David Smith <catfish.man>
Component: New BugsAssignee: Nobody <webkit-unassigned>
Status: CLOSED FIXED    
Severity: Minor    
Priority: P2    
Version: 420+   
Hardware: Mac   
OS: OS X 10.4   
Attachments:
Description Flags
Makes the change described in the bug timothy: review+

David Smith
Reported 2006-10-10 13:26:43 PDT
As discussed on irc, it seems cleaner to declare [has, add, remove]StyleClass() as functions on Element.prototype rather than having them take an element as an argument.
Attachments
Makes the change described in the bug (6.58 KB, patch)
2006-10-10 13:32 PDT, David Smith 		timothy: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
David Smith
Comment 1 2006-10-10 13:32:47 PDT
Created attachment 11022 [details] Makes the change described in the bug May be worth waiting for vladinecko's file browser patch, since this will likely be easier to update.
Timothy Hatcher
Comment 2 2006-10-10 14:07:21 PDT
Comment on attachment 11022 [details] Makes the change described in the bug Looks good!
Timothy Hatcher
Comment 3 2006-10-10 21:44:18 PDT
Landed in r16985.
Timothy Hatcher
Comment 4 2008-05-17 09:55:59 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.