Bug 95779

Summary: Fix style for Event.h
Product: WebKit Reporter: nbhargava
Component: New BugsAssignee: nbhargava
Status: RESOLVED FIXED    
Severity: Normal CC: eric, robert, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 95780    
Attachments:
Description Flags
Patch none

nbhargava
Reported 2012-09-04 13:44:58 PDT
Fix style for Event.h
Attachments
Patch (11.48 KB, patch)
2012-09-04 13:46 PDT, nbhargava 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
nbhargava
Comment 1 2012-09-04 13:46:14 PDT
Created attachment 162096 [details] Patch
Robert Hogan
Comment 2 2012-09-04 14:03:29 PDT
This seems like a bad idea to me - we only fix style issues as we go rather than wholesale.
Eric Seidel (no email)
Comment 3 2012-09-04 14:04:33 PDT
Comment on attachment 162096 [details] Patch LGTM.
Eric Seidel (no email)
Comment 4 2012-09-04 14:04:55 PDT
Any time you touch this file it will complain. someone has to re-indent it eventually. :)
nbhargava
Comment 5 2012-09-04 14:14:12 PDT
Yeah, I only changed this one because another set of changes I made touched this file. I figured it would be less messy/easier to understand by just breaking them into two separate commits.
WebKit Review Bot
Comment 6 2012-09-04 15:22:18 PDT
Comment on attachment 162096 [details] Patch Clearing flags on attachment: 162096 Committed r127514: <http://trac.webkit.org/changeset/127514>
WebKit Review Bot
Comment 7 2012-09-04 15:22:21 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.