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

Unified Diff: content/browser/web_contents/web_contents_impl.cc

Issue 2921823002: Rationalize WakeLock naming conventions (Closed)
Patch Set: fix Mac compile error Created 3 years, 7 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/web_contents/web_contents_impl.cc
diff --git a/content/browser/web_contents/web_contents_impl.cc b/content/browser/web_contents/web_contents_impl.cc
index 626c4c89bc47ec19d86e615e900ad8069517163c..23eec8df8410f48134fff564e41c8937df6049e6 100644
--- a/content/browser/web_contents/web_contents_impl.cc
+++ b/content/browser/web_contents/web_contents_impl.cc
@@ -2648,7 +2648,7 @@ device::mojom::WakeLockContext* WebContentsImpl::GetWakeLockContext() {
return wake_lock_context_host_->GetWakeLockContext();
}
-device::mojom::WakeLockService* WebContentsImpl::GetRendererWakeLock() {
+device::mojom::WakeLock* WebContentsImpl::GetRendererWakeLock() {
// WebContents creates a long-lived connection to one WakeLockServiceImpl.
blundell 2017/06/05 11:48:10 nit: update naming in this comment. It would be a
juncai 2017/06/05 18:58:25 Done.
// All the frames' requests will be added into the BindingSet of
// WakeLockServiceImpl via this connection.

Powered by Google App Engine
This is Rietveld 408576698