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

Unified Diff: content/browser/android/content_view_core_impl.cc

Issue 1041823004: Remove java ViewAndroid class (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: remove ViewAndroid Created 5 years, 9 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/browser/android/content_view_core_impl.cc
diff --git a/content/browser/android/content_view_core_impl.cc b/content/browser/android/content_view_core_impl.cc
index 81d59d0456bba48fbc00450cd13b12db381d371e..76c4550e33ce1f67aa7cd0a8eb84c9ad820742eb 100644
--- a/content/browser/android/content_view_core_impl.cc
+++ b/content/browser/android/content_view_core_impl.cc
@@ -214,7 +214,7 @@ ContentViewCoreImpl::ContentViewCoreImpl(
JNIEnv* env,
jobject obj,
WebContents* web_contents,
- jobject view_android,
+ jobject view_android_delegate,
ui::WindowAndroid* window_android,
jobject java_bridge_retained_object_set)
: WebContentsObserver(web_contents),
@@ -222,7 +222,7 @@ ContentViewCoreImpl::ContentViewCoreImpl(
web_contents_(static_cast<WebContentsImpl*>(web_contents)),
root_layer_(cc::SolidColorLayer::Create()),
dpi_scale_(GetPrimaryDisplayDeviceScaleFactor()),
- view_android_(new ui::ViewAndroid(view_android, window_android)),
+ view_android_(new ui::ViewAndroid(view_android_delegate, window_android)),
window_android_(window_android),
device_orientation_(0),
accessibility_enabled_(false) {
@@ -1245,6 +1245,22 @@ bool ContentViewCoreImpl::IsFullscreenRequiredForOrientationLock() const {
obj.obj());
}
+void ContentViewCoreImpl::IncrementKeepScreenOnCount() {
+ JNIEnv* env = AttachCurrentThread();
+ ScopedJavaLocalRef<jobject> obj = java_ref_.get(env);
+ if (obj.is_null())
+ return;
+ Java_ContentViewCore_incrementKeepScreenOnCount(env, obj.obj());
+}
+
+void ContentViewCoreImpl::DecrementKeepScreenOnCount() {
+ JNIEnv* env = AttachCurrentThread();
+ ScopedJavaLocalRef<jobject> obj = java_ref_.get(env);
+ if (obj.is_null())
+ return;
+ Java_ContentViewCore_decrementKeepScreenOnCount(env, obj.obj());
+}
+
void ContentViewCoreImpl::SetAccessibilityEnabledInternal(bool enabled) {
accessibility_enabled_ = enabled;
BrowserAccessibilityStateImpl* accessibility_state =
@@ -1365,11 +1381,12 @@ void ContentViewCoreImpl::WebContentsDestroyed() {
jlong Init(JNIEnv* env,
jobject obj,
jobject web_contents,
- jobject view_android,
+ jobject view_android_delegate,
jlong window_android,
jobject retained_objects_set) {
ContentViewCoreImpl* view = new ContentViewCoreImpl(
- env, obj, WebContents::FromJavaWebContents(web_contents), view_android,
+ env, obj, WebContents::FromJavaWebContents(web_contents),
+ view_android_delegate,
reinterpret_cast<ui::WindowAndroid*>(window_android),
retained_objects_set);
return reinterpret_cast<intptr_t>(view);

Powered by Google App Engine
This is Rietveld 408576698