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

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

Issue 10693134: Revert 146000 - Split out ContentViewCore from ContentView for embedders. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Created 8 years, 5 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/ContentSettings.java
===================================================================
--- content/public/android/java/src/org/chromium/content/browser/ContentSettings.java (revision 146004)
+++ content/public/android/java/src/org/chromium/content/browser/ContentSettings.java (working copy)
@@ -29,7 +29,7 @@
private int mNativeContentSettings = 0;
- private ContentViewCore mContentViewCore;
+ private ContentView mContentView;
// When ContentView is used in PERSONALITY_CHROME mode, settings can't
// be modified through the ContentSettings instance.
@@ -84,7 +84,7 @@
private Handler mHandler;
EventHandler() {
- mHandler = mContentViewCore.isPersonalityView() ?
+ mHandler = mContentView.isPersonalityView() ?
new Handler() {
@Override
public void handleMessage(Message msg) {
@@ -96,13 +96,13 @@
}
break;
case UPDATE_UA:
- synchronized (mContentViewCore) {
- mContentViewCore.setAllUserAgentOverridesInHistory();
+ synchronized (mContentView) {
+ mContentView.setAllUserAgentOverridesInHistory();
}
break;
case UPDATE_MULTI_TOUCH:
- synchronized (mContentViewCore) {
- mContentViewCore.updateMultiTouchZoomSupport();
+ synchronized (mContentView) {
+ mContentView.updateMultiTouchZoomSupport();
}
break;
}
@@ -140,10 +140,10 @@
* Package constructor to prevent clients from creating a new settings
* instance. Must be called on the UI thread.
*/
- ContentSettings(ContentViewCore contentViewCore, int nativeContentView) {
+ ContentSettings(ContentView contentView, int nativeContentView) {
ThreadUtils.assertOnUiThread();
- mContentViewCore = contentViewCore;
- mCanModifySettings = mContentViewCore.isPersonalityView();
+ mContentView = contentView;
+ mCanModifySettings = mContentView.isPersonalityView();
mNativeContentSettings = nativeInit(nativeContentView, mCanModifySettings);
assert mNativeContentSettings != 0;
« no previous file with comments | « content/content_jni.gypi ('k') | content/public/android/java/src/org/chromium/content/browser/ContentView.java » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698