Remove PermissionQueueController queueing checks in PermissionNavigationTest

Remove PermissionQueueController queueing checks in PermissionNavigationTest

When we enable the PermissionRequestManager on Android, the queueing in
the PermissionQueueController will no longer be relevant, as different
permission types will be queued by the PRM. To make the flag flip easier
this patch removes the checks in PermissionNavigationTest for queueing,
which would otherwise need to be removed at the same time as the flip.

BUG=606138
Review-Url: https://codereview.chromium.org/2947013003
Cr-Commit-Position: refs/heads/master@{#481127}
Committed: https://chromium.googlesource.com/chromium/src/+/95f41c431169a0dc380402e6cccb5a48e3fb486a

[Timothy Loh, 2017-06-21 03:22:44]

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

[commit-bot: I haz the power, 2017-06-21 03:22:50]

Dry run: CQ is trying da patch.

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

[Timothy Loh, 2017-06-21 03:30:50]

timloh@chromium.org changed reviewers:
+ dominickn@chromium.org

[Timothy Loh, 2017-06-21 03:30:51]

Maybe I should've planned ahead more and not added these checks in the test...

[dominickn, 2017-06-21 03:36:25]

On 2017/06/21 03:30:51, Timothy Loh wrote:
> Maybe I should've planned ahead more and not added these checks in the test...

Do you think it's worth keeping these and conditionally checking the queue
length if the PRM enabling flag is on?

[Timothy Loh, 2017-06-21 03:51:03]

On 2017/06/21 03:36:25, dominickn wrote:
> On 2017/06/21 03:30:51, Timothy Loh wrote:
> > Maybe I should've planned ahead more and not added these checks in the
test...
> 
> Do you think it's worth keeping these and conditionally checking the queue
> length if the PRM enabling flag is on?

I don't think it really buys us anything extra to leave it on since we want to
enable the flag soon.

[dominickn, 2017-06-21 04:05:42]

Fair enough. lgtm

[commit-bot: I haz the power, 2017-06-21 04:14:10]

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

[commit-bot: I haz the power, 2017-06-21 04:14:11]

Dry run: This issue passed the CQ dry run.

[Timothy Loh, 2017-06-21 06:37:02]

The CQ bit was checked by timloh@chromium.org

[commit-bot: I haz the power, 2017-06-21 06:37:24]

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-06-21 06:42:19]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1498027022421150, "parent_rev":
"99455058b030b33f127b6d0e897c2d13c955400a", "commit_rev":
"95f41c431169a0dc380402e6cccb5a48e3fb486a"}

[commit-bot: I haz the power, 2017-06-21 06:42:34]

Description was changed from

==========
Remove PermissionQueueController queueing checks in PermissionNavigationTest

When we enable the PermissionRequestManager on Android, the queueing in
the PermissionQueueController will no longer be relevant, as different
permission types will be queued by the PRM. To make the flag flip easier
this patch removes the checks in PermissionNavigationTest for queueing,
which would otherwise need to be removed at the same time as the flip.

BUG=606138
==========

to

==========
Remove PermissionQueueController queueing checks in PermissionNavigationTest

When we enable the PermissionRequestManager on Android, the queueing in
the PermissionQueueController will no longer be relevant, as different
permission types will be queued by the PRM. To make the flag flip easier
this patch removes the checks in PermissionNavigationTest for queueing,
which would otherwise need to be removed at the same time as the flip.

BUG=606138

Review-Url: https://codereview.chromium.org/2947013003
Cr-Commit-Position: refs/heads/master@{#481127}
Committed:
https://chromium.googlesource.com/chromium/src/+/95f41c431169a0dc380402e6cccb...
==========

[commit-bot: I haz the power, 2017-06-21 06:42:35]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/95f41c431169a0dc380402e6cccb...