Move content/browser/power_save_blocker to //device/power_save_blocker

content/public/browser/power_save_blocker_factory.h

Index: content/public/browser/power_save_blocker_factory.h
diff --git a/content/public/browser/power_save_blocker_factory.h b/content/public/browser/power_save_blocker_factory.h
index a40c0ce098bcec1bc889786b1357347aec55584e..1c9a593c10053551de34c042c73af19bb1fc1cf5 100644
--- a/content/public/browser/power_save_blocker_factory.h
+++ b/content/public/browser/power_save_blocker_factory.h
@@ -11,13 +11,13 @@
 #include "base/memory/ref_counted.h"
 #include "base/sequenced_task_runner.h"
 #include "content/common/content_export.h"
-#include "content/public/browser/power_save_blocker.h"
+#include "device/power_save_blocker/power_save_blocker.h"
 
 namespace content {
 
-CONTENT_EXPORT std::unique_ptr<PowerSaveBlocker> CreatePowerSaveBlocker(
-    PowerSaveBlocker::PowerSaveBlockerType type,
-    PowerSaveBlocker::Reason reason,
+CONTENT_EXPORT std::unique_ptr<device::PowerSaveBlocker> CreatePowerSaveBlocker(

[jam, 2016/06/16 17:04:04]

why isn't this moved?

[scottmg, 2016/06/16 18:10:34]

This was just supposed to be a helper for content code so that it's not so
verbose to create a PowerSaveBlocker when in content.
https://cs.chromium.org/chromium/src/content/browser/power_save_blocker_facto...

But I forgot to move it out of public/, did that now.

Doing that meant I needed to add a +browser_thread.h to
content/browser/loader/DEPS but it's not really a new dependency. I'll take a
look at that separately in the future, I'm not sure how loader should be getting
task runners at this point.

[jam, 2016/06/16 18:19:09]

we can pass them at construction time from content.

i think this factory serves very little purpose, and we should just remove it in
the followup (especially once the interface is gone)

+    device::PowerSaveBlocker::PowerSaveBlockerType type,
+    device::PowerSaveBlocker::Reason reason,
     const std::string& description);
 
 }  // namespace content