Bug 95779 - Fix style for Event.h
Summary: Fix style for Event.h
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: nbhargava
URL:
Keywords:
Depends on:
Blocks: 95780
  Show dependency tree / graph
 
Reported: 2012-09-04 13:44 PDT by nbhargava
Modified: 2012-09-04 15:22 PDT (History)
3 users (show)

See Also:

Attachments
Patch (11.48 KB, patch)
2012-09-04 13:46 PDT, nbhargava 		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 nbhargava 2012-09-04 13:44:58 PDT 
Fix style for Event.h
Comment 1 nbhargava 2012-09-04 13:46:14 PDT 
Created attachment 162096 [details]
Patch
Comment 2 Robert Hogan 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.
Comment 3 Eric Seidel (no email) 2012-09-04 14:04:33 PDT 
Comment on attachment 162096 [details]
Patch

LGTM.
Comment 4 Eric Seidel (no email) 2012-09-04 14:04:55 PDT 
Any time you touch this file it will complain.  someone has to re-indent it eventually. :)
Comment 5 nbhargava 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.
Comment 6 WebKit Review Bot 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>
Comment 7 WebKit Review Bot 2012-09-04 15:22:21 PDT 
All reviewed patches have been landed.  Closing bug.