Add stats collection for telemetry startup.warm.blank_page test.

components/html_viewer/stats_collection_controller.h

-// Copyright 2013 The Chromium Authors. All rights reserved.
+// Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
-#ifndef CONTENT_RENDERER_STATS_COLLECTION_CONTROLLER_H_
-#define CONTENT_RENDERER_STATS_COLLECTION_CONTROLLER_H_
+#ifndef COMPONENTS_HTML_VIEWER_STATS_COLLECTION_CONTROLLER_H_
+#define COMPONENTS_HTML_VIEWER_STATS_COLLECTION_CONTROLLER_H_
 
 #include "base/basictypes.h"
+#include "components/html_viewer/stats_collection_observer.h"
 #include "gin/wrappable.h"
+#include "mojo/services/tracing/tracing.mojom.h"
 
 namespace blink {
 class WebFrame;
 }
 
-namespace content {
+namespace mojo {
+class ApplicationImpl;
+}
+
+namespace html_viewer {
 
 // This class is exposed in JS as window.statsCollectionController and provides
 // functionality to read out statistics from the browser.
 // Its use must be enabled specifically via the
 // --enable-stats-collection-bindings command line flag.
 class StatsCollectionController
     : public gin::Wrappable<StatsCollectionController> {
  public:
   static gin::WrapperInfo kWrapperInfo;
 
-  static void Install(blink::WebFrame* frame);
+  static StatsCollectionController* Install(HTMLDocument* html_document,

[sky, 2015/08/12 23:54:36]

Can you make this take WebFrame, which would work with both HTMLDocument types?

[yzshen1, 2015/08/13 15:59:12]

Please comment about the lifespan and ownership of the returned value.

[msw, 2015/08/14 23:20:35]

Done.

[msw, 2015/08/14 23:20:35]

Done.

+                                            mojo::ApplicationImpl* app);
+
+  // Record timing for various blink::WebFrameClient events.
+  void DidHandleOnloadEvents(const base::Time& time);
+  void DidFirstVisuallyNonEmptyLayout(const base::Time& time);
 
  private:
-  StatsCollectionController();
+  StatsCollectionController(HTMLDocument* html_document,
+                            mojo::ApplicationImpl* app);
   ~StatsCollectionController() override;
 
   // gin::WrappableBase
   gin::ObjectTemplateBuilder GetObjectTemplateBuilder(
       v8::Isolate* isolate) override;
 
   // Retrieves a histogram and returns a JSON representation of it.
   std::string GetHistogram(const std::string& histogram_name);
 
-  // Retrieves a histogram from the browser process and returns a JSON
-  // representation of it.
+  // Retrieves a browser histogram and returns a JSON representation of it.
   std::string GetBrowserHistogram(const std::string& histogram_name);
 
   // Returns JSON representation of tab timing information for the current tab.
   std::string GetTabLoadTiming();
 
+  StatsCollectionObserver observer_;

[yzshen1, 2015/08/13 15:59:12]

Maybe consider merging the observer into this class?

[msw, 2015/08/14 23:20:34]

Removed.

+  tracing::StartupPerformanceControllerPtr startup_performance_controller_;
+
   DISALLOW_COPY_AND_ASSIGN(StatsCollectionController);
 };
 
-}  // namespace content
+}  // namespace html_viewer
 
-#endif  // CONTENT_RENDERER_STATS_COLLECTION_CONTROLLER_H_
+#endif  // COMPONENTS_HTML_VIEWER_STATS_COLLECTION_CONTROLLER_H_