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

Unified Diff: ui/android/java/src/org/chromium/ui/base/ViewAndroidDelegate.java

Issue 2790893004: Delete Android content detectors. (Closed)
Patch Set: Rebase Created 3 years, 8 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
« no previous file with comments | « third_party/WebKit/public/web/WebViewClient.h ('k') | ui/android/view_android.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ui/android/java/src/org/chromium/ui/base/ViewAndroidDelegate.java
diff --git a/ui/android/java/src/org/chromium/ui/base/ViewAndroidDelegate.java b/ui/android/java/src/org/chromium/ui/base/ViewAndroidDelegate.java
index 8e3dedacbdb50a3cdec821f05733d07df0216809..e4eb367d0688f66415677b543de496da4f074b65 100644
--- a/ui/android/java/src/org/chromium/ui/base/ViewAndroidDelegate.java
+++ b/ui/android/java/src/org/chromium/ui/base/ViewAndroidDelegate.java
@@ -6,7 +6,6 @@ package org.chromium.ui.base;
import android.annotation.TargetApi;
import android.content.ClipData;
-import android.content.Intent;
import android.graphics.Bitmap;
import android.os.Build;
import android.view.View;
@@ -15,22 +14,14 @@ import android.widget.FrameLayout.LayoutParams;
import android.widget.ImageView;
import org.chromium.base.ApiCompatibilityUtils;
-import org.chromium.base.Log;
import org.chromium.base.annotations.CalledByNative;
import org.chromium.base.annotations.JNINamespace;
-import java.net.URISyntaxException;
-
/**
* Class to acquire, position, and remove anchor views from the implementing View.
*/
@JNINamespace("ui")
public abstract class ViewAndroidDelegate {
- private static final String TAG = "ViewAndroidDelegate";
- private static final String GEO_SCHEME = "geo";
- private static final String TEL_SCHEME = "tel";
- private static final String MAILTO_SCHEME = "mailto";
-
/**
* @return An anchor view that can be used to anchor decoration views like Autofill popup.
*/
@@ -132,34 +123,6 @@ public abstract class ViewAndroidDelegate {
public void onBottomControlsChanged(float bottomControlsOffsetY, float bottomContentOffsetY) {}
/**
- * Called when a new content intent is requested to be started.
- * Invokes {@link #startContentIntent(Intent, String, boolean)} only if the parsed
- * intent is valid and the scheme is acceptable.
- */
- @CalledByNative
- private void onStartContentIntent(String intentUrl, boolean isMainFrame) {
- Intent intent;
- try {
- intent = Intent.parseUri(intentUrl, Intent.URI_INTENT_SCHEME);
- } catch (URISyntaxException e) {
- Log.d(TAG, "Bad URI %s", intentUrl, e);
- return;
- }
- String scheme = intent.getScheme();
- if (!(GEO_SCHEME.equals(scheme) || TEL_SCHEME.equals(scheme)
- || MAILTO_SCHEME.equals(scheme))) {
- Log.d(TAG, "Invalid scheme for URI %s", intentUrl);
- return;
- }
- startContentIntent(intent, intentUrl, isMainFrame);
- }
-
- /**
- * Start a new content intent.
- */
- public void startContentIntent(Intent intent, String intentUrl, boolean isMainFrame) {}
-
- /**
* @return container view that the anchor views are added to. May be null.
*/
@CalledByNative
« no previous file with comments | « third_party/WebKit/public/web/WebViewClient.h ('k') | ui/android/view_android.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698