Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1181)

Unified Diff: content/browser/renderer_host/render_view_host_browsertest.cc

Issue 2641403002: [refactor] - Move IsFocusedElementEditable() and ClearFocusedElement() from RenderViewHost to WebCo… (Closed)
Patch Set: Addressed sky@'s comment Created 3 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: content/browser/renderer_host/render_view_host_browsertest.cc
diff --git a/content/browser/renderer_host/render_view_host_browsertest.cc b/content/browser/renderer_host/render_view_host_browsertest.cc
index 9b260626a01a7fa5dd5d202ee4a108f9fc08e363..6efcbf302d97ffc09f832a7de6db8e62c153f9c7 100644
--- a/content/browser/renderer_host/render_view_host_browsertest.cc
+++ b/content/browser/renderer_host/render_view_host_browsertest.cc
@@ -120,10 +120,10 @@ IN_PROC_BROWSER_TEST_F(RenderViewHostTest, IsFocusedElementEditable) {
GURL test_url = embedded_test_server()->GetURL("/touch_selection.html");
NavigateToURL(shell(), test_url);
- RenderViewHost* rvh = shell()->web_contents()->GetRenderViewHost();
- EXPECT_FALSE(rvh->IsFocusedElementEditable());
+ WebContents* contents = shell()->web_contents();
+ EXPECT_FALSE(contents->IsFocusedElementEditable());
EXPECT_TRUE(ExecuteScript(shell(), "focus_textfield();"));
- EXPECT_TRUE(rvh->IsFocusedElementEditable());
+ EXPECT_TRUE(contents->IsFocusedElementEditable());
}
// Flaky on Linux (https://crbug.com/559192).

Powered by Google App Engine
This is Rietveld 408576698