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 25fe9512c627df23c7375233d37d2eccf59ff597..6f22fcf9811528abd9d58739344be7b401c1f721 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 |
@@ -16,7 +16,6 @@ import android.content.res.Configuration; |
import android.database.ContentObserver; |
import android.graphics.Bitmap; |
import android.graphics.Canvas; |
-import android.graphics.Color; |
import android.graphics.Rect; |
import android.net.Uri; |
import android.os.Build; |
@@ -860,10 +859,8 @@ public class ContentViewCore |
} |
public int getBackgroundColor() { |
- if (mNativeContentViewCore != 0) { |
- return nativeGetBackgroundColor(mNativeContentViewCore); |
- } |
- return Color.WHITE; |
+ assert mWebContents != null; |
+ return mWebContents.getBackgroundColor(); |
} |
@CalledByNative |
@@ -931,16 +928,16 @@ public class ContentViewCore |
@VisibleForTesting |
public void showInterstitialPage( |
String url, InterstitialPageDelegateAndroid delegate) { |
- if (mNativeContentViewCore == 0) return; |
- nativeShowInterstitialPage(mNativeContentViewCore, url, delegate.getNative()); |
+ assert mWebContents != null; |
+ mWebContents.showInterstitialPage(url, delegate.getNative()); |
} |
/** |
* @return Whether the page is currently showing an interstitial, such as a bad HTTPS page. |
*/ |
public boolean isShowingInterstitialPage() { |
- return mNativeContentViewCore == 0 ? |
- false : nativeIsShowingInterstitialPage(mNativeContentViewCore); |
+ assert mWebContents != null; |
+ return mWebContents.isShowingInterstitialPage(); |
} |
/** |
@@ -1406,7 +1403,8 @@ public class ContentViewCore |
* main frame's document. |
*/ |
void addStyleSheetByURL(String url) { |
- nativeAddStyleSheetByURL(mNativeContentViewCore, url); |
+ assert mWebContents != null; |
+ mWebContents.addStyleSheetByURL(url); |
} |
/** Callback interface for evaluateJavaScript(). */ |
@@ -1462,8 +1460,8 @@ public class ContentViewCore |
* To be called when the ContentView is shown. |
*/ |
public void onShow() { |
- assert mNativeContentViewCore != 0; |
- nativeOnShow(mNativeContentViewCore); |
+ assert mWebContents != null; |
+ mWebContents.onShow(); |
setAccessibilityState(mAccessibilityManager.isEnabled()); |
} |
@@ -1479,10 +1477,10 @@ public class ContentViewCore |
* To be called when the ContentView is hidden. |
*/ |
public void onHide() { |
- assert mNativeContentViewCore != 0; |
+ assert mWebContents != null; |
hidePopups(); |
setInjectedAccessibility(false); |
- nativeOnHide(mNativeContentViewCore); |
+ mWebContents.onHide(); |
} |
/** |
@@ -1676,11 +1674,8 @@ public class ContentViewCore |
} |
private void scrollFocusedEditableNodeIntoView() { |
- if (mNativeContentViewCore == 0) return; |
- // The native side keeps track of whether the zoom and scroll actually occurred. It is |
- // more efficient to do it this way and sometimes fire an unnecessary message rather |
- // than synchronize with the renderer and always have an additional message. |
- nativeScrollFocusedEditableNodeIntoView(mNativeContentViewCore); |
+ assert mWebContents != null; |
+ mWebContents.scrollFocusedEditableNodeIntoView(); |
} |
/** |
@@ -1688,8 +1683,8 @@ public class ContentViewCore |
* The caller can check if selection actually occurred by listening to OnSelectionChanged. |
*/ |
public void selectWordAroundCaret() { |
- if (mNativeContentViewCore == 0) return; |
- nativeSelectWordAroundCaret(mNativeContentViewCore); |
+ assert mWebContents != null; |
+ mWebContents.selectWordAroundCaret(); |
} |
/** |
@@ -2333,7 +2328,8 @@ public class ContentViewCore |
* Shows the IME if the focused widget could accept text input. |
*/ |
public void showImeIfNeeded() { |
- if (mNativeContentViewCore != 0) nativeShowImeIfNeeded(mNativeContentViewCore); |
+ assert mWebContents != null; |
+ mWebContents.showImeIfNeeded(); |
} |
/** |
@@ -2850,8 +2846,8 @@ public class ContentViewCore |
* once the texture is actually ready. |
*/ |
public boolean isReady() { |
- if (mNativeContentViewCore == 0) return false; |
- return nativeIsRenderWidgetHostViewReady(mNativeContentViewCore); |
+ assert mWebContents != null; |
+ return mWebContents.isReady(); |
} |
@CalledByNative |
@@ -3070,7 +3066,8 @@ public class ContentViewCore |
* Inform WebKit that Fullscreen mode has been exited by the user. |
*/ |
public void exitFullscreen() { |
- if (mNativeContentViewCore != 0) nativeExitFullscreen(mNativeContentViewCore); |
+ assert mWebContents != null; |
+ mWebContents.exitFullscreen(); |
} |
/** |
@@ -3082,10 +3079,9 @@ public class ContentViewCore |
*/ |
public void updateTopControlsState(boolean enableHiding, boolean enableShowing, |
boolean animate) { |
- if (mNativeContentViewCore != 0) { |
- nativeUpdateTopControlsState( |
- mNativeContentViewCore, enableHiding, enableShowing, animate); |
- } |
+ assert mWebContents != null; |
+ mWebContents.updateTopControlsState( |
+ enableHiding, enableShowing, animate); |
} |
/** |
@@ -3283,10 +3279,6 @@ public class ContentViewCore |
private native String nativeGetURL(long nativeContentViewCoreImpl); |
- private native void nativeShowInterstitialPage( |
- long nativeContentViewCoreImpl, String url, long nativeInterstitialPageDelegateAndroid); |
- private native boolean nativeIsShowingInterstitialPage(long nativeContentViewCoreImpl); |
- |
private native boolean nativeIsIncognito(long nativeContentViewCoreImpl); |
private native void nativeSetFocus(long nativeContentViewCoreImpl, boolean focused); |
@@ -3355,15 +3347,8 @@ public class ContentViewCore |
private native void nativeSelectPopupMenuItems(long nativeContentViewCoreImpl, int[] indices); |
- private native void nativeScrollFocusedEditableNodeIntoView(long nativeContentViewCoreImpl); |
- |
- private native void nativeSelectWordAroundCaret(long nativeContentViewCoreImpl); |
- |
private native void nativeClearHistory(long nativeContentViewCoreImpl); |
- private native void nativeAddStyleSheetByURL(long nativeContentViewCoreImpl, |
- String stylesheetUrl); |
- |
private native void nativeEvaluateJavaScript(long nativeContentViewCoreImpl, |
String script, JavaScriptCallback callback, boolean startRenderer); |
@@ -3374,11 +3359,6 @@ public class ContentViewCore |
private native int nativeGetCurrentRenderProcessId(long nativeContentViewCoreImpl); |
- private native int nativeGetBackgroundColor(long nativeContentViewCoreImpl); |
- |
- private native void nativeOnShow(long nativeContentViewCoreImpl); |
- private native void nativeOnHide(long nativeContentViewCoreImpl); |
- |
private native void nativeSetUseDesktopUserAgent(long nativeContentViewCoreImpl, |
boolean enabled, boolean reloadOnChange); |
private native boolean nativeGetUseDesktopUserAgent(long nativeContentViewCoreImpl); |
@@ -3402,14 +3382,6 @@ public class ContentViewCore |
private native void nativeWasResized(long nativeContentViewCoreImpl); |
- private native boolean nativeIsRenderWidgetHostViewReady(long nativeContentViewCoreImpl); |
- |
- private native void nativeExitFullscreen(long nativeContentViewCoreImpl); |
- private native void nativeUpdateTopControlsState(long nativeContentViewCoreImpl, |
- boolean enableHiding, boolean enableShowing, boolean animate); |
- |
- private native void nativeShowImeIfNeeded(long nativeContentViewCoreImpl); |
- |
private native void nativeSetAccessibilityEnabled( |
long nativeContentViewCoreImpl, boolean enabled); |