[tools/perf] Prepare V8DetachedContextAgeInGCTests for JS API migration

[tools/perf] Prepare V8DetachedContextAgeInGCTests for JS API migration

A previous attempt to land https://codereview.chromium.org/2687773003/
broke the catapult roll due to V8DetachedContextAgeInGCTests failures.

These tests use a FakeTab and poke at the arguments of calls to
EvaluateJavaScript in order to return fake histograms.

The signature and implementaion of the fake method is tweaked in order
to work both with the old and new JS APIs.

BUG=catapult:#3208

Review-Url: https://codereview.chromium.org/2694163003
Cr-Commit-Position: refs/heads/master@{#450341}
Committed: https://chromium.googlesource.com/chromium/src/+/edebc1fbd29714dce0151bfa57a08b49eea5293d

[perezju, 2017-02-14 11:42:04]

perezju@chromium.org changed reviewers:
+ nednguyen@google.com

[perezju, 2017-02-14 11:42:05]

https://codereview.chromium.org/2694163003/diff/1/tools/perf/measurements/v8_...
File tools/perf/measurements/v8_detached_context_age_in_gc_unittest.py (right):

https://codereview.chromium.org/2694163003/diff/1/tools/perf/measurements/v8_...
tools/perf/measurements/v8_detached_context_age_in_gc_unittest.py:32: if
kwargs.get('name') == histogram_name or histogram_name in script:
Tweaked to work both before and after applying the change in:
https://codereview.chromium.org/2687773003/diff/40001/telemetry/telemetry/val...

[perezju, 2017-02-14 11:44:11]

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

[commit-bot: I haz the power, 2017-02-14 11:44:30]

Dry run: 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-14 12:48:06]

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

[commit-bot: I haz the power, 2017-02-14 12:48:07]

Dry run: This issue passed the CQ dry run.

[nednguyen, 2017-02-14 13:46:43]

lgtm

[perezju, 2017-02-14 13:49:42]

The CQ bit was checked by perezju@chromium.org

[commit-bot: I haz the power, 2017-02-14 13:50:04]

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-14 13:53:52]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1487080182960480, "parent_rev":
"8ee279b6b1fbde62f6f5603bc0f8e0c7bb098072", "commit_rev":
"edebc1fbd29714dce0151bfa57a08b49eea5293d"}

[commit-bot: I haz the power, 2017-02-14 13:54:26]

Description was changed from

==========
[tools/perf] Prepare V8DetachedContextAgeInGCTests for JS API migration

A previous attempt to land https://codereview.chromium.org/2687773003/
broke the catapult roll due to V8DetachedContextAgeInGCTests failures.

These tests use a FakeTab and poke at the arguments of calls to
EvaluateJavaScript in order to return fake histograms.

The signature and implementaion of the fake method is tweaked in order
to work both with the old and new JS APIs.

BUG=catapult:#3208
==========

to

==========
[tools/perf] Prepare V8DetachedContextAgeInGCTests for JS API migration

A previous attempt to land https://codereview.chromium.org/2687773003/
broke the catapult roll due to V8DetachedContextAgeInGCTests failures.

These tests use a FakeTab and poke at the arguments of calls to
EvaluateJavaScript in order to return fake histograms.

The signature and implementaion of the fake method is tweaked in order
to work both with the old and new JS APIs.

BUG=catapult:#3208

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

[commit-bot: I haz the power, 2017-02-14 13:54:27]

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