Revert "Use WebRenderingStats::Enumerate to populate benchmarking extension renderingStats() fields…

Revert "Use WebRenderingStats::Enumerate to populate benchmarking extension renderingStats() fields."

BUG=156851
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=163128

[brianderson, 2012-10-19 18:54:30]

https://codereview.chromium.org/11229007/diff/1/content/renderer/gpu/gpu_benc...
File content/renderer/gpu/gpu_benchmarking_extension.cc (right):

https://codereview.chromium.org/11229007/diff/1/content/renderer/gpu/gpu_benc...
content/renderer/gpu/gpu_benchmarking_extension.cc:187:
v8::Integer::New(stats.totalCommitCount));
A straight up revert compiles, but leaves out implThreadScrollCount and
mainThreadScrollCount which have been added recently. Will resolve this in next
patch and double check the gpu_stats as well.

[dtu, 2012-10-19 19:09:06]

lgtm

lgtm

[jamesr, 2012-10-19 19:13:31]

lgtm2

[commit-bot: I haz the power, 2012-10-19 20:00:36]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/brianderson@chromium.org/11229007/5001

[commit-bot: I haz the power, 2012-10-19 20:00:39]

Presubmit check for 11229007-5001 failed and returned exit status 1.


Running presubmit commit checks ...

** Presubmit ERRORS **
Missing LGTM from an OWNER for files in these directories:
    content/common/gpu

Presubmit checks took 1.0s to calculate.

Was the presubmit check useful? Please send feedback & hate mail to
maruel@chromium.org!

[brianderson, 2012-10-19 20:02:57]

Adding apatrick to review content/renderer/gpu changes.

[brianderson, 2012-10-19 20:27:35]

Al is on vacation, adding Ken for OWNER's review instead.

[Ken Russell (switch to Gerrit), 2012-10-19 21:35:12]

lgtm

[commit-bot: I haz the power, 2012-10-19 21:36:27]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/brianderson@chromium.org/11229007/5001

[commit-bot: I haz the power, 2012-10-19 22:54:29]

Retried try job too often for step(s) sync_integration_tests

[commit-bot: I haz the power, 2012-10-20 00:18:24]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/brianderson@chromium.org/11229007/5001

[commit-bot: I haz the power, 2012-10-20 02:08:03]

Change committed as 163128