Refactor screenshot directory source

chrome/browser/ui/webui/screenshot_source.h

Index: chrome/browser/ui/webui/screenshot_source.h
diff --git a/chrome/browser/ui/webui/screenshot_source.h b/chrome/browser/ui/webui/screenshot_source.h
index 0b4201e9f184b073ceff3b2a60953e0b0fece6cd..2ba54d996f301956454e76bb638967f41a002a3f 100644
--- a/chrome/browser/ui/webui/screenshot_source.h
+++ b/chrome/browser/ui/webui/screenshot_source.h
@@ -16,6 +16,8 @@
 #if defined(OS_CHROMEOS)
 #include "chrome/browser/chromeos/gdata/drive_resource_metadata.h"
 #include "chrome/browser/chromeos/gdata/gdata_errorcode.h"
+#include "chrome/browser/chromeos/gdata/gdata_util.h"
+#include "chrome/browser/chromeos/login/user_manager.h"

[Jun Mukai, 2012/09/06 09:30:06]

These files seem not referred in the header file.  Could you move them to .cc?

 #endif
 
 typedef std::vector<unsigned char> ScreenshotData;
@@ -32,6 +34,14 @@ class ScreenshotSource : public ChromeURLDataManager::DataSource {
       std::vector<unsigned char>* current_screenshot,
       Profile* profile);
 
+  // Queries the browser process to determine if screenshots are disabled.
+  static bool AreScreenshotsDisabled();
+  // Get the basefilename for screenshots
+  static std::string GetScreenshotBaseFilename();
+  // Common access for the screenshot directory, parameter is set to the
+  // requested directory and return value of true is given upon success.
+  static bool GetScreenshotDirectory(FilePath* directory);
+
   // Called when the network layer has requested a resource underneath
   // the path we registered.
   virtual void StartDataRequest(const std::string& path,