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

Unified Diff: ios/web/web_state/web_state_impl.mm

Issue 2737353006: Replaced webPageOrderedClose with WebStateDelegate API. (Closed)
Patch Set: Created 3 years, 9 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: ios/web/web_state/web_state_impl.mm
diff --git a/ios/web/web_state/web_state_impl.mm b/ios/web/web_state/web_state_impl.mm
index 7bd6758722a6a1c4ee415e3316a3dd44ce5ce8a5..e41fdeab7e0b3a0f1112eb053d836dadc9d69633 100644
--- a/ios/web/web_state/web_state_impl.mm
+++ b/ios/web/web_state/web_state_impl.mm
@@ -514,6 +514,12 @@ WebState* WebStateImpl::CreateNewWebState(const GURL& url,
return nullptr;
}
+void WebStateImpl::CloseWebState() {
+ if (delegate_) {
+ return delegate_->CloseWebState(this);
rohitrao (ping after 24h) 2017/03/13 12:30:02 Any need for "return" if this method returns void?
Eugene But (OOO till 7-30) 2017/03/14 01:20:24 Done.
+ }
+}
+
void WebStateImpl::OnAuthRequired(
NSURLProtectionSpace* protection_space,
NSURLCredential* proposed_credential,

Powered by Google App Engine
This is Rietveld 408576698