Add WebRuntimeFeatures::enableInputModeAttribute

Add WebRuntimeFeatures::enableInputModeAttribute

Chromium will use this function to enable inputmode attribute of <input> or <text> Element.

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

[yoichio, 2013-08-22 04:43:22]

Please take a look.

[tkent, 2013-08-22 04:49:02]

> Chrome will use this function to enable inputmode attribute of <input> or
<text> Element as trial.

Do you mean Chromium?
'as trial' is unclear.  Why do you need this function in addition to
WebRuntimeFeatures::enableTestOnlyFeatures()?

[yoichio, 2013-08-22 05:18:16]

On 2013/08/22 04:49:02, tkent wrote:
> > Chrome will use this function to enable inputmode attribute of <input> or
> <text> Element as trial.
> 
> Do you mean Chromium?
> 'as trial' is unclear.  Why do you need this function in addition to
> WebRuntimeFeatures::enableTestOnlyFeatures()?

We will use this function at content browser tests.
Content browser tests runs not under WebRuntimeFeatures::enableTestOnlyFeatures.
If we enable all test only features, some tests that suppose test only features
is disabled will fail.
I don't get which such a test is, but to enable only inputmode attribute feature
is less flaky than to enable 
all test features.

[yoichio, 2013-08-22 05:18:42]

On 2013/08/22 05:18:16, yoichio wrote:
> On 2013/08/22 04:49:02, tkent wrote:
> > > Chrome will use this function to enable inputmode attribute of <input> or
> > <text> Element as trial.
> > 
> > Do you mean Chromium?
> > 'as trial' is unclear.  Why do you need this function in addition to
> > WebRuntimeFeatures::enableTestOnlyFeatures()?
> 
> We will use this function at content browser tests.
https://chromiumcodereview.appspot.com/23264032/

> Content browser tests runs not under
WebRuntimeFeatures::enableTestOnlyFeatures.
> If we enable all test only features, some tests that suppose test only
features
> is disabled will fail.
> I don't get which such a test is, but to enable only inputmode attribute
feature
> is less flaky than to enable 
> all test features.

[tkent, 2013-08-22 05:22:15]

lgtm

It makes sense.

[commit-bot: I haz the power, 2013-08-22 05:22:55]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/yoichio@chromium.org/23112014/1

[commit-bot: I haz the power, 2013-08-22 06:43:21]

Change committed as 156548