Documentation for webview settings in Settings.in

Documentation for webview settings in Settings.in

Clarified what the wide viewport and overview mode settings are for.

R=aelias@chromium.org

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=191722

[aelias_OOO_until_Jul13, 2015-03-06 20:06:17]

lgtm modulo suggestion

https://codereview.chromium.org/979243005/diff/1/Source/core/frame/Settings.in
File Source/core/frame/Settings.in (right):

https://codereview.chromium.org/979243005/diff/1/Source/core/frame/Settings.i...
Source/core/frame/Settings.in:236: # For
android.webkit.WebSettings.setLoadWithOverviewMode()
How about linking
http://developer.android.com/reference/android/webkit/WebSettings.html#setUse...
and
http://developer.android.com/reference/android/webkit/WebSettings.html#setLoa...
?

[aelias_OOO_until_Jul13, 2015-03-06 20:15:51]

Also, everything between "useLegacyBackgroundSizeShorthandBehavior" and
"forceZeroLayoutHeight" inclusively, plus "mainFrameClipsContent" and these two
settings, are exclusively used by Android WebView.  How about moving all of
those settings together into their own "section" (maybe with ######### walls
around it) in this file?

[rune, 2015-03-11 15:24:27]

On 2015/03/06 at 20:15:51, aelias wrote:
> Also, everything between "useLegacyBackgroundSizeShorthandBehavior" and
"forceZeroLayoutHeight" inclusively, plus "mainFrameClipsContent" and these two
settings, are exclusively used by Android WebView.  How about moving all of
those settings together into their own "section" (maybe with ######### walls
around it) in this file?

Done.

[rune, 2015-03-11 15:24:55]

On 2015/03/06 at 20:06:17, aelias wrote:
> lgtm modulo suggestion
> 
> https://codereview.chromium.org/979243005/diff/1/Source/core/frame/Settings.in
> File Source/core/frame/Settings.in (right):
> 
>
https://codereview.chromium.org/979243005/diff/1/Source/core/frame/Settings.i...
> Source/core/frame/Settings.in:236: # For
android.webkit.WebSettings.setLoadWithOverviewMode()
> How about linking
http://developer.android.com/reference/android/webkit/WebSettings.html#setUse...
and
http://developer.android.com/reference/android/webkit/WebSettings.html#setLoa...
?

Done.

[rune, 2015-03-11 15:26:08]

The CQ bit was checked by rune@opera.com

[rune, 2015-03-11 15:26:08]

The patchset sent to the CQ was uploaded after l-g-t-m from aelias@chromium.org
Link to the patchset: https://codereview.chromium.org/979243005/#ps20001 (title:
"Addressed review issues")

[commit-bot: I haz the power, 2015-03-11 15:26:34]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/979243005/20001

[commit-bot: I haz the power, 2015-03-11 18:39:26]

Committed patchset #2 (id:20001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=191722