Bug 100335 - webkitRegionLayoutUpdate is incorrectly named
Summary: webkitRegionLayoutUpdate is incorrectly named
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: Adam Barth
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-10-24 23:27 PDT by Adam Barth
Modified: 2012-11-05 14:32 PST (History)
8 users (show)

See Also:

Attachments
Patch (10.88 KB, patch)
2012-10-24 23:28 PDT, Adam Barth 		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 Adam Barth 2012-10-24 23:27:25 PDT 
webkitRegionLayoutUpdate is incorrectly named
Comment 1 Adam Barth 2012-10-24 23:28:04 PDT 
Created attachment 170559 [details]
Patch
Comment 2 Eric Seidel (no email) 2012-10-24 23:32:42 PDT 
I take it event names are case-sensitive?
Comment 3 Adam Barth 2012-10-24 23:33:56 PDT 
> I take it event names are case-sensitive?

Yep.
Comment 4 Adam Barth 2012-10-24 23:34:36 PDT 
Some old events, like DOMContentLoaded, use mixed case, but the prevailing style is to use all lower case.
Comment 5 Adam Barth 2012-10-24 23:44:47 PDT 
Looks like the spec is wrong too:

http://dev.w3.org/csswg/css3-regions/#region-flow-layout-events
Comment 6 Adam Barth 2012-10-25 00:32:02 PDT 
> Looks like the spec is wrong too:

"wrong" is probably too strong a term.  Rather, I should say that the naming of this event isn't consistent with the rest of the platform.
Comment 7 Alan Stearns 2012-11-05 13:17:00 PST 
I've updated the spec to use 'regionlayoutupdate'
Comment 8 Adam Barth 2012-11-05 14:07:06 PST 
Thanks Alan!
Comment 9 WebKit Review Bot 2012-11-05 14:32:33 PST 
Comment on attachment 170559 [details]
Patch

Clearing flags on attachment: 170559

Committed r133527: <http://trac.webkit.org/changeset/133527>
Comment 10 WebKit Review Bot 2012-11-05 14:32:37 PST 
All reviewed patches have been landed.  Closing bug.