Restore default-to-zero for subsequence caching record time metric.

Restore default-to-zero for subsequence caching record time metric.

http://crrev.com/1552693002 landed as r367465 and added
record_time_subsequence_caching_disabled_ms. The current Android reference build
is 48.0.2564.95 which is built at r359700.

This change is a partial revert of http://crrev.com/2352783004. I've checked the
other TODOs I removed and those look to be fine, with the next-newest being
http://crrev.com/1104433005 as r326586 which should be present in the Android
reference build.

BUG=651200
Committed: https://crrev.com/ff30596cef625e4438c6875f7273d5f323367389
Cr-Commit-Position: refs/heads/master@{#421694}

[wkorman, 2016-09-28 21:11:28]

Description was changed from

==========
Restore default-to-zero for subsequence caching record time metric.

http://crrev.com/1552693002 landed as r367465 and added
record_time_subsequence_caching_disabled_ms. The current Android reference build
is 48.0.2564.95 which is built at r359700.

This change is a partial revert of http://crrev.com/2352783004. I've checked the
other TODOs I removed and those look to be fine, with the next-newest being
http://crrev.com/1104433005 as r326586 which should be present in the Android
reference build.

BUG=651200
==========

to

==========
Restore default-to-zero for subsequence caching record time metric.

http://crrev.com/1552693002 landed as r367465 and added
record_time_subsequence_caching_disabled_ms. The current Android reference build
is 48.0.2564.95 which is built at r359700.

This change is a partial revert of http://crrev.com/2352783004. I've checked the
other TODOs I removed and those look to be fine, with the next-newest being
http://crrev.com/1104433005 as r326586 which should be present in the Android
reference build.

BUG=651200
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.perf:android_s5_perf_cq;master.tryserver.chromium.perf:linux_perf_cq;master.tryserver.chromium.perf:mac_retina_perf_cq;master.tryserver.chromium.perf:winx64_10_perf_cq
==========

[wkorman, 2016-09-28 21:12:01]

wkorman@chromium.org changed reviewers:
+ nednguyen@google.com, sullivan@chromium.org

[nednguyen, 2016-09-28 21:14:17]

lgtm

[wkorman, 2016-09-28 21:17:56]

The CQ bit was checked by wkorman@chromium.org

[commit-bot: I haz the power, 2016-09-28 21:18:34]

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, 2016-09-28 23:19:18]

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

[commit-bot: I haz the power, 2016-09-28 23:19:19]

Try jobs failed on following builders:
  android_s5_perf_cq on master.tryserver.chromium.perf (JOB_TIMED_OUT, no build
URL)
  linux_perf_cq on master.tryserver.chromium.perf (JOB_TIMED_OUT, no build URL)
  mac_retina_perf_cq on master.tryserver.chromium.perf (JOB_TIMED_OUT, no build
URL)
  winx64_10_perf_cq on master.tryserver.chromium.perf (JOB_TIMED_OUT, no build
URL)

[sullivan, 2016-09-29 00:25:05]

Description was changed from

==========
Restore default-to-zero for subsequence caching record time metric.

http://crrev.com/1552693002 landed as r367465 and added
record_time_subsequence_caching_disabled_ms. The current Android reference build
is 48.0.2564.95 which is built at r359700.

This change is a partial revert of http://crrev.com/2352783004. I've checked the
other TODOs I removed and those look to be fine, with the next-newest being
http://crrev.com/1104433005 as r326586 which should be present in the Android
reference build.

BUG=651200
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.perf:android_s5_perf_cq;master.tryserver.chromium.perf:linux_perf_cq;master.tryserver.chromium.perf:mac_retina_perf_cq;master.tryserver.chromium.perf:winx64_10_perf_cq
==========

to

==========
Restore default-to-zero for subsequence caching record time metric.

http://crrev.com/1552693002 landed as r367465 and added
record_time_subsequence_caching_disabled_ms. The current Android reference build
is 48.0.2564.95 which is built at r359700.

This change is a partial revert of http://crrev.com/2352783004. I've checked the
other TODOs I removed and those look to be fine, with the next-newest being
http://crrev.com/1104433005 as r326586 which should be present in the Android
reference build.

BUG=651200
==========

[sullivan, 2016-09-29 00:25:11]

The CQ bit was checked by sullivan@chromium.org

[commit-bot: I haz the power, 2016-09-29 00:25:30]

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, 2016-09-29 00:33:03]

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

[commit-bot: I haz the power, 2016-09-29 00:33:04]

Failed to apply the patch.
On branch working_branch
Your branch is up-to-date with 'origin/refs/pending/heads/master'.
nothing to commit, working tree clean

[commit-bot: I haz the power, 2016-09-29 00:45:13]

Description was changed from

==========
Restore default-to-zero for subsequence caching record time metric.

http://crrev.com/1552693002 landed as r367465 and added
record_time_subsequence_caching_disabled_ms. The current Android reference build
is 48.0.2564.95 which is built at r359700.

This change is a partial revert of http://crrev.com/2352783004. I've checked the
other TODOs I removed and those look to be fine, with the next-newest being
http://crrev.com/1104433005 as r326586 which should be present in the Android
reference build.

BUG=651200
==========

to

==========
Restore default-to-zero for subsequence caching record time metric.

http://crrev.com/1552693002 landed as r367465 and added
record_time_subsequence_caching_disabled_ms. The current Android reference build
is 48.0.2564.95 which is built at r359700.

This change is a partial revert of http://crrev.com/2352783004. I've checked the
other TODOs I removed and those look to be fine, with the next-newest being
http://crrev.com/1104433005 as r326586 which should be present in the Android
reference build.

BUG=651200

Committed: https://crrev.com/ff30596cef625e4438c6875f7273d5f323367389
Cr-Commit-Position: refs/heads/master@{#421694}
==========

[commit-bot: I haz the power, 2016-09-29 00:45:14]

Patchset 1 (id:??) landed as
https://crrev.com/ff30596cef625e4438c6875f7273d5f323367389
Cr-Commit-Position: refs/heads/master@{#421694}