OLD | NEW |
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. | 1 // Copyright (c) 2012 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 "content/app/android/child_process_service.h" | 5 #include "content/app/android/child_process_service.h" |
6 | 6 |
7 #include <android/native_window_jni.h> | 7 #include <android/native_window_jni.h> |
8 #include <cpu-features.h> | 8 #include <cpu-features.h> |
9 | 9 |
10 #include "base/android/jni_array.h" | 10 #include "base/android/jni_array.h" |
(...skipping 54 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
65 // Overridden from GpuSurfaceLookup: | 65 // Overridden from GpuSurfaceLookup: |
66 virtual gfx::AcceleratedWidget AcquireNativeWidget(int surface_id) OVERRIDE { | 66 virtual gfx::AcceleratedWidget AcquireNativeWidget(int surface_id) OVERRIDE { |
67 JNIEnv* env = base::android::AttachCurrentThread(); | 67 JNIEnv* env = base::android::AttachCurrentThread(); |
68 gfx::ScopedJavaSurface surface( | 68 gfx::ScopedJavaSurface surface( |
69 content::Java_ChildProcessService_getViewSurface( | 69 content::Java_ChildProcessService_getViewSurface( |
70 env, service_.obj(), surface_id)); | 70 env, service_.obj(), surface_id)); |
71 | 71 |
72 if (surface.j_surface().is_null()) | 72 if (surface.j_surface().is_null()) |
73 return NULL; | 73 return NULL; |
74 | 74 |
75 ANativeWindow* native_window = ANativeWindow_fromSurface( | 75 // Note: This ensures that any local references used by |
76 env, surface.j_surface().obj()); | 76 // ANativeWindow_fromSurface are released immediately. This is needed as a |
| 77 // workaround for https://code.google.com/p/android/issues/detail?id=68174 |
| 78 base::android::ScopedJavaLocalFrame scoped_local_reference_frame(env); |
| 79 ANativeWindow* native_window = |
| 80 ANativeWindow_fromSurface(env, surface.j_surface().obj()); |
77 | 81 |
78 return native_window; | 82 return native_window; |
79 } | 83 } |
80 | 84 |
81 // Overridden from SurfaceTextureLookup: | 85 // Overridden from SurfaceTextureLookup: |
82 virtual gfx::AcceleratedWidget AcquireNativeWidget(int primary_id, | 86 virtual gfx::AcceleratedWidget AcquireNativeWidget(int primary_id, |
83 int secondary_id) | 87 int secondary_id) |
84 OVERRIDE { | 88 OVERRIDE { |
85 JNIEnv* env = base::android::AttachCurrentThread(); | 89 JNIEnv* env = base::android::AttachCurrentThread(); |
86 gfx::ScopedJavaSurface surface( | 90 gfx::ScopedJavaSurface surface( |
87 content::Java_ChildProcessService_getSurfaceTextureSurface( | 91 content::Java_ChildProcessService_getSurfaceTextureSurface( |
88 env, service_.obj(), primary_id, secondary_id)); | 92 env, service_.obj(), primary_id, secondary_id)); |
89 | 93 |
90 if (surface.j_surface().is_null()) | 94 if (surface.j_surface().is_null()) |
91 return NULL; | 95 return NULL; |
92 | 96 |
| 97 // Note: This ensures that any local references used by |
| 98 // ANativeWindow_fromSurface are released immediately. This is needed as a |
| 99 // workaround for https://code.google.com/p/android/issues/detail?id=68174 |
| 100 base::android::ScopedJavaLocalFrame scoped_local_reference_frame(env); |
93 ANativeWindow* native_window = | 101 ANativeWindow* native_window = |
94 ANativeWindow_fromSurface(env, surface.j_surface().obj()); | 102 ANativeWindow_fromSurface(env, surface.j_surface().obj()); |
95 | 103 |
96 return native_window; | 104 return native_window; |
97 } | 105 } |
98 | 106 |
99 private: | 107 private: |
100 // The instance of org.chromium.content.app.ChildProcessService. | 108 // The instance of org.chromium.content.app.ChildProcessService. |
101 base::android::ScopedJavaGlobalRef<jobject> service_; | 109 base::android::ScopedJavaGlobalRef<jobject> service_; |
102 | 110 |
(...skipping 58 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
161 | 169 |
162 bool RegisterChildProcessService(JNIEnv* env) { | 170 bool RegisterChildProcessService(JNIEnv* env) { |
163 return RegisterNativesImpl(env); | 171 return RegisterNativesImpl(env); |
164 } | 172 } |
165 | 173 |
166 void ShutdownMainThread(JNIEnv* env, jobject obj) { | 174 void ShutdownMainThread(JNIEnv* env, jobject obj) { |
167 ChildThread::ShutdownThread(); | 175 ChildThread::ShutdownThread(); |
168 } | 176 } |
169 | 177 |
170 } // namespace content | 178 } // namespace content |
OLD | NEW |