Revert of Added DCHECK for monotonicTimeToDOMHighResTimeStamp in PerformanceBase

Revert of Added DCHECK for monotonicTimeToDOMHighResTimeStamp in PerformanceBase (patchset #4 id:80001 of https://codereview.chromium.org/2622283007/ )

Reason for revert:
* in chrome on android, go to google.com
* click on sign in at top left
* start typing in the input box for user name (I was trying to type "webviewteam")
* crash

Fix: potentially get rid of the DCHECK and add an if statement to make sure the output time is never negative.

Original issue's description:
> Added DCHECK for monotonicTimeToDOMHighResTimeStamp in PerformanceBase. A few tests in FirstMeaningfulPaintDetectorTest break the DCHECK. Need to reset the initial value of their mock time function to be timeOrigin that was set in PerformanceBase.
> 
> BUG=680623
> CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
> 
> Review-Url: https://codereview.chromium.org/2622283007
> Cr-Commit-Position: refs/heads/master@{#444134}
> Committed: https://chromium.googlesource.com/chromium/src/+/eeda089e4c29b7ed01c16c7e95987b28cf8c5620

TBR=panicker@chromium.org,ksakamoto@chromium.org,skobes@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=680623
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2

Review-Url: https://codereview.chromium.org/2655123002
Cr-Commit-Position: refs/heads/master@{#446267}
Committed: https://chromium.googlesource.com/chromium/src/+/6468f57d306b277ce67c2ed44e5eff4caa8d7286

[sunjian, 2017-01-25 21:17:57]

The CQ bit was checked by sunjian@chromium.org

[sunjian, 2017-01-25 21:17:58]

Created Revert of Added DCHECK for monotonicTimeToDOMHighResTimeStamp in
PerformanceBase

[commit-bot: I haz the power, 2017-01-25 21:18: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-01-25 21:18:25]

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

[commit-bot: I haz the power, 2017-01-25 21:18:26]

No L-G-T-M from a valid reviewer yet. 
CQ run can only be started by full committers or once the patch has
received an L-G-T-M from a full committer.
Even if an L-G-T-M may have been provided, it was from a non-committer,
_not_ a full super star committer.
See http://www.chromium.org/getting-involved/become-a-committer
Note that this has nothing to do with OWNERS files.

[skobes, 2017-01-25 21:19:09]

lgtm

[sunjian, 2017-01-25 21:19:38]

The CQ bit was checked by sunjian@chromium.org

[panicker, 2017-01-25 21:19:45]

lgtm

[commit-bot: I haz the power, 2017-01-25 21:20:32]

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-01-26 00:00:43]

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

[commit-bot: I haz the power, 2017-01-26 00:00:44]

Try jobs failed on following builders:
  linux_android_rel_ng on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[sunjian, 2017-01-26 01:18:33]

Description was changed from

==========
Revert of Added DCHECK for monotonicTimeToDOMHighResTimeStamp in PerformanceBase
(patchset #4 id:80001 of https://codereview.chromium.org/2622283007/ )

Reason for revert:
* in chrome on android, go to google.com
* click on sign in at top left
* start typing in the input box for user name (I was trying to type
"webviewteam")
* crash

Fix: potentially get rid of the DCHECK and add an if statement to make sure the
output time is never negative.

Original issue's description:
> Added DCHECK for monotonicTimeToDOMHighResTimeStamp in PerformanceBase. A few
tests in FirstMeaningfulPaintDetectorTest break the DCHECK. Need to reset the
initial value of their mock time function to be timeOrigin that was set in
PerformanceBase.
> 
> BUG=680623
>
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
> 
> Review-Url: https://codereview.chromium.org/2622283007
> Cr-Commit-Position: refs/heads/master@{#444134}
> Committed:
https://chromium.googlesource.com/chromium/src/+/eeda089e4c29b7ed01c16c7e9598...

TBR=panicker@chromium.org,ksakamoto@chromium.org,skobes@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=680623
==========

to

==========
Revert of Added DCHECK for monotonicTimeToDOMHighResTimeStamp in PerformanceBase
(patchset #4 id:80001 of https://codereview.chromium.org/2622283007/ )

Reason for revert:
* in chrome on android, go to google.com
* click on sign in at top left
* start typing in the input box for user name (I was trying to type
"webviewteam")
* crash

Fix: potentially get rid of the DCHECK and add an if statement to make sure the
output time is never negative.

Original issue's description:
> Added DCHECK for monotonicTimeToDOMHighResTimeStamp in PerformanceBase. A few
tests in FirstMeaningfulPaintDetectorTest break the DCHECK. Need to reset the
initial value of their mock time function to be timeOrigin that was set in
PerformanceBase.
> 
> BUG=680623
>
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
> 
> Review-Url: https://codereview.chromium.org/2622283007
> Cr-Commit-Position: refs/heads/master@{#444134}
> Committed:
https://chromium.googlesource.com/chromium/src/+/eeda089e4c29b7ed01c16c7e9598...

TBR=panicker@chromium.org,ksakamoto@chromium.org,skobes@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=680623
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

[sunjian, 2017-01-26 01:19:13]

The CQ bit was checked by sunjian@chromium.org

[sunjian, 2017-01-26 01:19:13]

The patchset sent to the CQ was uploaded after l-g-t-m from skobes@chromium.org,
panicker@chromium.org
Link to the patchset: https://codereview.chromium.org/2655123002/#ps100001
(title: "sync")

[commit-bot: I haz the power, 2017-01-26 01:20: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, 2017-01-26 03:13:45]

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

[commit-bot: I haz the power, 2017-01-26 03:13:47]

Try jobs failed on following builders:
  win_chromium_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[Kunihiko Sakamoto, 2017-01-26 05:44:38]

The CQ bit was checked by ksakamoto@chromium.org

[Kunihiko Sakamoto, 2017-01-26 05:44:38]

lgtm

[commit-bot: I haz the power, 2017-01-26 05:45:03]

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-01-26 07:24:58]

CQ is committing da patch.

Bot data: {"patchset_id": 100001, "attempt_start_ts": 1485409478241510,
"parent_rev": "791c54b0a42f4915e712dc66ed981596214d19dd", "commit_rev":
"6468f57d306b277ce67c2ed44e5eff4caa8d7286"}

[commit-bot: I haz the power, 2017-01-26 07:25:36]

Description was changed from

==========
Revert of Added DCHECK for monotonicTimeToDOMHighResTimeStamp in PerformanceBase
(patchset #4 id:80001 of https://codereview.chromium.org/2622283007/ )

Reason for revert:
* in chrome on android, go to google.com
* click on sign in at top left
* start typing in the input box for user name (I was trying to type
"webviewteam")
* crash

Fix: potentially get rid of the DCHECK and add an if statement to make sure the
output time is never negative.

Original issue's description:
> Added DCHECK for monotonicTimeToDOMHighResTimeStamp in PerformanceBase. A few
tests in FirstMeaningfulPaintDetectorTest break the DCHECK. Need to reset the
initial value of their mock time function to be timeOrigin that was set in
PerformanceBase.
> 
> BUG=680623
>
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
> 
> Review-Url: https://codereview.chromium.org/2622283007
> Cr-Commit-Position: refs/heads/master@{#444134}
> Committed:
https://chromium.googlesource.com/chromium/src/+/eeda089e4c29b7ed01c16c7e9598...

TBR=panicker@chromium.org,ksakamoto@chromium.org,skobes@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=680623
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

to

==========
Revert of Added DCHECK for monotonicTimeToDOMHighResTimeStamp in PerformanceBase
(patchset #4 id:80001 of https://codereview.chromium.org/2622283007/ )

Reason for revert:
* in chrome on android, go to google.com
* click on sign in at top left
* start typing in the input box for user name (I was trying to type
"webviewteam")
* crash

Fix: potentially get rid of the DCHECK and add an if statement to make sure the
output time is never negative.

Original issue's description:
> Added DCHECK for monotonicTimeToDOMHighResTimeStamp in PerformanceBase. A few
tests in FirstMeaningfulPaintDetectorTest break the DCHECK. Need to reset the
initial value of their mock time function to be timeOrigin that was set in
PerformanceBase.
> 
> BUG=680623
>
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
> 
> Review-Url: https://codereview.chromium.org/2622283007
> Cr-Commit-Position: refs/heads/master@{#444134}
> Committed:
https://chromium.googlesource.com/chromium/src/+/eeda089e4c29b7ed01c16c7e9598...

TBR=panicker@chromium.org,ksakamoto@chromium.org,skobes@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=680623
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2

Review-Url: https://codereview.chromium.org/2655123002
Cr-Commit-Position: refs/heads/master@{#446267}
Committed:
https://chromium.googlesource.com/chromium/src/+/6468f57d306b277ce67c2ed44e5e...
==========

[commit-bot: I haz the power, 2017-01-26 07:25:37]

Committed patchset #2 (id:100001) as
https://chromium.googlesource.com/chromium/src/+/6468f57d306b277ce67c2ed44e5e...