Remove content/browser/power_save_blocker_factory.*

content/browser/wake_lock/wake_lock_service_context.cc

Index: content/browser/wake_lock/wake_lock_service_context.cc
diff --git a/content/browser/wake_lock/wake_lock_service_context.cc b/content/browser/wake_lock/wake_lock_service_context.cc
index 48d3454e703236577b4a0d174b04041ca1b80dcf..cbc2d8765602d44e38c3343752795ecd39798d49 100644
--- a/content/browser/wake_lock/wake_lock_service_context.cc
+++ b/content/browser/wake_lock/wake_lock_service_context.cc
@@ -8,12 +8,12 @@
 
 #include "base/bind.h"
 #include "build/build_config.h"
-#include "content/browser/power_save_blocker_factory.h"
 #include "content/public/browser/browser_thread.h"
 #include "content/public/browser/render_frame_host.h"
 #include "content/public/browser/render_process_host.h"
 #include "content/public/browser/web_contents.h"
 #include "content/public/common/service_registry.h"
+#include "device/power_save_blocker/power_save_blocker.h"
 #include "device/power_save_blocker/power_save_blocker_impl.h"
 
 namespace content {
@@ -68,9 +68,11 @@ bool WakeLockServiceContext::HasWakeLockForTests() const {
 
 void WakeLockServiceContext::CreateWakeLock() {
   DCHECK(!wake_lock_);
-  wake_lock_ = CreatePowerSaveBlocker(
+  wake_lock_ = device::PowerSaveBlocker::CreateWithTaskRunners(

[jam, 2016/06/17 22:11:57]

can we also get rid of this method?

what about the interface and impl as well? we only had that separation because
of the content api. we don't need that anymore.

       device::PowerSaveBlocker::kPowerSaveBlockPreventDisplaySleep,
-      device::PowerSaveBlocker::kReasonOther, "Wake Lock API");
+      device::PowerSaveBlocker::kReasonOther, "Wake Lock API",
+      BrowserThread::GetMessageLoopProxyForThread(BrowserThread::UI),
+      BrowserThread::GetMessageLoopProxyForThread(BrowserThread::FILE));
 
 #if defined(OS_ANDROID)
   // On Android, additionaly associate the blocker with this WebContents.