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

Issue 156943004: Remove minimum layout delay (Closed)

Created:
6 years, 10 months ago by abarth-chromium
Modified:
6 years, 10 months ago
Reviewers:
ojan, eseidel
CC:
blink-reviews, dglazkov+blink, sof, eae+blinkwatch, adamk+blink_chromium.org, Inactive
Visibility:
Public.

Description

Remove minimum layout delay This CL removes the concept of a minium layout delay. Now that we use animation frames to control the timing of layout, this concept doesn't make much sense. It's possible we'll add back a feature like this in the future, but we'll need to structure the code a bit differently. R=ojan@chromium.org BUG=341627 Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=166669

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -38 lines) Patch
M Source/core/dom/Document.h View 1 chunk +0 lines, -1 line 0 comments Download
M Source/core/dom/Document.cpp View 2 chunks +1 line, -14 lines 0 comments Download
M Source/core/frame/Frame.cpp View 1 chunk +0 lines, -3 lines 0 comments Download
M Source/core/frame/FrameView.h View 2 chunks +0 lines, -2 lines 0 comments Download
M Source/core/frame/FrameView.cpp View 5 chunks +0 lines, -18 lines 0 comments Download

Messages

Total messages: 5 (0 generated)
abarth-chromium
6 years, 10 months ago (2014-02-06 22:44:39 UTC) #1
ojan
lgtm
6 years, 10 months ago (2014-02-06 22:52:26 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/abarth@chromium.org/156943004/1
6 years, 10 months ago (2014-02-06 22:52:39 UTC) #3
commit-bot: I haz the power
Change committed as 166669
6 years, 10 months ago (2014-02-07 01:18:59 UTC) #4
eseidel
6 years, 10 months ago (2014-02-07 01:22:44 UTC) #5
Message was sent while issue was closed.
FYI, this was my old patch to do this:
https://codereview.chromium.org/20140002/

Powered by Google App Engine
This is Rietveld 408576698