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

Unified Diff: content/public/android/java/src/org/chromium/content/browser/ContentViewClient.java

Issue 2536223003: Refactor ContentViewClient (3/6) (Closed)
Patch Set: addressed comments Created 4 years 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 | « content/public/android/java/src/org/chromium/content/browser/ContentView.java ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/public/android/java/src/org/chromium/content/browser/ContentViewClient.java
diff --git a/content/public/android/java/src/org/chromium/content/browser/ContentViewClient.java b/content/public/android/java/src/org/chromium/content/browser/ContentViewClient.java
index 4c24f7dc136a3aac19428430d2e9c7b19ef27880..be5613242a9506dd822172201bdb0ca901d45833 100644
--- a/content/public/android/java/src/org/chromium/content/browser/ContentViewClient.java
+++ b/content/public/android/java/src/org/chromium/content/browser/ContentViewClient.java
@@ -8,7 +8,6 @@ import android.content.ActivityNotFoundException;
import android.content.Context;
import android.content.Intent;
import android.view.KeyEvent;
-import android.view.View.MeasureSpec;
import org.chromium.base.Log;
import org.chromium.base.metrics.RecordUserAction;
@@ -24,15 +23,13 @@ import org.chromium.base.metrics.RecordUserAction;
* TODO(mkosiba): Rid this class of default implementations. This class is used by both WebView and
* the browser and we don't want a the browser-specific default implementation to accidentally leak
* over to WebView.
+ *
+ * WARNING: ConteViewClient is going away. Do not add new stuff in this class.
*/
public class ContentViewClient {
// Tag used for logging.
private static final String TAG = "cr_ContentViewClient";
- // Default value to signal that the ContentView's size should not be overridden.
- private static final int UNSPECIFIED_MEASURE_SPEC =
- MeasureSpec.makeMeasureSpec(0, MeasureSpec.UNSPECIFIED);
-
private static final String GEO_SCHEME = "geo";
private static final String TEL_SCHEME = "tel";
private static final String MAILTO_SCHEME = "mailto";
@@ -150,28 +147,6 @@ public class ContentViewClient {
}
/**
- * ContentViewClient users can return a custom value to override the width of
- * the ContentView. By default, this method returns MeasureSpec.UNSPECIFIED, which
- * indicates that the value should not be overridden.
- *
- * @return The desired width of the ContentView.
- */
- public int getDesiredWidthMeasureSpec() {
- return UNSPECIFIED_MEASURE_SPEC;
- }
-
- /**
- * ContentViewClient users can return a custom value to override the height of
- * the ContentView. By default, this method returns MeasureSpec.UNSPECIFIED, which
- * indicates that the value should not be overridden.
- *
- * @return The desired height of the ContentView.
- */
- public int getDesiredHeightMeasureSpec() {
- return UNSPECIFIED_MEASURE_SPEC;
- }
-
- /**
* Returns the left system window inset in pixels. The system window inset represents the area
* of a full-screen window that is partially or fully obscured by the status bar, navigation
* bar, IME or other system windows.
« no previous file with comments | « content/public/android/java/src/org/chromium/content/browser/ContentView.java ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698