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

Unified Diff: ios/web/public/web_state/web_state_delegate_bridge.h

Issue 2074733002: Add public API for handling Javascript alerts and prompt. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Cleanup. Created 4 years, 6 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/public/web_state/web_state_delegate_bridge.h
diff --git a/ios/web/public/web_state/web_state_delegate_bridge.h b/ios/web/public/web_state/web_state_delegate_bridge.h
index d43e6077cf0388d649ec0b4e0acfbf801b97bb03..6120cc1dee129f1c65c3abc67ff0c3497f783b25 100644
--- a/ios/web/public/web_state/web_state_delegate_bridge.h
+++ b/ios/web/public/web_state/web_state_delegate_bridge.h
@@ -25,6 +25,11 @@
- (BOOL)webState:(web::WebState*)webState
handleContextMenu:(const web::ContextMenuParams&)params;
+// Returns a pointer to a service to manage dialogs. May return nil in which
Eugene But (OOO till 7-30) 2016/06/27 21:09:37 NIT: s/nil/null nil is a specific value for Object
michaeldo 2016/06/27 22:25:10 Done.
+// case dialogs aren't shown.
+- (web::JavaScriptDialogPresenter*)javaScriptDialogPresenterForWebState:
+ (web::WebState*)webState;
+
@end
namespace web {
@@ -39,6 +44,8 @@ class WebStateDelegateBridge : public web::WebStateDelegate {
void LoadProgressChanged(WebState* source, double progress) override;
bool HandleContextMenu(WebState* source,
const ContextMenuParams& params) override;
+ JavaScriptDialogPresenter* GetJavaScriptDialogPresenter(
+ WebState* source) override;
private:
// CRWWebStateDelegate which receives forwarded calls.

Powered by Google App Engine
This is Rietveld 408576698