PageLoadMetrics renderer and browser implementation.

content/public/test/mock_navigation_handle.h

+// 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 "content/public/browser/navigation_handle.h"
+
+#include "testing/gmock/include/gmock/gmock.h"
+#include "url/gurl.h"
+
+namespace content {
+
+// MockNavigations are instantiated with a url, and all their boolean
+// methods return false by default. Make sure to change this mock class
+// if you add anything to NavigationHandle.
+class MockNavigationHandle : public NavigationHandle {

[clamy, 2015/09/10 12:16:24]

Sorry I missed this on my last pass on the code. I'm not ok with adding this
testing class to the content/public API. It was a deliberate choice on our part
not to include such a class initially. We found that the navigation logic is
very often modified, and that unit tests that fake navigation observers call
would go out of sync with what really happens when the code is run.

In order to avoid this, we want people to use the various navigation methods of
WebContentsTester and RenderFrameHost tester, as this ensure your unit tests
keep matching the logic inside content. For an example of what it looks like
with a NavigationHandle, please have a look at
https://code.google.com/p/chromium/codesearch#chromium/src/chrome/browser/cap....

+ public:
+  explicit MockNavigationHandle(GURL url);
+  ~MockNavigationHandle() override;
+  MOCK_CONST_METHOD0(GetURL, const GURL&());
+  MOCK_CONST_METHOD0(GetNetErrorCode, net::Error());
+  MOCK_CONST_METHOD0(IsInMainFrame, bool());
+  MOCK_CONST_METHOD0(IsSamePage, bool());
+  MOCK_CONST_METHOD0(HasCommittedDocument, bool());
+  MOCK_CONST_METHOD0(HasCommittedErrorPage, bool());
+
+ private:
+  GURL url_;
+};
+
+} //  namespace content