[Android] Enable spellcheck on non low-end devices by default.

[Android] Enable spellcheck on non low-end devices by default.

Currently spellcheck is being rolled out to all builds of M56 via
finch. This patch ensures spellcheck functionality continues to be
on for M57 and up on non low-end devices (i.e. having > 512MB RAM).
Also ensure that spellcheck can be disabled/enabled on non low-end
devices via chrome://flags.

BUG=629609, 683922
Review-Url: https://codereview.chromium.org/2636843004
Cr-Commit-Position: refs/heads/master@{#445454}
Committed: https://chromium.googlesource.com/chromium/src/+/be9a2649f4d349f42acc658d90cc78ef7b170d87

[timvolodine, 2017-01-17 13:39:30]

The CQ bit was checked by timvolodine@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-01-17 13:39:34]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[timvolodine, 2017-01-17 13:46:22]

timvolodine@chromium.org changed reviewers:
+ groby@chromium.org, tobiasjs@chromium.org

[timvolodine, 2017-01-17 13:46:23]

will need to land this preferably before branch point this week :)

[Tobias Sargeant, 2017-01-17 13:51:59]

lgtm

[commit-bot: I haz the power, 2017-01-17 14:21:49]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-01-17 14:21:50]

Dry run: This issue passed the CQ dry run.

[groby-ooo-7-16, 2017-01-17 14:44:06]

On 2017/01/17 13:51:59, Tobias Sargeant wrote:
> lgtm

lgtm

[timvolodine, 2017-01-19 20:27:55]

Description was changed from

==========
[Android] Enable spellcheck on non low-end devices by default.

Currently spellcheck is being rolled out to all builds of M56 via
finch. This patch ensures spellcheck functionality continues to be
on for M57 and up on non low-end devices (i.e. having > 512MB RAM).
Also ensure that spellcheck can be enabled via chrome://flags on any
device.

BUG=629609
==========

to

==========
[Android] Enable spellcheck on non low-end devices by default.

Currently spellcheck is being rolled out to all builds of M56 via
finch. This patch ensures spellcheck functionality continues to be
on for M57 and up on non low-end devices (i.e. having > 512MB RAM).
Also ensure that spellcheck can be disabled/enabled on non low-end
devices via chrome://flags.

BUG=629609
==========

[timvolodine, 2017-01-19 20:30:18]

thanks for the quick turnaround!

[ also updated about_flags to allow disabling spellcheck if a user doesn't like
it for some reason -- will never happen of course ;) ]

[timvolodine, 2017-01-19 20:30:23]

The CQ bit was checked by timvolodine@chromium.org

[timvolodine, 2017-01-19 20:30:23]

The patchset sent to the CQ was uploaded after l-g-t-m from groby@chromium.org,
tobiasjs@chromium.org
Link to the patchset: https://codereview.chromium.org/2636843004/#ps60001
(title: "keep the logic")

[commit-bot: I haz the power, 2017-01-19 20:30:46]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-01-19 21:57:39]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-01-19 21:57:40]

Try jobs failed on following builders:
  android_n5x_swarming_rel on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_n5x_...)

[timvolodine, 2017-01-19 22:09:26]

The CQ bit was checked by timvolodine@chromium.org

[commit-bot: I haz the power, 2017-01-19 22:10:04]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-01-19 23:44:19]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-01-19 23:44:19]

Try jobs failed on following builders:
  linux_android_rel_ng on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[timvolodine, 2017-01-20 01:39:55]

The CQ bit was checked by timvolodine@chromium.org

[timvolodine, 2017-01-20 01:40:19]

The CQ bit was unchecked by timvolodine@chromium.org

[timvolodine, 2017-01-23 17:52:20]

The CQ bit was checked by timvolodine@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-01-23 17:52:42]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[timvolodine, 2017-01-23 17:53:09]

timvolodine@chromium.org changed reviewers:
+ isherman@chromium.org

[timvolodine, 2017-01-23 17:53:09]

+isherman@: for histograms.xml

[timvolodine, 2017-01-23 17:54:48]

Description was changed from

==========
[Android] Enable spellcheck on non low-end devices by default.

Currently spellcheck is being rolled out to all builds of M56 via
finch. This patch ensures spellcheck functionality continues to be
on for M57 and up on non low-end devices (i.e. having > 512MB RAM).
Also ensure that spellcheck can be disabled/enabled on non low-end
devices via chrome://flags.

BUG=629609
==========

to

==========
[Android] Enable spellcheck on non low-end devices by default.

Currently spellcheck is being rolled out to all builds of M56 via
finch. This patch ensures spellcheck functionality continues to be
on for M57 and up on non low-end devices (i.e. having > 512MB RAM).
Also ensure that spellcheck can be disabled/enabled on non low-end
devices via chrome://flags.

BUG=629609,683922
==========

[commit-bot: I haz the power, 2017-01-23 19:07:55]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-01-23 19:07:57]

Dry run: This issue passed the CQ dry run.

[Ilya Sherman, 2017-01-23 20:02:57]

histograms.xml lgtm

[Ilya Sherman, 2017-01-23 20:03:11]

The CQ bit was checked by isherman@chromium.org

[Ilya Sherman, 2017-01-23 20:03:12]

The patchset sent to the CQ was uploaded after l-g-t-m from groby@chromium.org,
tobiasjs@chromium.org
Link to the patchset: https://codereview.chromium.org/2636843004/#ps80001
(title: "update histograms.xml")

[commit-bot: I haz the power, 2017-01-23 20:04:28]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-01-23 20:10:31]

CQ is committing da patch.

Bot data: {"patchset_id": 80001, "attempt_start_ts": 1485201791719470,
"parent_rev": "f5600ef4dc75558dd888842e742df522550fefc5", "commit_rev":
"be9a2649f4d349f42acc658d90cc78ef7b170d87"}

[commit-bot: I haz the power, 2017-01-23 20:11:02]

Description was changed from

==========
[Android] Enable spellcheck on non low-end devices by default.

Currently spellcheck is being rolled out to all builds of M56 via
finch. This patch ensures spellcheck functionality continues to be
on for M57 and up on non low-end devices (i.e. having > 512MB RAM).
Also ensure that spellcheck can be disabled/enabled on non low-end
devices via chrome://flags.

BUG=629609,683922
==========

to

==========
[Android] Enable spellcheck on non low-end devices by default.

Currently spellcheck is being rolled out to all builds of M56 via
finch. This patch ensures spellcheck functionality continues to be
on for M57 and up on non low-end devices (i.e. having > 512MB RAM).
Also ensure that spellcheck can be disabled/enabled on non low-end
devices via chrome://flags.

BUG=629609,683922

Review-Url: https://codereview.chromium.org/2636843004
Cr-Commit-Position: refs/heads/master@{#445454}
Committed:
https://chromium.googlesource.com/chromium/src/+/be9a2649f4d349f42acc658d90cc...
==========

[commit-bot: I haz the power, 2017-01-23 20:11:03]

Committed patchset #5 (id:80001) as
https://chromium.googlesource.com/chromium/src/+/be9a2649f4d349f42acc658d90cc...