Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(116)

Issue 2750653002: [fullscreen] (not landed) Force a layout on exiting fullscreen. (Closed)

Created:
3 years, 9 months ago by Eric Seckler
Modified:
3 years, 9 months ago
Reviewers:
CC:
mlamouri (slow - plz ping), blink-reviews, chromium-reviews, kinuko+watch
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

[fullscreen] (not landed) Force a layout on exiting fullscreen. This avoids a race between ViewMsg_Resize and the layout, which can cause a bad scale factor be applied to the page during resize. BUG=698315

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+4 lines, -0 lines) Patch
M third_party/WebKit/Source/web/FullscreenController.cpp View 1 chunk +4 lines, -0 lines 0 comments Download

Messages

Total messages: 12 (10 generated)
mlamouri (slow - plz ping)
Shouldn't this be closed?
3 years, 9 months ago (2017-03-20 11:12:11 UTC) #9
Eric Seckler
3 years, 9 months ago (2017-03-20 11:19:58 UTC) #10
On 2017/03/20 11:12:11, mlamouri wrote:
> Shouldn't this be closed?

aelias fixed this in an even cleverer way:
https://codereview.chromium.org/2745313002

but yes, I'm gonna close this as it's no longer needed now :)

Powered by Google App Engine
This is Rietveld 408576698