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

Side by Side Diff: content/browser/wake_lock/wake_lock_context_host.cc

Issue 2843353003: Move ownership of PowerSaveBlocker from WakeLockServiceContext to WakeLockServiceImpl (Closed)
Patch Set: error fix, non-frame client. 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 unified diff | Download patch
OLDNEW
1 // Copyright 2017 The Chromium Authors. All rights reserved. 1 // Copyright 2017 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/wake_lock/wake_lock_context_host.h" 5 #include "content/browser/wake_lock/wake_lock_context_host.h"
6 6
7 #include "base/atomic_sequence_num.h" 7 #include "base/atomic_sequence_num.h"
8 #include "base/lazy_instance.h" 8 #include "base/lazy_instance.h"
9 #include "content/public/common/service_manager_connection.h" 9 #include "content/public/common/service_manager_connection.h"
10 #include "device/wake_lock/public/interfaces/wake_lock_context_provider.mojom.h" 10 #include "device/wake_lock/public/interfaces/wake_lock_context_provider.mojom.h"
(...skipping 23 matching lines...) Expand all
34 34
35 // Connect to a WakeLockContext, associating it with |id_| (note that in some 35 // Connect to a WakeLockContext, associating it with |id_| (note that in some
36 // testing contexts, the service manager connection isn't initialized). 36 // testing contexts, the service manager connection isn't initialized).
37 if (ServiceManagerConnection::GetForProcess()) { 37 if (ServiceManagerConnection::GetForProcess()) {
38 service_manager::Connector* connector = 38 service_manager::Connector* connector =
39 ServiceManagerConnection::GetForProcess()->GetConnector(); 39 ServiceManagerConnection::GetForProcess()->GetConnector();
40 DCHECK(connector); 40 DCHECK(connector);
41 device::mojom::WakeLockContextProviderPtr context_provider; 41 device::mojom::WakeLockContextProviderPtr context_provider;
42 connector->BindInterface(device::mojom::kServiceName, 42 connector->BindInterface(device::mojom::kServiceName,
43 mojo::MakeRequest(&context_provider)); 43 mojo::MakeRequest(&context_provider));
44 context_provider->GetContext(id_, mojo::MakeRequest(&wake_lock_context_)); 44 context_provider->GetWakeLockContextForID(
blundell 2017/05/02 11:27:43 nit: Let's make this naming change in the CL that
ke.he 2017/05/04 11:54:13 Done.
45 id_, mojo::MakeRequest(&wake_lock_context_));
45 } 46 }
46 } 47 }
47 48
48 WakeLockContextHost::~WakeLockContextHost() { 49 WakeLockContextHost::~WakeLockContextHost() {
49 g_id_to_context_host.Get().erase(id_); 50 g_id_to_context_host.Get().erase(id_);
50 } 51 }
51 52
52 // static 53 // static
53 gfx::NativeView WakeLockContextHost::GetNativeViewForContext(int context_id) { 54 gfx::NativeView WakeLockContextHost::GetNativeViewForContext(int context_id) {
54 WakeLockContextHost* context_host = ContextHostFromId(context_id); 55 WakeLockContextHost* context_host = ContextHostFromId(context_id);
55 if (context_host) 56 if (context_host)
56 return context_host->web_contents_->GetNativeView(); 57 return context_host->web_contents_->GetNativeView();
57 return nullptr; 58 return nullptr;
58 } 59 }
59 60
60 } // namespace content 61 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698