Wrap NavigateToURL(...) calls in ASSERT_TRUE or EXPECT_TRUE.
Chrome Large Scale Change doc covering these changes:
https://docs.google.com/document/d/1wwbUlsk5o9fsURHWyiQVdvjiJBTx9sIqA6vFcF330Pc/edit?usp=sharing
Change-Id: I0f239337d09dd85d6b2017124b63048cac5fc1c7
AX-Relnotes: n/a.
Bug: 1246568
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3140783
Commit-Queue: Łukasz Anforowicz <[email protected]>
Reviewed-by: Avi Drissman <[email protected]>
Owners-Override: Avi Drissman <[email protected]>
Cr-Commit-Position: refs/heads/main@{#919098}
diff --git a/chrome/browser/iframe_browsertest.cc b/chrome/browser/iframe_browsertest.cc
index 02658c19..6b0928ea 100644
--- a/chrome/browser/iframe_browsertest.cc
+++ b/chrome/browser/iframe_browsertest.cc
@@ -25,7 +25,7 @@
GURL url = ui_test_utils::GetTestUrl(
base::FilePath(), base::FilePath().AppendASCII(file));
- ui_test_utils::NavigateToURL(browser(), url);
+ ASSERT_TRUE(ui_test_utils::NavigateToURL(browser(), url));
EXPECT_EQ(base::ASCIIToUTF16(page_title),
browser()->tab_strip_model()->GetActiveWebContents()->GetTitle());
}
@@ -54,8 +54,8 @@
// Note: For the bug to occur, the parent and child frame need to be in
// the same site, otherwise they would each get a RenderWidgetHost and
// existing code will properly clear the internal state.
- ui_test_utils::NavigateToURL(browser(),
- embedded_test_server()->GetURL("/iframe.html"));
+ ASSERT_TRUE(ui_test_utils::NavigateToURL(
+ browser(), embedded_test_server()->GetURL("/iframe.html")));
NavigateIframeToURL(tab, "test", file_input_url);
// Invoke the file chooser and remove the iframe from the main document.
