Speculatively deflake BrowsingHistoryHandler::MergeDuplicateResults on Android.

Speculatively deflake BrowsingHistoryHandler::MergeDuplicateResults on Android.

This CL removes global time variables in browsing history tests, one of
which is unused. These variables were tripping a NOTREACHED() in the
implementation of base::Time::LocalMidnight() on Android, where the time
was being computed as null.

This speculatively deflakes the test on Android.

BUG=611747
TBR=dbeam@chromium.org

Review-Url: https://codereview.chromium.org/2704283002
Cr-Commit-Position: refs/heads/master@{#451919}
Committed: https://chromium.googlesource.com/chromium/src/+/899bc281768913fa13729754b9725d1feadd3c9c

[dominickn, 2017-02-21 03:00:18]

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

[commit-bot: I haz the power, 2017-02-21 03:00:28]

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

[dominickn, 2017-02-21 03:26:19]

Description was changed from

==========
Deflake BrowsingHistoryHandler::MergeDuplicateResults on Android.

This CL removes global time variables in browsing history tests, one of
which is unused. These variables were tripping a NOTREACHED() in the
implementation of base::Time::LocalMidnight() on Android, where the time
was being computed as null.

As a positive side effect, this deflakes the test on Android.

BUG=611747
==========

to

==========
Speculatively deflake BrowsingHistoryHandler::MergeDuplicateResults on Android.

This CL removes global time variables in browsing history tests, one of
which is unused. These variables were tripping a NOTREACHED() in the
implementation of base::Time::LocalMidnight() on Android, where the time
was being computed as null.

This speculatively deflakes the test on Android.

BUG=611747
==========

[dominickn, 2017-02-21 03:48:23]

dominickn@chromium.org changed reviewers:
+ dbeam@chromium.org, sdefresne@chromium.org

[dominickn, 2017-02-21 03:48:24]

sdefresne: PTAL at history
dbeam: PTAL at webui

Thanks!

[commit-bot: I haz the power, 2017-02-21 03:58:27]

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

[commit-bot: I haz the power, 2017-02-21 03:58:28]

Dry run: This issue passed the CQ dry run.

[sdefresne, 2017-02-21 09:51:01]

lgtm

[dominickn, 2017-02-22 06:51:49]

Description was changed from

==========
Speculatively deflake BrowsingHistoryHandler::MergeDuplicateResults on Android.

This CL removes global time variables in browsing history tests, one of
which is unused. These variables were tripping a NOTREACHED() in the
implementation of base::Time::LocalMidnight() on Android, where the time
was being computed as null.

This speculatively deflakes the test on Android.

BUG=611747
==========

to

==========
Speculatively deflake BrowsingHistoryHandler::MergeDuplicateResults on Android.

This CL removes global time variables in browsing history tests, one of
which is unused. These variables were tripping a NOTREACHED() in the
implementation of base::Time::LocalMidnight() on Android, where the time
was being computed as null.

This speculatively deflakes the test on Android.

BUG=611747
TBR=dbeam@chromium.org
==========

[dominickn, 2017-02-22 06:52:16]

Since this is just removing an unused variable in a webui test, TBR-ing dbeam.

[dominickn, 2017-02-22 06:52:25]

The CQ bit was checked by dominickn@chromium.org

[commit-bot: I haz the power, 2017-02-22 06:53:42]

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-02-22 08:08:34]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1487746345582150, "parent_rev":
"719470136b9ee03e656d8db35b63c876c00fd647", "commit_rev":
"899bc281768913fa13729754b9725d1feadd3c9c"}

[commit-bot: I haz the power, 2017-02-22 08:09:21]

Description was changed from

==========
Speculatively deflake BrowsingHistoryHandler::MergeDuplicateResults on Android.

This CL removes global time variables in browsing history tests, one of
which is unused. These variables were tripping a NOTREACHED() in the
implementation of base::Time::LocalMidnight() on Android, where the time
was being computed as null.

This speculatively deflakes the test on Android.

BUG=611747
TBR=dbeam@chromium.org
==========

to

==========
Speculatively deflake BrowsingHistoryHandler::MergeDuplicateResults on Android.

This CL removes global time variables in browsing history tests, one of
which is unused. These variables were tripping a NOTREACHED() in the
implementation of base::Time::LocalMidnight() on Android, where the time
was being computed as null.

This speculatively deflakes the test on Android.

BUG=611747
TBR=dbeam@chromium.org

Review-Url: https://codereview.chromium.org/2704283002
Cr-Commit-Position: refs/heads/master@{#451919}
Committed:
https://chromium.googlesource.com/chromium/src/+/899bc281768913fa13729754b972...
==========

[commit-bot: I haz the power, 2017-02-22 08:09:23]

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