Wake Lock API implementation (Chromium part)

content/renderer/wake_lock/wake_lock_dispatcher.cc

+// Copyright 2015 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "content/renderer/wake_lock/wake_lock_dispatcher.h"
+
+#include "content/public/common/service_registry.h"
+#include "content/public/renderer/render_frame.h"

[mlamouri (slow - plz ping), 2015/08/21 10:03:36]

nit: not sure you need that header.

[alogvinov, 2015/08/26 14:47:19]

That header is needed, see render_frame()->GetServiceRegistry() below.

+
+namespace content {
+
+WakeLockDispatcher::WakeLockDispatcher(RenderFrame* render_frame)
+    : RenderFrameObserver(render_frame),
+      blink::WebWakeLockClient() {
+}
+
+WakeLockDispatcher::~WakeLockDispatcher() {
+}
+
+void WakeLockDispatcher::requestKeepScreenAwake(bool keepScreenAwake) {
+  if (!wake_lock_service_) {
+    render_frame()->GetServiceRegistry()->ConnectToRemoteService(
+        mojo::GetProxy(&wake_lock_service_));
+  }
+  if (keepScreenAwake) {

[mlamouri (slow - plz ping), 2015/08/21 10:03:35]

nit: add an empty line before this condition

[alogvinov, 2015/08/26 14:47:19]

Done.

+    wake_lock_service_->RequestWakeLock();
+  } else {
+    wake_lock_service_->CancelWakeLock();
+  }
+}
+
+}  // namespace content