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

Unified Diff: device/power_save_blocker/power_save_blocker_android.cc

Issue 2103243002: Factor out ContentViewAndroidDelegate (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Addressed comments chrome/webview Created 4 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: device/power_save_blocker/power_save_blocker_android.cc
diff --git a/device/power_save_blocker/power_save_blocker_android.cc b/device/power_save_blocker/power_save_blocker_android.cc
index 252ff02d6f71d0bb83a032ed708a1e7d95227547..90293285afab2bd5c0998f429134287ead7afb71 100644
--- a/device/power_save_blocker/power_save_blocker_android.cc
+++ b/device/power_save_blocker/power_save_blocker_android.cc
@@ -34,6 +34,8 @@ class PowerSaveBlocker::Delegate
base::android::ScopedJavaGlobalRef<jobject> java_power_save_blocker_;
+ ui::ViewAndroid::ScopedAnchorView anchor_view_;
+
scoped_refptr<base::SequencedTaskRunner> ui_task_runner_;
DISALLOW_COPY_AND_ASSIGN(Delegate);
@@ -52,22 +54,23 @@ PowerSaveBlocker::Delegate::~Delegate() {}
void PowerSaveBlocker::Delegate::ApplyBlock() {
DCHECK(ui_task_runner_->RunsTasksOnCurrentThread());
- ScopedJavaLocalRef<jobject> obj(java_power_save_blocker_);
+ if (!view_android_)
+ return;
+
JNIEnv* env = AttachCurrentThread();
- if (view_android_) {
- Java_PowerSaveBlocker_applyBlock(
- env, obj.obj(), view_android_->GetViewAndroidDelegate().obj());
- }
+ anchor_view_ = view_android_->AcquireAnchorView();
+ const ScopedJavaLocalRef<jobject>& popup_view = anchor_view_.view();
boliu 2016/07/29 20:05:15 ditto
Jinsuk Kim 2016/08/01 02:02:47 Done.
+ if (popup_view.is_null())
+ return;
+ ScopedJavaLocalRef<jobject> obj(java_power_save_blocker_);
+ Java_PowerSaveBlocker_applyBlock(env, obj.obj(), popup_view.obj());
}
void PowerSaveBlocker::Delegate::RemoveBlock() {
DCHECK(ui_task_runner_->RunsTasksOnCurrentThread());
ScopedJavaLocalRef<jobject> obj(java_power_save_blocker_);
- JNIEnv* env = AttachCurrentThread();
- if (view_android_) {
- Java_PowerSaveBlocker_removeBlock(
- env, obj.obj(), view_android_->GetViewAndroidDelegate().obj());
- }
+ Java_PowerSaveBlocker_removeBlock(AttachCurrentThread(), obj.obj());
+ anchor_view_.Reset();
}
PowerSaveBlocker::PowerSaveBlocker(

Powered by Google App Engine
This is Rietveld 408576698