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

Side by Side Diff: ui/android/window_android.cc

Issue 2514633002: Revert of Android: support multiple displays on C++ side (Closed)
Patch Set: Created 4 years, 1 month 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 unified diff | Download patch
« no previous file with comments | « ui/android/window_android.h ('k') | ui/display/BUILD.gn » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "ui/android/window_android.h" 5 #include "ui/android/window_android.h"
6 6
7 #include "base/android/context_utils.h" 7 #include "base/android/context_utils.h"
8 #include "base/android/jni_android.h" 8 #include "base/android/jni_android.h"
9 #include "base/android/jni_array.h" 9 #include "base/android/jni_array.h"
10 #include "base/android/jni_string.h" 10 #include "base/android/jni_string.h"
11 #include "base/android/jni_weak_ref.h" 11 #include "base/android/jni_weak_ref.h"
12 #include "base/android/scoped_java_ref.h" 12 #include "base/android/scoped_java_ref.h"
13 #include "jni/WindowAndroid_jni.h" 13 #include "jni/WindowAndroid_jni.h"
14 #include "ui/android/window_android_compositor.h" 14 #include "ui/android/window_android_compositor.h"
15 #include "ui/android/window_android_observer.h" 15 #include "ui/android/window_android_observer.h"
16 16
17 namespace ui { 17 namespace ui {
18 18
19 using base::android::AttachCurrentThread; 19 using base::android::AttachCurrentThread;
20 using base::android::JavaParamRef; 20 using base::android::JavaParamRef;
21 using base::android::JavaRef; 21 using base::android::JavaRef;
22 using base::android::ScopedJavaLocalRef; 22 using base::android::ScopedJavaLocalRef;
23 23
24 WindowAndroid::WindowAndroid(JNIEnv* env, jobject obj, int display_id) 24 WindowAndroid::WindowAndroid(JNIEnv* env, jobject obj) : compositor_(NULL) {
25 : display_id_(display_id), compositor_(NULL) {
26 java_window_.Reset(env, obj); 25 java_window_.Reset(env, obj);
27 } 26 }
28 27
29 void WindowAndroid::Destroy(JNIEnv* env, const JavaParamRef<jobject>& obj) { 28 void WindowAndroid::Destroy(JNIEnv* env, const JavaParamRef<jobject>& obj) {
30 delete this; 29 delete this;
31 } 30 }
32 31
33 ScopedJavaLocalRef<jobject> WindowAndroid::GetJavaObject() { 32 ScopedJavaLocalRef<jobject> WindowAndroid::GetJavaObject() {
34 return base::android::ScopedJavaLocalRef<jobject>(java_window_); 33 return base::android::ScopedJavaLocalRef<jobject>(java_window_);
35 } 34 }
36 35
37 bool WindowAndroid::RegisterWindowAndroid(JNIEnv* env) { 36 bool WindowAndroid::RegisterWindowAndroid(JNIEnv* env) {
38 return RegisterNativesImpl(env); 37 return RegisterNativesImpl(env);
39 } 38 }
40 39
41 WindowAndroid::~WindowAndroid() { 40 WindowAndroid::~WindowAndroid() {
42 DCHECK(parent_ == nullptr) << "WindowAndroid must be a root view."; 41 DCHECK(parent_ == nullptr) << "WindowAndroid must be a root view.";
43 DCHECK(!compositor_); 42 DCHECK(!compositor_);
44 Java_WindowAndroid_clearNativePointer(AttachCurrentThread(), GetJavaObject()); 43 Java_WindowAndroid_clearNativePointer(AttachCurrentThread(), GetJavaObject());
45 } 44 }
46 45
47 WindowAndroid* WindowAndroid::CreateForTesting() { 46 WindowAndroid* WindowAndroid::CreateForTesting() {
48 JNIEnv* env = AttachCurrentThread(); 47 JNIEnv* env = AttachCurrentThread();
49 const JavaRef<jobject>& context = base::android::GetApplicationContext(); 48 const JavaRef<jobject>& context = base::android::GetApplicationContext();
50 long native_pointer = Java_WindowAndroid_createForTesting(env, context); 49 return new WindowAndroid(
51 return reinterpret_cast<WindowAndroid*>(native_pointer); 50 env, Java_WindowAndroid_createForTesting(env, context).obj());
52 } 51 }
53 52
54 void WindowAndroid::DestroyForTesting() { 53 void WindowAndroid::DestroyForTesting() {
55 delete this; 54 delete this;
56 } 55 }
57 56
58 void WindowAndroid::OnCompositingDidCommit() { 57 void WindowAndroid::OnCompositingDidCommit() {
59 for (WindowAndroidObserver& observer : observer_list_) 58 for (WindowAndroidObserver& observer : observer_list_)
60 observer.OnCompositingDidCommit(); 59 observer.OnCompositingDidCommit();
61 } 60 }
(...skipping 86 matching lines...) Expand 10 before | Expand all | Expand 10 after
148 147
149 WindowAndroid* WindowAndroid::GetWindowAndroid() const { 148 WindowAndroid* WindowAndroid::GetWindowAndroid() const {
150 DCHECK(parent_ == nullptr); 149 DCHECK(parent_ == nullptr);
151 return const_cast<WindowAndroid*>(this); 150 return const_cast<WindowAndroid*>(this);
152 } 151 }
153 152
154 // ---------------------------------------------------------------------------- 153 // ----------------------------------------------------------------------------
155 // Native JNI methods 154 // Native JNI methods
156 // ---------------------------------------------------------------------------- 155 // ----------------------------------------------------------------------------
157 156
158 jlong Init(JNIEnv* env, const JavaParamRef<jobject>& obj, int sdk_display_id) { 157 jlong Init(JNIEnv* env, const JavaParamRef<jobject>& obj) {
159 WindowAndroid* window = new WindowAndroid(env, obj, sdk_display_id); 158 WindowAndroid* window = new WindowAndroid(env, obj);
160 return reinterpret_cast<intptr_t>(window); 159 return reinterpret_cast<intptr_t>(window);
161 } 160 }
162 161
163 } // namespace ui 162 } // namespace ui
OLDNEW
« no previous file with comments | « ui/android/window_android.h ('k') | ui/display/BUILD.gn » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698