Add UMA histogram DomDistiller.Time.RunJavaScript

components/dom_distiller/content/distiller_page_web_contents.cc

 // Copyright 2013 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/dom_distiller/content/distiller_page_web_contents.h"
 
 #include "base/callback.h"
 #include "base/memory/scoped_ptr.h"
+#include "base/metrics/histogram.h"
 #include "base/strings/utf_string_conversions.h"
 #include "components/dom_distiller/content/web_contents_main_frame_observer.h"
 #include "components/dom_distiller/core/distiller_page.h"
 #include "components/dom_distiller/core/dom_distiller_service.h"
 #include "content/public/browser/browser_context.h"
 #include "content/public/browser/navigation_controller.h"
 #include "content/public/browser/render_frame_host.h"
 #include "content/public/browser/render_view_host.h"
 #include "content/public/browser/web_contents.h"
 #include "content/public/browser/web_contents_observer.h"
@@ skipping 138 matching lines @@
 void DistillerPageWebContents::DidFailLoad(
     content::RenderFrameHost* render_frame_host,
     const GURL& validated_url,
     int error_code,
     const base::string16& error_description) {
   if (!render_frame_host->GetParent()) {
     content::WebContentsObserver::Observe(NULL);
     DCHECK(state_ == LOADING_PAGE || state_ == EXECUTING_JAVASCRIPT);
     state_ = PAGELOAD_FAILED;
     scoped_ptr<base::Value> empty = base::Value::CreateNullValue();
-    OnWebContentsDistillationDone(GURL(), empty.get());
+    OnWebContentsDistillationDone(GURL(), base::TimeTicks(), empty.get());
   }
 }
 
 void DistillerPageWebContents::ExecuteJavaScript() {
   content::RenderFrameHost* frame =
       source_page_handle_->web_contents()->GetMainFrame();
   DCHECK(frame);
   DCHECK_EQ(LOADING_PAGE, state_);
   state_ = EXECUTING_JAVASCRIPT;
   content::WebContentsObserver::Observe(NULL);
   // Stop any pending navigation since the intent is to distill the current
   // page.
   source_page_handle_->web_contents()->Stop();
   DVLOG(1) << "Beginning distillation";
   frame->ExecuteJavaScript(
       base::UTF8ToUTF16(script_),
       base::Bind(&DistillerPageWebContents::OnWebContentsDistillationDone,
                  base::Unretained(this),
-                 source_page_handle_->web_contents()->GetLastCommittedURL()));
+                 source_page_handle_->web_contents()->GetLastCommittedURL(),
+                 base::TimeTicks::Now()));
 }
 
 void DistillerPageWebContents::OnWebContentsDistillationDone(
     const GURL& page_url,
+    const base::TimeTicks& javascript_start,
     const base::Value* value) {
   DCHECK(state_ == IDLE || state_ == LOADING_PAGE ||  // TODO(nyquist): 493795.
          state_ == PAGELOAD_FAILED || state_ == EXECUTING_JAVASCRIPT);
   state_ = IDLE;
+
+  if (!javascript_start.is_null()) {
+    base::TimeDelta javascript_time = base::TimeTicks::Now() - javascript_start;
+    UMA_HISTOGRAM_TIMES("DomDistiller.Time.RunJavaScript", javascript_time);
+    DVLOG(1) << "DomDistiller.Time.RunJavaScript = " << javascript_time;
+  }
+
   DistillerPage::OnDistillationDone(page_url, value);
 }
 
 }  // namespace dom_distiller