Bug 11246 - Minor Drosera code cleanup
Summary: Minor Drosera code cleanup
Status: CLOSED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 420+
Hardware: Mac OS X 10.4
: P2 Minor
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2006-10-10 13:26 PDT by David Smith
Modified: 2008-05-17 09:55 PDT (History)
0 users

See Also:

Attachments
Makes the change described in the bug (6.58 KB, patch)
2006-10-10 13:32 PDT, David Smith 		timothy: 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 David Smith 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.
Comment 1 David Smith 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.
Comment 2 Timothy Hatcher 2006-10-10 14:07:21 PDT 
Comment on attachment 11022 [details]
Makes the change described in the bug

Looks good!
Comment 3 Timothy Hatcher 2006-10-10 21:44:18 PDT 
Landed in r16985.
Comment 4 Timothy Hatcher 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.