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

Unified Diff: third_party/WebKit/Source/web/tests/WebViewTest.cpp

Issue 2910233002: Do not fallback to FocusedOrMainFrame() in FocusedLocalFrameInWidget() (Closed)
Patch Set: Fixed a crashing test Created 3 years, 7 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: third_party/WebKit/Source/web/tests/WebViewTest.cpp
diff --git a/third_party/WebKit/Source/web/tests/WebViewTest.cpp b/third_party/WebKit/Source/web/tests/WebViewTest.cpp
index 7f4f3c7952fd4af5ca71aa39109c91397a17e89f..80601d5c161fbedeb98befaa45f1813fffecc50e 100644
--- a/third_party/WebKit/Source/web/tests/WebViewTest.cpp
+++ b/third_party/WebKit/Source/web/tests/WebViewTest.cpp
@@ -2324,6 +2324,7 @@ TEST_P(WebViewTest, FinishComposingTextDoesNotDismissHandles) {
base_url_ + "longpress_selection.html", true);
web_view->Resize(WebSize(500, 300));
web_view->UpdateAllLifecyclePhases();
+ web_view->SetInitialFocus(false);
EhsanK 2017/05/30 22:39:22 Without it line 2332 returns nullptr after the cha
RunPendingTasks();
WebString target = WebString::FromUTF8("target");

Powered by Google App Engine
This is Rietveld 408576698