ProfileErrorBrowserTest should wait for layout.

chrome/browser/ui/profile_error_browsertest.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 <string>
 
 #include "base/files/file_path.h"
 #include "base/files/file_util.h"
 #include "base/path_service.h"
 #include "base/strings/string_util.h"
 #include "base/test/histogram_tester.h"
+#include "chrome/browser/ui/browser.h"
 #include "chrome/browser/ui/simple_message_box_internal.h"
+#include "chrome/browser/ui/tabs/tab_strip_model.h"
 #include "chrome/common/chrome_constants.h"
 #include "chrome/common/chrome_paths.h"
 #include "chrome/test/base/in_process_browser_test.h"
 #include "chrome/test/base/testing_profile.h"
 #include "chrome/test/base/ui_test_utils.h"
+#include "content/public/test/browser_test_utils.h"
 
 namespace {
 
 class ProfileErrorBrowserTest : public InProcessBrowserTest,
                                 public testing::WithParamInterface<bool> {
   // A fixture that allows testing histograms reporting when faced with a
   // corrupted profile. The boolean parameter forces the creation of an empty or
   // corrupted profile, allowing to test both the corruption case and that what
   // it is testing indeed happens differently when not under corruption.
  public:
@@ skipping 48 matching lines @@
 #else
 #define MAYBE_CorruptedProfile CorruptedProfile
 #endif
 
 IN_PROC_BROWSER_TEST_P(ProfileErrorBrowserTest, MAYBE_CorruptedProfile) {
   const char kPaintHistogram[] = "Startup.FirstWebContents.NonEmptyPaint";
   const char kLoadHistogram[] = "Startup.FirstWebContents.MainFrameLoad";
 
   // Navigate to a URL so the first non-empty paint is registered.
   ui_test_utils::NavigateToURL(browser(), GURL("http://www.example.com/"));
+
+  content::WebContents* contents =
+      browser()->tab_strip_model()->GetActiveWebContents();
+
+  // Wait for the page to produce a frame, the first visually non-empty paint
+  // metric is not valid until then.
+  bool loaded = false;
+  ASSERT_TRUE(content::ExecuteScriptAndExtractBool(
+      contents,
+      "requestAnimationFrame(function() {"
+      "  window.domAutomationController.send(true);"
+      "});",
+      &loaded));
+  ASSERT_TRUE(loaded);
+
   if (do_corrupt_) {
     histogram_tester_.ExpectTotalCount(kPaintHistogram, 0);
     histogram_tester_.ExpectTotalCount(kLoadHistogram, 0);
   } else {
     histogram_tester_.ExpectTotalCount(kPaintHistogram, 1);
     histogram_tester_.ExpectTotalCount(kLoadHistogram, 1);
   }
 }
 
 INSTANTIATE_TEST_CASE_P(ProfileErrorBrowserTestInstance,
                         ProfileErrorBrowserTest,
                         testing::Bool());
 
 }  // namespace