Bug 102101 - [CSS Regions] regionlayoutupdate event fires continuously
Summary: [CSS Regions] regionlayoutupdate event fires continuously
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 treegraph
Reported: 2012-11-13 10:04 PST by Mike Sierra
Modified: 2012-12-02 07:36 PST (History)
10 users (show)

See Also:

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

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]
Comment 3 Dave Hyatt 2012-11-30 12:42:38 PST
Comment on attachment 176654 [details]

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.