[Prerender] first contentful paint histograms.

chrome/browser/prerender/prerender_test_utils.cc

 // Copyright (c) 2012 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 "chrome/browser/prerender/prerender_test_utils.h"
 
 #include <memory>
 #include <set>
 #include <string>
 #include <utility>
 #include <vector>
 
 #include "base/command_line.h"
-#include "base/strings/string_split.h"
+#include "base/memory/ptr_util.h"
+#include "base/run_loop.h"
 #include "base/strings/utf_string_conversions.h"
 #include "chrome/browser/loader/chrome_resource_dispatcher_host_delegate.h"
 #include "chrome/browser/prerender/prerender_manager.h"
 #include "chrome/browser/prerender/prerender_manager_factory.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/safe_browsing/local_database_manager.h"
 #include "chrome/browser/ui/browser.h"
 #include "chrome/browser/ui/tabs/tab_strip_model.h"
 #include "chrome/common/pref_names.h"
 #include "chrome/grit/generated_resources.h"
@@ skipping 395 matching lines @@
   contents_ = nullptr;
   final_status_ = contents->final_status();
   stopped_ = true;
   stop_loop_.Quit();
   // If there is a WaitForLoads call and it has yet to see the expected number
   // of loads, stop the loop so the test fails instead of timing out.
   if (load_waiter_)
     load_waiter_->Quit();
 }
 
+// static
+FirstContentfulPaintManagerWaiter* FirstContentfulPaintManagerWaiter::Create(
+    PrerenderManager* manager) {
+  auto fcp_waiter = base::WrapUnique(new FirstContentfulPaintManagerWaiter());

[pasko, 2016/11/17 19:52:16]

WrapUnique is discouraged, is MakeUnique somehow not suitable here?

[mattcary, 2016/11/18 09:21:11]

The constructor for FirstContentfulPaintManagerWaiter is private in order to
ensure that it's only created via this factory function, which means it's not
visible to the unique_ptr<FCPMW> class.

IIRC the unique_ptr discussions, I think this is the most common reason for
WrapUnique. The other may be some sort of polymorphism issue.

+  auto fcp_waiter_ptr = fcp_waiter.get();
+  manager->AddObserver(std::move(fcp_waiter));
+  return fcp_waiter_ptr;
+}
+
+FirstContentfulPaintManagerWaiter::FirstContentfulPaintManagerWaiter()
+    : saw_fcp_(false) {}
+
+FirstContentfulPaintManagerWaiter::~FirstContentfulPaintManagerWaiter() {}
+
+void FirstContentfulPaintManagerWaiter::OnFirstContentfulPaint() {
+  saw_fcp_ = true;
+  if (waiter_)
+    content::BrowserThread::PostTask(content::BrowserThread::UI, FROM_HERE,

[pasko, 2016/11/17 19:52:16]

Confused.

I assumed this class lives on UI thread (please add a comment in the .h to
clarify this). If so, why posting a task into UI thread from UI thread? (I know
it can be necessary, but would like to know whether it was actually necessary)

[mattcary, 2016/11/18 09:21:11]

This is how I think things are happening. I may very well be wrong, so please
correct/improve the idiom:

on UI thread, browser test calls NavigateToDestURL
  ->renderer starts doing its thing in another process
fcp_waiter->Wait() called on UI thread
  ->a message loop is created which blocks

In renderer, the FCP happens. A timing update is sent to the browser, which then
runs OnFirstContentfulPaint in the UI thread
 -> we post the quit closure to our spinning message loop in the same thread

Is it possible to just call waiter_->Quit() directly? I don't have a solid
understanding of Chrome's message loops.

+                                     waiter_->QuitClosure());
+}
+
+void FirstContentfulPaintManagerWaiter::Wait() {
+  if (saw_fcp_)
+    return;
+  waiter_ = base::MakeUnique<base::RunLoop>();
+  waiter_->Run();
+  waiter_.reset();
+}
+
 TestPrerenderContentsFactory::TestPrerenderContentsFactory() {}
 
 TestPrerenderContentsFactory::~TestPrerenderContentsFactory() {
   EXPECT_TRUE(expected_contents_queue_.empty());
 }
 
 std::unique_ptr<TestPrerender>
 TestPrerenderContentsFactory::ExpectPrerenderContents(
     FinalStatus final_status) {
   std::unique_ptr<TestPrerender> handle(new TestPrerender());
@@ skipping 212 matching lines @@
 void CreateMockInterceptorOnIO(const GURL& url, const base::FilePath& file) {
   CHECK(content::BrowserThread::CurrentlyOn(content::BrowserThread::IO));
   net::URLRequestFilter::GetInstance()->AddUrlInterceptor(
       url, net::URLRequestMockHTTPJob::CreateInterceptorForSingleFile(
                file, content::BrowserThread::GetBlockingPool()));
 }
 
 }  // namespace test_utils
 
 }  // namespace prerender