Pass SequencedTaskRunner to PowerSaveBlocker for ui/file operations

content/browser/power_save_blocker_impl.h

Index: content/browser/power_save_blocker_impl.h
diff --git a/content/browser/power_save_blocker_impl.h b/content/browser/power_save_blocker_impl.h
index a01fc60928a7230c0eb87215291430287bdb56ff..c03430a67d6563e1c63ac6a7bb7883647be1f5e4 100644
--- a/content/browser/power_save_blocker_impl.h
+++ b/content/browser/power_save_blocker_impl.h
@@ -18,9 +18,12 @@ class WebContents;
 
 class PowerSaveBlockerImpl : public PowerSaveBlocker {
  public:
-  PowerSaveBlockerImpl(PowerSaveBlockerType type,
-                       Reason reason,
-                       const std::string& description);
+  PowerSaveBlockerImpl(
+      PowerSaveBlockerType type,
+      Reason reason,
+      const std::string& description,
+      scoped_refptr<base::SequencedTaskRunner> ui_task_runner,
+      scoped_refptr<base::SequencedTaskRunner> file_task_runner);
   ~PowerSaveBlockerImpl() override;
 
 #if defined(OS_ANDROID)
@@ -52,6 +55,9 @@ class PowerSaveBlockerImpl : public PowerSaveBlocker {
   scoped_refptr<Delegate> freedesktop_suspend_delegate_;
 #endif
 
+  scoped_refptr<base::SequencedTaskRunner> ui_task_runner_;
+  scoped_refptr<base::SequencedTaskRunner> file_task_runner_;

[hashimoto, 2016/05/24 06:02:42]

This task runner has nothing to do with files.
How about renaming this to something like blocking_task_runner_?

This task runner is used only by power_save_blocker_x11.cc and it's using this
task runner to perform D-Bus method calls.

[scottmg, 2016/05/24 17:18:03]

Done.

+
   DISALLOW_COPY_AND_ASSIGN(PowerSaveBlockerImpl);
 };