Add tests for DocumentModeAssassin#isOptedOutOfDocumentMode()

Add tests for DocumentModeAssassin#isOptedOutOfDocumentMode()

BUG=646146
Committed: https://crrev.com/2595aa00b5c63681da6665a22f7eb75f24459cb2
Cr-Commit-Position: refs/heads/master@{#418954}

[Theresa, 2016-09-15 01:04:56]

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

[commit-bot: I haz the power, 2016-09-15 01:05:22]

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

[Theresa, 2016-09-15 01:08:09]

twellington@chromium.org changed reviewers:
+ dfalcantara@chromium.org

[Theresa, 2016-09-15 01:08:10]

ptal

[commit-bot: I haz the power, 2016-09-15 02:09:55]

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

[commit-bot: I haz the power, 2016-09-15 02:09:56]

Dry run: This issue passed the CQ dry run.

[gone, 2016-09-15 17:50:49]

lgtm

https://codereview.chromium.org/2337143004/diff/1/chrome/android/javatests/sr...
File
chrome/android/javatests/src/org/chromium/chrome/browser/tabmodel/DocumentModeAssassinTest.java
(right):

https://codereview.chromium.org/2337143004/diff/1/chrome/android/javatests/sr...
chrome/android/javatests/src/org/chromium/chrome/browser/tabmodel/DocumentModeAssassinTest.java:646:
sharedPreferencesEditor.remove(DocumentModeAssassin.OPT_OUT_STATE);
Should explicitly put that this depends on MIGRATION_ON_UPGRADE_ATTEMPTED being
set to true.

[Theresa, 2016-09-15 19:53:08]

https://codereview.chromium.org/2337143004/diff/1/chrome/android/javatests/sr...
File
chrome/android/javatests/src/org/chromium/chrome/browser/tabmodel/DocumentModeAssassinTest.java
(right):

https://codereview.chromium.org/2337143004/diff/1/chrome/android/javatests/sr...
chrome/android/javatests/src/org/chromium/chrome/browser/tabmodel/DocumentModeAssassinTest.java:646:
sharedPreferencesEditor.remove(DocumentModeAssassin.OPT_OUT_STATE);
On 2016/09/15 17:50:49, dfalcantara wrote:
> Should explicitly put that this depends on MIGRATION_ON_UPGRADE_ATTEMPTED
being
> set to true.

Done.

[Theresa, 2016-09-15 19:53:51]

The CQ bit was checked by twellington@chromium.org

[Theresa, 2016-09-15 19:53:51]

The patchset sent to the CQ was uploaded after l-g-t-m from
dfalcantara@chromium.org
Link to the patchset: https://codereview.chromium.org/2337143004/#ps20001
(title: "Add tests for DocumentModeAssassin#isOptedOutOfDocumentMode()")

[commit-bot: I haz the power, 2016-09-15 19:54:17]

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, 2016-09-15 20:30:54]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-09-15 20:33:22]

Description was changed from

==========
Add tests for DocumentModeAssassin#isOptedOutOfDocumentMode()

BUG=646146
==========

to

==========
Add tests for DocumentModeAssassin#isOptedOutOfDocumentMode()

BUG=646146

Committed: https://crrev.com/2595aa00b5c63681da6665a22f7eb75f24459cb2
Cr-Commit-Position: refs/heads/master@{#418954}
==========

[commit-bot: I haz the power, 2016-09-15 20:33:22]

Patchset 2 (id:??) landed as
https://crrev.com/2595aa00b5c63681da6665a22f7eb75f24459cb2
Cr-Commit-Position: refs/heads/master@{#418954}