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

Unified Diff: chrome/browser/ui/browser.cc

Issue 9521013: Remove web intents from Chrome on Android build (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Resolve nits Created 8 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: chrome/browser/ui/browser.cc
diff --git a/chrome/browser/ui/browser.cc b/chrome/browser/ui/browser.cc
index 97dd65d45989d1a9a0ab1cd83e392089c56b4a4e..175f9420acd193d34b5b1f72cfef41443392e533 100644
--- a/chrome/browser/ui/browser.cc
+++ b/chrome/browser/ui/browser.cc
@@ -2859,6 +2859,11 @@ void Browser::RegisterIntentHandlerHelper(WebContents* tab,
const string16& href,
const string16& title,
const string16& disposition) {
+ // TODO(yfriedman): remove OS_ANDROID clause when browser is excluded from
+ // Android build.
+#if defined(OS_ANDROID)
+ return;
+#else
if (!web_intents::IsWebIntentsEnabled())
return;
@@ -2890,6 +2895,7 @@ void Browser::RegisterIntentHandlerHelper(WebContents* tab,
service,
favicon_service,
tab->GetURL());
+#endif // defined(OS_ANDROID)
}
// static
@@ -4209,6 +4215,9 @@ void Browser::RegisterIntentHandler(WebContents* tab,
void Browser::WebIntentDispatch(
WebContents* tab, content::WebIntentsDispatcher* intents_dispatcher) {
+ // TODO(yfriedman): remove OS_ANDROID clause when browser is excluded from
+ // Android build.
+#if !defined(OS_ANDROID)
if (!web_intents::IsWebIntentsEnabled())
return;
@@ -4219,6 +4228,7 @@ void Browser::WebIntentDispatch(
this,
intents_dispatcher->GetIntent().action,
intents_dispatcher->GetIntent().type);
+#endif // !defined(OS_ANDROID)
}
void Browser::FindReply(WebContents* tab,

Powered by Google App Engine
This is Rietveld 408576698