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

components/html_viewer/html_frame.cc

 // 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.
 
 #include "components/html_viewer/html_frame.h"
 
 #include <algorithm>
 #include <limits>
 
 #include "base/bind.h"
 #include "base/single_thread_task_runner.h"
 #include "base/stl_util.h"
 #include "base/strings/string_util.h"
 #include "base/thread_task_runner_handle.h"
 #include "components/html_viewer/ax_provider_impl.h"
 #include "components/html_viewer/blink_basic_type_converters.h"
 #include "components/html_viewer/blink_input_events_type_converters.h"
 #include "components/html_viewer/blink_text_input_type_converters.h"
 #include "components/html_viewer/blink_url_request_type_converters.h"
 #include "components/html_viewer/geolocation_client_impl.h"
 #include "components/html_viewer/global_state.h"
 #include "components/html_viewer/html_frame_delegate.h"
 #include "components/html_viewer/html_frame_properties.h"
 #include "components/html_viewer/html_frame_tree_manager.h"
 #include "components/html_viewer/media_factory.h"
+#include "components/html_viewer/stats_collection_controller.h"
 #include "components/html_viewer/touch_handler.h"
 #include "components/html_viewer/web_layer_impl.h"
 #include "components/html_viewer/web_layer_tree_view_impl.h"
 #include "components/html_viewer/web_storage_namespace_impl.h"
 #include "components/html_viewer/web_url_loader_impl.h"
 #include "components/view_manager/ids.h"
 #include "components/view_manager/public/cpp/scoped_view_ptr.h"
 #include "components/view_manager/public/cpp/view.h"
 #include "components/view_manager/public/cpp/view_manager.h"
 #include "components/view_manager/public/interfaces/surfaces.mojom.h"
@@ skipping 152 matching lines @@
     NOTREACHED();
   }
 
   if (!IsLocal()) {
     blink::WebRemoteFrame* remote_web_frame = web_frame_->toWebRemoteFrame();
     if (remote_web_frame) {
       remote_web_frame->setReplicatedOrigin(state_.origin);
       remote_web_frame->setReplicatedName(state_.name);
     }
   }
+
+  startup_performance_data_collector_ =

[sky, 2015/08/17 20:09:30]

Do we really need this for all frames? Should it only be on the top most frame,
and only if that frame is local? As an example of why it matters
didHandleOnloadEvents won't be called for remote frames.

[msw, 2015/08/17 21:25:50]

Done; thanks for the advice!

+      StatsCollectionController::Install(web_frame_, GetLocalRootApp());
 }
 
 void HTMLFrame::OnViewUnembed() {
   // The view we're associated with is being unembedded (the view on the server
   // side still exists, but we're no longer responsible for rendering to it).
   // If we're local, we need to swap the frame to remote, unless we're at the
   // top of the hierarchy, in which case someone else is going to render the
   // tree.
   if (!IsLocal() || frame_tree_manager_->local_root_ == this)
     return;
@@ skipping 536 matching lines @@
 }
 
 void HTMLFrame::didAddMessageToConsole(const blink::WebConsoleMessage& message,
                                        const blink::WebString& source_name,
                                        unsigned source_line,
                                        const blink::WebString& stack_trace) {
   VLOG(1) << "[" << source_name.utf8() << "(" << source_line << ")] "
           << message.text.utf8();
 }
 
+void HTMLFrame::didHandleOnloadEvents(blink::WebLocalFrame* frame) {
+  if (startup_performance_data_collector_.get()) {

[yzshen1, 2015/08/17 05:46:38]

nit: .get() is not necessary.

[msw, 2015/08/17 21:25:50]

Done.

+    startup_performance_data_collector_->SetFirstWebContentsMainFrameLoadTime(
+        base::Time::Now().ToInternalValue());
+  }
+}
+
 void HTMLFrame::didFinishLoad(blink::WebLocalFrame* frame) {
   if (GetLocalRoot() == this)
     delegate_->OnFrameDidFinishLoad();
 }
 
 void HTMLFrame::didNavigateWithinPage(blink::WebLocalFrame* frame,
                                       const blink::WebHistoryItem& history_item,
                                       blink::WebHistoryCommitType commit_type) {
   GetLocalRoot()->server_->DidNavigateLocally(id_,
                                               history_item.urlString().utf8());
 }
 
+void HTMLFrame::didFirstVisuallyNonEmptyLayout(blink::WebLocalFrame* frame) {
+  if (startup_performance_data_collector_) {
+    startup_performance_data_collector_->SetFirstVisuallyNonEmptyLayoutTime(
+        base::Time::Now().ToInternalValue());
+  }
+}
+
 blink::WebGeolocationClient* HTMLFrame::geolocationClient() {
   if (!geolocation_client_impl_)
     geolocation_client_impl_.reset(new GeolocationClientImpl);
   return geolocation_client_impl_.get();
 }
 
 blink::WebEncryptedMediaClient* HTMLFrame::encryptedMediaClient() {
   return global_state()->media_factory()->GetEncryptedMediaClient();
 }
 
@@ skipping 102 matching lines @@
 
 void HTMLFrame::reload(bool ignore_cache, bool is_client_redirect) {
   NOTIMPLEMENTED();
 }
 
 void HTMLFrame::forwardInputEvent(const blink::WebInputEvent* event) {
   NOTIMPLEMENTED();
 }
 
 }  // namespace mojo