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

Unified Diff: chrome/test/chromedriver/chrome/web_view_impl.h

Issue 1669453002: [chromedriver] Apply page load timeout to slow cross-process navigations (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Remove DEPS modifications. Created 4 years, 10 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: chrome/test/chromedriver/chrome/web_view_impl.h
diff --git a/chrome/test/chromedriver/chrome/web_view_impl.h b/chrome/test/chromedriver/chrome/web_view_impl.h
index ba639b7408e99178a789805d404b1a84775f0bbb..a7d777c6027fa4a40eaaab6030e89ae644e5c21c 100644
--- a/chrome/test/chromedriver/chrome/web_view_impl.h
+++ b/chrome/test/chromedriver/chrome/web_view_impl.h
@@ -51,7 +51,7 @@ class WebViewImpl : public WebView {
Status ConnectIfNecessary() override;
Status HandleReceivedEvents() override;
Status GetUrl(std::string* url) override;
- Status Load(const std::string& url) override;
+ Status Load(const std::string& url, const Timeout* timeout) override;
Status Reload() override;
Status TraverseHistory(int delta) override;
Status EvaluateScript(const std::string& frame,
@@ -83,9 +83,10 @@ class WebViewImpl : public WebView {
Status GetCookies(scoped_ptr<base::ListValue>* cookies) override;
Status DeleteCookie(const std::string& name, const std::string& url) override;
Status WaitForPendingNavigations(const std::string& frame_id,
- const base::TimeDelta& timeout,
+ const Timeout& timeout,
bool stop_load_on_timeout) override;
Status IsPendingNavigation(const std::string& frame_id,
+ const Timeout* timeout,
bool* is_pending) override;
JavaScriptDialogManager* GetJavaScriptDialogManager() override;
Status OverrideGeolocation(const Geoposition& geoposition) override;
@@ -117,6 +118,7 @@ class WebViewImpl : public WebView {
const base::TimeDelta& timeout,
scoped_ptr<base::Value>* result);
Status IsNotPendingNavigation(const std::string& frame_id,
+ const Timeout* timeout,
bool* is_not_pending);
Status InitProfileInternal();

Powered by Google App Engine
This is Rietveld 408576698