Bug 87785 - [V8] Clean up V8LazyEventListener to use one less function call
Summary: [V8] Clean up V8LazyEventListener to use one less function call
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Erik Arvidsson
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-05-29 14:58 PDT by Erik Arvidsson
Modified: 2012-05-30 11:33 PDT (History)
6 users (show)

See Also:

Attachments
Patch (4.91 KB, patch)
2012-05-29 15:12 PDT, Erik Arvidsson 		no flags 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 Erik Arvidsson 2012-05-29 14:58:03 PDT 
[V8] Clean up V8LazyEventListener to use one less function call
Comment 1 Erik Arvidsson 2012-05-29 15:12:08 PDT 
Created attachment 144631 [details]
Patch
Comment 2 Kentaro Hara 2012-05-29 16:50:10 PDT 
Comment on attachment 144631 [details]
Patch

Looks OK.
Comment 3 Erik Arvidsson 2012-05-29 17:13:43 PDT 
I don't have a win build so it is very likely that my manual rebaseline of the expected file is wrong.
Comment 4 Erik Arvidsson 2012-05-30 10:20:49 PDT 
Rafael, Stephen: This may need a rebaseline for chromium-win since I manually updated the expectation since I don't have a windows build:

inspector/debugger/debugger-scripts.html
Comment 5 WebKit Review Bot 2012-05-30 11:33:20 PDT 
Comment on attachment 144631 [details]
Patch

Clearing flags on attachment: 144631

Committed r118944: <http://trac.webkit.org/changeset/118944>
Comment 6 WebKit Review Bot 2012-05-30 11:33:24 PDT 
All reviewed patches have been landed.  Closing bug.