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

Unified Diff: chrome/browser/chrome_content_browser_client.cc

Issue 2619603002: Remove android_java_ui as it is not used (Closed)
Patch Set: Rebase to master Created 3 years, 11 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/chrome_content_browser_client.cc
diff --git a/chrome/browser/chrome_content_browser_client.cc b/chrome/browser/chrome_content_browser_client.cc
index 086093c6b86720474d185ee6ddd417ddbf417b30..62af2d512a5e24dcbab2198a2e39430e2a3f91bb 100644
--- a/chrome/browser/chrome_content_browser_client.cc
+++ b/chrome/browser/chrome_content_browser_client.cc
@@ -262,16 +262,13 @@
#include "components/crash/content/browser/crash_handler_host_linux.h"
#endif
-#if BUILDFLAG(ANDROID_JAVA_UI)
+#if defined(OS_ANDROID)
+#include "chrome/browser/android/devtools_manager_delegate_android.h"
#include "chrome/browser/android/ntp/new_tab_page_url_handler.h"
#include "chrome/browser/android/service_tab_launcher.h"
#include "chrome/browser/android/webapps/single_tab_mode_tab_helper.h"
#include "components/payments/payment_request.mojom.h"
#include "content/public/browser/android/java_interfaces.h"
-#endif
-
-#if defined(OS_ANDROID)
-#include "chrome/browser/android/devtools_manager_delegate_android.h"
#include "ui/base/ui_base_paths.h"
#include "ui/gfx/android/device_display_info.h"
#else
@@ -700,17 +697,6 @@ class CertificateReportingServiceCertReporter : public SSLCertReporter {
DISALLOW_COPY_AND_ASSIGN(CertificateReportingServiceCertReporter);
};
-#if BUILDFLAG(ANDROID_JAVA_UI)
-void HandleSingleTabModeBlockOnUIThread(const BlockedWindowParams& params) {
- WebContents* web_contents = tab_util::GetWebContentsByFrameID(
- params.render_process_id(), params.opener_render_frame_id());
- if (!web_contents)
- return;
-
- SingleTabModeTabHelper::FromWebContents(web_contents)->HandleOpenUrl(params);
-}
-#endif // BUILDFLAG(ANDROID_JAVA_UI)
-
#if defined(OS_ANDROID)
float GetDeviceScaleAdjustment() {
static const float kMinFSM = 1.05f;
@@ -731,6 +717,15 @@ float GetDeviceScaleAdjustment() {
(kWidthForMaxFSM - kWidthForMinFSM);
return ratio * (kMaxFSM - kMinFSM) + kMinFSM;
}
+
+void HandleSingleTabModeBlockOnUIThread(const BlockedWindowParams& params) {
+ WebContents* web_contents = tab_util::GetWebContentsByFrameID(
+ params.render_process_id(), params.opener_render_frame_id());
+ if (!web_contents)
+ return;
+
+ SingleTabModeTabHelper::FromWebContents(web_contents)->HandleOpenUrl(params);
+}
#endif // defined(OS_ANDROID)
#if BUILDFLAG(ENABLE_EXTENSIONS)
@@ -1531,7 +1526,7 @@ void MaybeAppendBlinkSettingsSwitchForFieldTrial(
base::JoinString(blink_settings, ","));
}
-#if BUILDFLAG(ANDROID_JAVA_UI)
+#if defined(OS_ANDROID)
void ForwardShareServiceRequest(
base::WeakPtr<service_manager::InterfaceProvider> interface_provider,
blink::mojom::ShareServiceRequest request) {
@@ -2387,7 +2382,7 @@ bool ChromeContentBrowserClient::CanCreateWindow(
}
}
-#if BUILDFLAG(ANDROID_JAVA_UI)
+#if defined(OS_ANDROID)
if (SingleTabModeTabHelper::IsRegistered(opener_render_process_id,
opener_render_frame_id)) {
BrowserThread::PostTask(BrowserThread::UI,
@@ -2560,7 +2555,7 @@ void ChromeContentBrowserClient::BrowserURLHandlerCreated(
handler->AddHandlerPair(&WillHandleBrowserAboutURL,
BrowserURLHandler::null_handler());
-#if BUILDFLAG(ANDROID_JAVA_UI)
+#if defined(OS_ANDROID)
// Handler to rewrite chrome://newtab on Android.
handler->AddHandlerPair(&chrome::android::HandleAndroidNativePageURL,
BrowserURLHandler::null_handler());
@@ -2996,7 +2991,7 @@ void ChromeContentBrowserClient::RegisterRenderFrameMojoInterfaces(
BindSensitiveInputVisibilityService,
render_frame_host));
-#if BUILDFLAG(ANDROID_JAVA_UI)
+#if defined(OS_ANDROID)
content::WebContents* web_contents =
content::WebContents::FromRenderFrameHost(render_frame_host);
if (web_contents) {
@@ -3006,6 +3001,11 @@ void ChromeContentBrowserClient::RegisterRenderFrameMojoInterfaces(
registry->AddInterface(
base::Bind(&ForwardShareServiceRequest,
web_contents->GetJavaInterfaces()->GetWeakPtr()));
+ if (AreExperimentalWebPlatformFeaturesEnabled()) {
+ registry->AddInterface(
+ web_contents->GetJavaInterfaces()
+ ->CreateInterfaceFactory<blink::mojom::BarcodeDetection>());
+ }
}
#elif defined(OS_LINUX) || defined(OS_CHROMEOS) || defined(OS_WIN)
// TODO(crbug.com/679127): Enable for MacViews implementation.
@@ -3019,21 +3019,6 @@ void ChromeContentBrowserClient::RegisterRenderFrameMojoInterfaces(
}
#endif
-#if defined(OS_ANDROID)
- if (AreExperimentalWebPlatformFeaturesEnabled()) {
- content::WebContents* web_contents =
- content::WebContents::FromRenderFrameHost(render_frame_host);
- if (web_contents) {
- registry->AddInterface(
- web_contents->GetJavaInterfaces()
- ->CreateInterfaceFactory<blink::mojom::BarcodeDetection>());
- registry->AddInterface(
- web_contents->GetJavaInterfaces()
- ->CreateInterfaceFactory<blink::mojom::TextDetection>());
- }
- }
-#endif
-
#if defined(OS_LINUX) || defined(OS_WIN)
if (!ChromeOriginTrialPolicy().IsFeatureDisabled("WebShare")) {
registry->AddInterface(base::Bind(&ShareServiceImpl::Create));
@@ -3120,7 +3105,7 @@ void ChromeContentBrowserClient::OpenURL(
const base::Callback<void(content::WebContents*)>& callback) {
DCHECK_CURRENTLY_ON(BrowserThread::UI);
-#if BUILDFLAG(ANDROID_JAVA_UI)
+#if defined(OS_ANDROID)
ServiceTabLauncher::GetInstance()->LaunchTab(browser_context, params,
callback);
#else
« no previous file with comments | « chrome/browser/browsing_data/chrome_browsing_data_remover_delegate.cc ('k') | chrome/browser/dom_distiller/profile_utils.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698