Convert GpuBenchmarkingExtension to a gin::Wrappable class

Convert GpuBenchmarkingExtension to a gin::Wrappable class

v8 extensions are slowing down context creation and we're trying to get
rid of them.

BUG=334679
R=ernstm@chromium.org,nduca@chromium.org

Committed: https://crrev.com/26f63992473dda66e4ed2b22f35d4ccd0702ddcd
Cr-Commit-Position: refs/heads/master@{#300957}

[jochen (gone - plz use gerrit), 2014-10-09 14:57:01]

ptal

looks scary, but it's really only moving the handling of optional arguments from
JS to C++ and gin has slightly different argument parsing...

[jochen (gone - plz use gerrit), 2014-10-09 14:58:34]

I ran smoothness.top_25 locally and that seems to work.

Is there a way to trigger other perf tests for this CL?

[jochen (gone - plz use gerrit), 2014-10-09 15:09:34]

On 2014/10/09 at 14:58:34, jochen wrote:
> I ran smoothness.top_25 locally and that seems to work.
> 
> Is there a way to trigger other perf tests for this CL?

also ran smoothness.tough_pinch_zoom_cases

[jochen (gone - plz use gerrit), 2014-10-09 18:20:35]

I guess the regular trybots already exercise some of this, and I still have some
bugs...

[ernstm, 2014-10-09 21:41:27]

On 2014/10/09 18:20:35, jochen wrote:
> I guess the regular trybots already exercise some of this, and I still have
some
> bugs...

You should verify that rasterize_and_record_micro.top_25 and
repaint.key_mobile_sites still work. We do have smoke tests for those, but
checking if they still produce reasonable metrics might be a good idea.

[jochen (gone - plz use gerrit), 2014-10-10 09:42:30]

+hajimehoshi/kouhei fyi about crbug.com422244

[jochen (gone - plz use gerrit), 2014-10-22 07:54:20]

ptal

I checked with kouhei that disabling the inspector test is ok until the
underlying issue is fixed.

I ran repaint.key_mobile_sites which looks sane.
rasterize_and_record_micro.top_25 fails for me with and without the patch - it's
complaining about wrong chrome version. Is it support to work with content_shell
at all?

[ernstm, 2014-10-23 20:44:36]

On 2014/10/22 07:54:20, jochen (OOO real soon now) wrote:
> ptal
> 
> I checked with kouhei that disabling the inspector test is ok until the
> underlying issue is fixed.
> 
> I ran repaint.key_mobile_sites which looks sane.
> rasterize_and_record_micro.top_25 fails for me with and without the patch -
it's
> complaining about wrong chrome version. Is it support to work with
content_shell
> at all?

rasterize_and_record_micro benchmark checks for a specific chrome version. It
won't work with content_shell. But you are covered by the benchmark smoke tests
and measurement unit tests in telemetry, and those have passed. LGTM.

[jochen (gone - plz use gerrit), 2014-10-23 20:48:33]

The CQ bit was checked by jochen@chromium.org

[commit-bot: I haz the power, 2014-10-23 20:50:05]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/647433003/160001

[commit-bot: I haz the power, 2014-10-23 22:00:09]

Committed patchset #3 (id:160001)

[commit-bot: I haz the power, 2014-10-23 22:00:57]

Patchset 3 (id:??) landed as
https://crrev.com/26f63992473dda66e4ed2b22f35d4ccd0702ddcd
Cr-Commit-Position: refs/heads/master@{#300957}