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

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

Issue 481803004: Removing ContentViewCore dependencies from few functions which acts as direct wrapper to WebContents (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fixed the nit. Created 6 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
Index: content/public/android/java/src/org/chromium/content/browser/ContentViewCore.java
diff --git a/content/public/android/java/src/org/chromium/content/browser/ContentViewCore.java b/content/public/android/java/src/org/chromium/content/browser/ContentViewCore.java
index aebda91e321f592e556799e3420e4d064a3ef890..73cc1f5d29fd75d4cad6e086cc789901974893c0 100644
--- a/content/public/android/java/src/org/chromium/content/browser/ContentViewCore.java
+++ b/content/public/android/java/src/org/chromium/content/browser/ContentViewCore.java
@@ -823,11 +823,6 @@ public class ContentViewCore
return mContentViewClient;
}
- public int getBackgroundColor() {
- assert mWebContents != null;
- return mWebContents.getBackgroundColor();
- }
-
@CalledByNative
private void onBackgroundColorChanged(int color) {
getContentViewClient().onBackgroundColorChanged(color);
@@ -859,34 +854,6 @@ public class ContentViewCore
}
/**
- * Stops loading the current web contents.
- */
- public void stopLoading() {
- assert mWebContents != null;
- mWebContents.stop();
- }
-
- /**
- * Get the URL of the current page.
- *
- * @return The URL of the current page.
- */
- public String getUrl() {
- assert mWebContents != null;
- return mWebContents.getUrl();
- }
-
- /**
- * Get the title of the current page.
- *
- * @return The title of the current page.
- */
- public String getTitle() {
- assert mWebContents != null;
- return mWebContents.getTitle();
- }
-
- /**
* Shows an interstitial page driven by the passed in delegate.
*
* @param url The URL being blocked by the interstitial.
@@ -2188,14 +2155,6 @@ public class ContentViewCore
}
/**
- * Shows the IME if the focused widget could accept text input.
- */
- public void showImeIfNeeded() {
- assert mWebContents != null;
- mWebContents.showImeIfNeeded();
- }
-
- /**
* Hides the IME if the containerView is the active view for IME.
*/
public void hideImeIfNeeded() {

Powered by Google App Engine
This is Rietveld 408576698