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

Issue 13741020: Remove unused runtime setting paginateduringlayoutenabled (Closed)

Created:
7 years, 8 months ago by pilgrim_google
Modified:
7 years, 8 months ago
CC:
blink-reviews, Stephen Chennney, Tom Hudson, edisonn
Visibility:
Public.

Description

Remove unused runtime setting paginateduringlayoutenabled This setting existed in WebKit but Chromium never changed it from its default value. We're removing all such unchanging settings in Blink, along with their associated code. This one isn't even referenced anymore except in the Settings.in file. R=abarth@chromium.org BUG=226711 Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=147845

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+0 lines, -1 line) Patch
M Source/WebCore/page/Settings.in View 1 chunk +0 lines, -1 line 0 comments Download

Messages

Total messages: 6 (0 generated)
pilgrim_google
7 years, 8 months ago (2013-04-08 19:53:38 UTC) #1
abarth-chromium
lgtm
7 years, 8 months ago (2013-04-08 19:56:02 UTC) #2
pilgrim_google
Committed patchset #1 manually as r147845 (presubmit successful).
7 years, 8 months ago (2013-04-08 20:15:05 UTC) #3
eseidel
Was this used for testing in some DRT port? Or was this some printing flag ...
7 years, 8 months ago (2013-04-09 03:08:19 UTC) #4
Tom Hudson
Edison has been picking up Skia's PDF output backend; maybe he knows if anybody is ...
7 years, 8 months ago (2013-04-09 05:30:07 UTC) #5
Tom Hudson
7 years, 8 months ago (2013-04-09 05:30:27 UTC) #6
Message was sent while issue was closed.

          

Powered by Google App Engine
This is Rietveld 408576698