Bug 118618

Summary:	Don't force layout for offsetWidth of fixed width elements
Product:	WebKit	Reporter:	Ryosuke Niwa <rniwa>
Component:	Layout and Rendering	Assignee:	Nobody <webkit-unassigned>
Status:	NEW ---
Severity:	Normal	CC:	benjamin, dino, hyatt, simon.fraser
Priority:	P2	Keywords:	BlinkMergeCandidate
Version:	528+ (Nightly build)
Hardware:	Unspecified
OS:	Unspecified
Bug Depends on:	118930
Bug Blocks:

Description Ryosuke Niwa 2013-07-12 14:49:11 PDT

Consider merging https://chromium.googlesource.com/chromium/blink/+/5117d5329f6cdf071dfec25d9026a105070142b6

This improves page load time of The Economist by 41% (3.4s -> 2.0s) on
my z620 workstation.

I believe the change in the inspector/styles/device-metrics-fit-window.html expectations is a progression.
The div being measured has a fixed width of 1024 and there is a zoom on the page.
It looks to me like there was a rounding error previously.

Also see the bug 118032

Comment 1 Simon Fraser (smfr) 2013-07-12 14:59:34 PDT

Nice. I don't like the "fixed" terminology though; too easily confused with fixed position.

Comment 2 Ryosuke Niwa 2013-07-15 14:53:02 PDT

Also see https://chromium.googlesource.com/chromium/blink/+/ff234b1593b2b493d47f38f687d09a87bc42c9eb.

Comment 3 Ryosuke Niwa 2013-07-15 14:53:28 PDT

Also see the bug 118032.

Comment 4 Ryosuke Niwa 2013-07-25 16:20:31 PDT

Also merge:
https://src.chromium.org/viewvc/blink?revision=154203&view=revision
https://src.chromium.org/viewvc/blink?revision=154292&view=revision

Comment 5 Ryosuke Niwa 2013-11-05 14:40:54 PST

Also see https://chromium.googlesource.com/chromium/blink/+/58a5fac292a5b3f79b00cdc71074fbf8be2ebc2b

Comment 6 Ryosuke Niwa 2013-11-06 20:30:03 PST

The optimization removed in https://chromium.googlesource.com/chromium/blink/+/e6648e2c65b3472faf5025cb19029afe71b7a7f0