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

Side by Side Diff: content/browser/power_save_blocker_android.cc

Issue 1000373002: favor DCHECK_CURRENTLY_ON for better logs in content/browser/[f-p]* (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rebase on master 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 unified diff | Download patch
OLDNEW
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/browser/power_save_blocker_android.h" 5 #include "content/browser/power_save_blocker_android.h"
6 6
7 #include "base/android/jni_android.h" 7 #include "base/android/jni_android.h"
8 #include "base/android/jni_weak_ref.h" 8 #include "base/android/jni_weak_ref.h"
9 #include "base/logging.h" 9 #include "base/logging.h"
10 #include "content/browser/power_save_blocker_impl.h" 10 #include "content/browser/power_save_blocker_impl.h"
(...skipping 23 matching lines...) Expand all
34 private: 34 private:
35 friend class base::RefCountedThreadSafe<Delegate>; 35 friend class base::RefCountedThreadSafe<Delegate>;
36 ~Delegate() {} 36 ~Delegate() {}
37 37
38 JavaObjectWeakGlobalRef j_view_android_; 38 JavaObjectWeakGlobalRef j_view_android_;
39 39
40 DISALLOW_COPY_AND_ASSIGN(Delegate); 40 DISALLOW_COPY_AND_ASSIGN(Delegate);
41 }; 41 };
42 42
43 void PowerSaveBlockerImpl::Delegate::ApplyBlock() { 43 void PowerSaveBlockerImpl::Delegate::ApplyBlock() {
44 DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI)); 44 DCHECK_CURRENTLY_ON(BrowserThread::UI);
45 JNIEnv* env = AttachCurrentThread(); 45 JNIEnv* env = AttachCurrentThread();
46 ScopedJavaLocalRef<jobject> j_object = j_view_android_.get(env); 46 ScopedJavaLocalRef<jobject> j_object = j_view_android_.get(env);
47 if (j_object.obj()) 47 if (j_object.obj())
48 Java_PowerSaveBlocker_applyBlock(env, j_object.obj()); 48 Java_PowerSaveBlocker_applyBlock(env, j_object.obj());
49 } 49 }
50 50
51 void PowerSaveBlockerImpl::Delegate::RemoveBlock() { 51 void PowerSaveBlockerImpl::Delegate::RemoveBlock() {
52 DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI)); 52 DCHECK_CURRENTLY_ON(BrowserThread::UI);
53 JNIEnv* env = AttachCurrentThread(); 53 JNIEnv* env = AttachCurrentThread();
54 ScopedJavaLocalRef<jobject> j_object = j_view_android_.get(env); 54 ScopedJavaLocalRef<jobject> j_object = j_view_android_.get(env);
55 if (j_object.obj()) 55 if (j_object.obj())
56 Java_PowerSaveBlocker_removeBlock(env, j_object.obj()); 56 Java_PowerSaveBlocker_removeBlock(env, j_object.obj());
57 } 57 }
58 58
59 PowerSaveBlockerImpl::PowerSaveBlockerImpl(PowerSaveBlockerType type, 59 PowerSaveBlockerImpl::PowerSaveBlockerImpl(PowerSaveBlockerType type,
60 Reason reason, 60 Reason reason,
61 const std::string& description) { 61 const std::string& description) {
62 // Don't support kPowerSaveBlockPreventAppSuspension 62 // Don't support kPowerSaveBlockPreventAppSuspension
(...skipping 16 matching lines...) Expand all
79 BrowserThread::PostTask( 79 BrowserThread::PostTask(
80 BrowserThread::UI, FROM_HERE, 80 BrowserThread::UI, FROM_HERE,
81 base::Bind(&Delegate::ApplyBlock, delegate_)); 81 base::Bind(&Delegate::ApplyBlock, delegate_));
82 } 82 }
83 83
84 bool RegisterPowerSaveBlocker(JNIEnv* env) { 84 bool RegisterPowerSaveBlocker(JNIEnv* env) {
85 return RegisterNativesImpl(env); 85 return RegisterNativesImpl(env);
86 } 86 }
87 87
88 } // namespace content 88 } // namespace content
OLDNEW
« no previous file with comments | « content/browser/power_profiler/power_profiler_service.cc ('k') | content/browser/power_save_blocker_chromeos.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698