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

Side by Side Diff: Source/core/frame/Settings.in

Issue 146683003: Settings should not call into inspector (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: incorporated review comments Created 6 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « Source/core/frame/Settings.cpp ('k') | Source/core/frame/SettingsDelegate.h » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 # Defines properties which are available on the Settings object. 1 # Defines properties which are available on the Settings object.
2 # 2 #
3 # Please think carefully before adding a new Setting. Some questions to 3 # Please think carefully before adding a new Setting. Some questions to
4 # consider are: 4 # consider are:
5 # - Should this be a RuntimeEnabledFeature instead? Settings are for things 5 # - Should this be a RuntimeEnabledFeature instead? Settings are for things
6 # which we support either values of at runtime. Features are set at renderer 6 # which we support either values of at runtime. Features are set at renderer
7 # process startup and are never changed. Features also tend to be set to a 7 # process startup and are never changed. Features also tend to be set to a
8 # value based on the platform or the stability of the code in question, where 8 # value based on the platform or the stability of the code in question, where
9 # as settings both codepaths need to be stable. 9 # as settings both codepaths need to be stable.
10 # - How will you ensure test coverage of all relevant values of your setting? 10 # - How will you ensure test coverage of all relevant values of your setting?
(...skipping 268 matching lines...) Expand 10 before | Expand all | Expand 10 after
279 touchEventEmulationEnabled initial=false 279 touchEventEmulationEnabled initial=false
280 280
281 281
282 # FIXME: This is a temporary flag and should be removed once 282 # FIXME: This is a temporary flag and should be removed once
283 # accelerated overflow scroll is ready (crbug.com/254111). 283 # accelerated overflow scroll is ready (crbug.com/254111).
284 compositorDrivenAcceleratedScrollingEnabled initial=false 284 compositorDrivenAcceleratedScrollingEnabled initial=false
285 285
286 # FIXME: This is a temporary flag and should be removed 286 # FIXME: This is a temporary flag and should be removed
287 # when squashing is ready. (crbug.com/261605) 287 # when squashing is ready. (crbug.com/261605)
288 layerSquashingEnabled initial=false 288 layerSquashingEnabled initial=false
289
290 # Clients that execute script should call ScriptController::canExecuteScripts()
291 # instead of this function. ScriptController::canExecuteScripts() checks the
292 # HTML sandbox, plug-in sandboxing, and other important details.
293 scriptEnabled initial=false, invalidate=ScriptEnable
294
295 # Compensates for poor text legibility on mobile devices. This value is
296 # multiplied by the font scale factor when performing text autosizing of
297 # websites that do not set an explicit viewport description.
298 deviceScaleAdjustment type=double, initial=1.0, invalidate=TextAutosizing
OLDNEW
« no previous file with comments | « Source/core/frame/Settings.cpp ('k') | Source/core/frame/SettingsDelegate.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698