Bug 73661 - [chromium] Eliminate unnecessary state on previous CL
Summary: [chromium] Eliminate unnecessary state on previous CL
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: Jonathan Backer
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-12-02 08:15 PST by Jonathan Backer
Modified: 2011-12-02 15:03 PST (History)
5 users (show)

See Also:

Attachments
Patch (4.94 KB, patch)
2011-12-02 08:15 PST, Jonathan Backer 		no flags Details | Formatted Diff | Diff
Patch (4.92 KB, patch)
2011-12-02 10:28 PST, Jonathan Backer 		no flags Details | Formatted Diff | Diff
Show Obsolete (1) 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 Jonathan Backer 2011-12-02 08:15:08 PST 
[chromium] NOT FOR REVIEW Eliminate unnecessary state on previous CL
Comment 1 Jonathan Backer 2011-12-02 08:15:44 PST 
Created attachment 117628 [details]
Patch
Comment 2 Jonathan Backer 2011-12-02 08:17:20 PST 
Address nduca's comments post landing on https://bugs.webkit.org/show_bug.cgi?id=72956
Comment 3 Nat Duca 2011-12-02 08:29:17 PST 
Comment on attachment 117628 [details]
Patch

Unofficially LGTM. Thanks!
Comment 4 Jonathan Backer 2011-12-02 10:28:38 PST 
Created attachment 117650 [details]
Patch
Comment 5 Kenneth Russell 2011-12-02 10:31:08 PST 
Comment on attachment 117650 [details]
Patch

Looks fine. r=me
Comment 6 WebKit Review Bot 2011-12-02 15:03:24 PST 
Comment on attachment 117650 [details]
Patch

Clearing flags on attachment: 117650

Committed r101864: <http://trac.webkit.org/changeset/101864>
Comment 7 WebKit Review Bot 2011-12-02 15:03:29 PST 
All reviewed patches have been landed.  Closing bug.