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

Unified Diff: components/web_contents_delegate_android/web_contents_delegate_android.cc

Issue 23264020: Move Android to use new Popup Blocker API. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Cleanup Created 7 years, 4 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
« chrome/chrome_browser_ui.gypi ('K') | « chrome/chrome_browser_ui.gypi ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/web_contents_delegate_android/web_contents_delegate_android.cc
diff --git a/components/web_contents_delegate_android/web_contents_delegate_android.cc b/components/web_contents_delegate_android/web_contents_delegate_android.cc
index 8f599a16b03bcb97f0e50d73b32dc5f7f20474b1..bad6730d0726263235caef03bf73b78be98cdea5 100644
--- a/components/web_contents_delegate_android/web_contents_delegate_android.cc
+++ b/components/web_contents_delegate_android/web_contents_delegate_android.cc
@@ -144,6 +144,10 @@ void WebContentsDelegateAndroid::AddNewContents(
NULL,
user_gesture);
}
+
+ if (was_blocked)
+ *was_blocked = false;
joth 2013/08/20 22:44:36 seems like this should be !handled instead of fal
David Trainor- moved to gerrit 2013/08/21 17:25:13 I wasn't sure if whether or not we handled it actu
+
if (!handled)
delete new_contents;
}
« chrome/chrome_browser_ui.gypi ('K') | « chrome/chrome_browser_ui.gypi ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698