Fix app data clearing timeouts for PersonalDataManagerTest.

Fix app data clearing timeouts for PersonalDataManagerTest.

Was using the wrong context from InstrumentationRegistry in
ClearAppDataTestRule, so now that's fixed.

Also enables the temporarily disabled tests now that they work again.

BUG=739575
Review-Url: https://codereview.chromium.org/2970263002
Cr-Commit-Position: refs/heads/master@{#485001}
Committed: https://chromium.googlesource.com/chromium/src/+/f0f0169b8e343d9cd4b75fdde7124a475b6fab72

[troyhildebrandt, 2017-07-06 17:37:09]

Description was changed from

==========
Increase timeout in ApplicationData.clearAppData because of test timeouts.

BUG=739575
==========

to

==========
Fix app data clearing timeouts for PersonalDataManagerTest.

Adds a private inner test rule to PersonalDataManagerTest for the autofill
test helper, so that we can properly order the initialization steps using
RuleChain.

BUG=739575
==========

[troyhildebrandt, 2017-07-06 17:38:36]

Description was changed from

==========
Fix app data clearing timeouts for PersonalDataManagerTest.

Adds a private inner test rule to PersonalDataManagerTest for the autofill
test helper, so that we can properly order the initialization steps using
RuleChain.

BUG=739575
==========

to

==========
Fix app data clearing timeouts for PersonalDataManagerTest.

Adds a private inner test rule to PersonalDataManagerTest for the autofill
test helper, so that we can properly order the initialization steps using
RuleChain.

BUG=739575
==========

[troyhildebrandt, 2017-07-06 21:42:37]

Description was changed from

==========
Fix app data clearing timeouts for PersonalDataManagerTest.

Adds a private inner test rule to PersonalDataManagerTest for the autofill
test helper, so that we can properly order the initialization steps using
RuleChain.

BUG=739575
==========

to

==========
Fix app data clearing timeouts for PersonalDataManagerTest.

Was using the wrong context from InstrumentationRegistry in
ClearAppDataTestRule, so now that's fixed.

BUG=739575
==========

[troyhildebrandt, 2017-07-07 15:20:54]

Description was changed from

==========
Fix app data clearing timeouts for PersonalDataManagerTest.

Was using the wrong context from InstrumentationRegistry in
ClearAppDataTestRule, so now that's fixed.

BUG=739575
==========

to

==========
Fix app data clearing timeouts for PersonalDataManagerTest.

Was using the wrong context from InstrumentationRegistry in
ClearAppDataTestRule, so now that's fixed.

Also enables the temporarily disabled tests now that they work again.

BUG=739575
==========

[troyhildebrandt, 2017-07-07 17:13:38]

Description was changed from

==========
Fix app data clearing timeouts for PersonalDataManagerTest.

Was using the wrong context from InstrumentationRegistry in
ClearAppDataTestRule, so now that's fixed.

Also enables the temporarily disabled tests now that they work again.

BUG=739575
==========

to

==========
Fix app data clearing timeouts for PersonalDataManagerTest.

Was using the wrong context from InstrumentationRegistry in
ClearAppDataTestRule, so now that's fixed.

Also enables the temporarily disabled tests now that they work again.

BUG=739575
==========

[troyhildebrandt, 2017-07-07 17:13:38]

thildebr@chromium.org changed reviewers:
+ yfriedman@chromium.org

[troyhildebrandt, 2017-07-07 17:55:09]

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

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

Dry run: CQ is trying da patch.

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

[Yaron, 2017-07-07 18:12:05]

lgtm

[troyhildebrandt, 2017-07-07 18:16:14]

thildebr@chromium.org changed reviewers:
+ avayvod@chromium.org

[commit-bot: I haz the power, 2017-07-07 18:39:14]

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

[commit-bot: I haz the power, 2017-07-07 18:39:15]

Dry run: This issue passed the CQ dry run.

[whywhat, 2017-07-07 18:53:22]

lgtm

[troyhildebrandt, 2017-07-07 19:06:42]

The CQ bit was checked by thildebr@chromium.org

[commit-bot: I haz the power, 2017-07-07 19:07:02]

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-07-07 19:12:10]

CQ is committing da patch.

Bot data: {"patchset_id": 60001, "attempt_start_ts": 1499454402703390,
"parent_rev": "cc73643302ad7ea271c063fb21fbb8c047cf837e", "commit_rev":
"f0f0169b8e343d9cd4b75fdde7124a475b6fab72"}

[commit-bot: I haz the power, 2017-07-07 19:12:28]

Description was changed from

==========
Fix app data clearing timeouts for PersonalDataManagerTest.

Was using the wrong context from InstrumentationRegistry in
ClearAppDataTestRule, so now that's fixed.

Also enables the temporarily disabled tests now that they work again.

BUG=739575
==========

to

==========
Fix app data clearing timeouts for PersonalDataManagerTest.

Was using the wrong context from InstrumentationRegistry in
ClearAppDataTestRule, so now that's fixed.

Also enables the temporarily disabled tests now that they work again.

BUG=739575

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

[commit-bot: I haz the power, 2017-07-07 19:12:29]

Committed patchset #4 (id:60001) as
https://chromium.googlesource.com/chromium/src/+/f0f0169b8e343d9cd4b75fdde712...