Removes wifi dependency for testHistoryEntriesStatusCell.

Removes wifi dependency for testHistoryEntriesStatusCell, so that it
passes reliably when running on devices with WIFI turned off.

The test, was checking the web contents of the URL as the last step
which is unnecessary. This test is meant to test the history entries,
and that the right number of tabs are opened in the end. The
contents of the URL are not important.

BUG=709135
Review-Url: https://codereview.chromium.org/2806253002
Cr-Commit-Position: refs/heads/master@{#463441}
Committed: https://chromium.googlesource.com/chromium/src/+/ac504ddefaadc3b933db8cffde4a599539c89b79

[ramyasharma, 2017-04-10 06:50:13]

Description was changed from

==========
Removes wifi dependency for testHistoryEntriesStatusCell.

testHistoryEntriesStatusCell

BUG=709135
==========

to

==========
Removes wifi dependency for testHistoryEntriesStatusCell, so that it
passes reliably when running on devices with WIFI turned off.

BUG=709135
==========

[ramyasharma, 2017-04-10 06:52:00]

Description was changed from

==========
Removes wifi dependency for testHistoryEntriesStatusCell, so that it
passes reliably when running on devices with WIFI turned off.

BUG=709135
==========

to

==========
Removes wifi dependency for testHistoryEntriesStatusCell, so that it
passes reliably when running on devices with WIFI turned off.

The test, was checking the web contents of the URL as the last step
which is unnecessary. This test is meant to test the history entries,
and that the right number of tabs are opened in the end. The
contents of the URL are not important.

BUG=709135
==========

[ramyasharma, 2017-04-10 06:55:18]

ramyasharma@chromium.org changed reviewers:
+ sczs@chromium.org

[ramyasharma, 2017-04-10 06:58:06]

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

[commit-bot: I haz the power, 2017-04-10 06:58:22]

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

[ramyasharma, 2017-04-10 07:00:09]

Patchset #1 (id:1) has been deleted

[commit-bot: I haz the power, 2017-04-10 07:08:59]

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

[commit-bot: I haz the power, 2017-04-10 07:09:00]

Dry run: This issue passed the CQ dry run.

[sczs, 2017-04-10 18:45:01]

lgtm

Now we will check if a new Tab was opened but not if the "correct" tab was
opened, right? (Just want to make sure)

I think that's a good tradeoff since we can't rely on WIFI, I guess is better to
test some of it than none of it.

[ramyasharma, 2017-04-10 23:14:03]

On 2017/04/10 18:45:01, sczs wrote:
> lgtm
> 
> Now we will check if a new Tab was opened but not if the "correct" tab was
> opened, right? (Just want to make sure)
> 
> I think that's a good tradeoff since we can't rely on WIFI, I guess is better
to
> test some of it than none of it.

Yes, that is correct. We only check if a new tab is opened, rather than checking
the contents of the new tab.

[ramyasharma, 2017-04-10 23:14:14]

The CQ bit was checked by ramyasharma@chromium.org

[commit-bot: I haz the power, 2017-04-10 23:14:56]

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-04-10 23:22:10]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1491866054047940,
"parent_rev": "857cc7b94a374decb02c7c98a3b84dd14ec6a275", "commit_rev":
"ac504ddefaadc3b933db8cffde4a599539c89b79"}

[commit-bot: I haz the power, 2017-04-10 23:23:09]

Description was changed from

==========
Removes wifi dependency for testHistoryEntriesStatusCell, so that it
passes reliably when running on devices with WIFI turned off.

The test, was checking the web contents of the URL as the last step
which is unnecessary. This test is meant to test the history entries,
and that the right number of tabs are opened in the end. The
contents of the URL are not important.

BUG=709135
==========

to

==========
Removes wifi dependency for testHistoryEntriesStatusCell, so that it
passes reliably when running on devices with WIFI turned off.

The test, was checking the web contents of the URL as the last step
which is unnecessary. This test is meant to test the history entries,
and that the right number of tabs are opened in the end. The
contents of the URL are not important.

BUG=709135

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

[commit-bot: I haz the power, 2017-04-10 23:23:10]

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