Convert the SkiaBenchmarkingExtension to a gin::Wrappable class

content/renderer/render_view_impl.cc

Index: content/renderer/render_view_impl.cc
diff --git a/content/renderer/render_view_impl.cc b/content/renderer/render_view_impl.cc
index 78feab5013be884eba5f44391cb19df9e52dcac2..079c415971c021792ba376393fc441b3c3ea3dd0 100644
--- a/content/renderer/render_view_impl.cc
+++ b/content/renderer/render_view_impl.cc
@@ -116,6 +116,7 @@
 #include "content/renderer/renderer_webcolorchooser_impl.h"
 #include "content/renderer/resizing_mode_selector.h"
 #include "content/renderer/savable_resources.h"
+#include "content/renderer/skia_benchmarking_extension.h"
 #include "content/renderer/speech_recognition_dispatcher.h"
 #include "content/renderer/stats_collection_controller.h"
 #include "content/renderer/stats_collection_observer.h"
@@ -3642,11 +3643,18 @@ void RenderViewImpl::didClearWindowObject(WebFrame* frame, int world_id) {
   FOR_EACH_OBSERVER(RenderViewObserver, observers_,
                     DidClearWindowObject(frame, world_id));
 
-  if ((enabled_bindings_ & BINDINGS_POLICY_DOM_AUTOMATION) && (world_id == 0))
-    DomAutomationController::Install(this, frame);
+  if (world_id == 0) {
+    if ((enabled_bindings_ & BINDINGS_POLICY_DOM_AUTOMATION))
+      DomAutomationController::Install(this, frame);
 
-  if (enabled_bindings_ & BINDINGS_POLICY_STATS_COLLECTION)
-    StatsCollectionController::Install(frame);
+    if (enabled_bindings_ & BINDINGS_POLICY_STATS_COLLECTION)
+      StatsCollectionController::Install(frame);

[piman, 2014/01/21 20:59:31]

I'm not sure what this does. Did you mean to include it the if (world_id == 0)?
It sounds like a change that's unrelated to the skia benchmarking.

[jochen (gone - plz use gerrit), 2014/01/22 12:17:05]

yes, I forgot about moving the stats controller under the if in my previous
change here.

+
+    const CommandLine& command_line = *CommandLine::ForCurrentProcess();
+
+    if (command_line.HasSwitch(switches::kEnableSkiaBenchmarking))
+      SkiaBenchmarking::Install(frame);
+  }
 }
 
 void RenderViewImpl::didCreateDocumentElement(WebFrame* frame) {