Fix for reverted cl http://codereview.chromium.org/147123

A recent change broke the load times. It also revealed some deficiencies.
This adds a new time marker for when a load is committed, which is
a more interesting value than the start of the load (which we still keep).
Also, the first layout time wasn't an interesting time to keep, instead
we keep the time of the first paint.
The histograms were modified to use the new values when appropriate.

Review URL: http://codereview.chromium.org/149099

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@19465 0039d316-1c4b-4281-b951-d872f2087c98

2 files changed, 0 insertions, 9 deletions

diff --git a/webkit/glue/webframeloaderclient_impl.cc b/webkit/glue/webframeloaderclient_impl.cc
index 5def74f..c886b80 100644
--- a/webkit/glue/webframeloaderclient_impl.cc
+++ b/webkit/glue/webframeloaderclient_impl.cc
@@ -777,10 +777,6 @@ void WebFrameLoaderClient::dispatchDidFinishLoad() {
 }
 
 void WebFrameLoaderClient::dispatchDidFirstLayout() {
-  WebViewImpl* webview = webframe_->GetWebViewImpl();
-  WebViewDelegate* d = webview->delegate();
-  if (d)
-    d->DidFirstLayout(webview, webframe_);
 }
 
 void WebFrameLoaderClient::dispatchDidFirstVisuallyNonEmptyLayout() {
diff --git a/webkit/glue/webview_delegate.h b/webkit/glue/webview_delegate.h
index 94fd4f5..917c9d1 100644
--- a/webkit/glue/webview_delegate.h
+++ b/webkit/glue/webview_delegate.h
@@ -363,11 +363,6 @@ class WebViewDelegate : virtual public WebWidgetDelegate {
   virtual void DidFinishDocumentLoadForFrame(WebView* webview, WebFrame* frame) {
   }
 
-  // Called after layout runs for the first time after a new document is loaded
-  // into a frame.  All resources have not necessarily finished loading.
-  virtual void DidFirstLayout(WebView* webview, WebFrame* frame) {
-  }
-
   // This method is called when we load a resource from an in-memory cache.
   // A return value of |false| indicates the load should proceed, but WebCore
   // appears to largely ignore the return value.