Bug 102101 - [CSS Regions] regionlayoutupdate event fires continuously
Summary: [CSS Regions] regionlayoutupdate event fires continuously
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: CSS (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Mihnea Ovidenie
URL: https://github.com/mike-sierra/webpla...
Keywords: AdobeTracked
Depends on:
Blocks: 57312
  Show dependency tree / graph
 
Reported: 2012-11-13 10:04 PST by Mike Sierra
Modified: 2012-12-02 07:36 PST (History)
10 users (show)

See Also:

Attachments
Patch (8.03 KB, patch)
2012-11-28 22:55 PST, Mihnea Ovidenie 		no flags Details | Formatted Diff | Diff
Patch for landing (8.42 KB, patch)
2012-12-02 07:14 PST, Mihnea Ovidenie 		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 Mike Sierra 2012-11-13 10:04:47 PST 
Copy HTML source into local file and open with Canary or Webkit nightly.
The page displays content that threads through a series of CSS regions,
the size of each thresholded with min-/max-height CSS.
Each time the regionlayoutupdate event fires, 
the page increments a counter that displays at the top.

Expected behavior is for event to fire on the NamedFlow only when:

* the regionOverset of any region in the chain changes between fit/empty/overset values.

* possibly if there are less significant changes to layout, such as shifts in linebreaks within any of the regions 

Actual behavior is either:

* event fires continuously onload

* event fires continuously after resizing page
Comment 1 Mihnea Ovidenie 2012-11-14 07:54:25 PST 
Taking a quick look, the event fires only when the regions are auto-size regions.
Comment 2 Mihnea Ovidenie 2012-11-28 22:55:15 PST 
Created attachment 176654 [details]
Patch
Comment 3 Dave Hyatt 2012-11-30 12:42:38 PST 
Comment on attachment 176654 [details]
Patch

r=me
Comment 4 Mihnea Ovidenie 2012-12-02 07:14:19 PST 
Created attachment 177140 [details]
Patch for landing
Comment 5 WebKit Review Bot 2012-12-02 07:36:18 PST 
Comment on attachment 177140 [details]
Patch for landing

Clearing flags on attachment: 177140

Committed r136346: <http://trac.webkit.org/changeset/136346>
Comment 6 WebKit Review Bot 2012-12-02 07:36:22 PST 
All reviewed patches have been landed.  Closing bug.