Get rid of FileManagerPrivateAPI class

chrome/browser/chromeos/extensions/file_manager/event_router.h

 // Copyright (c) 2012 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.
 
 #ifndef CHROME_BROWSER_CHROMEOS_EXTENSIONS_FILE_MANAGER_EVENT_ROUTER_H_
 #define CHROME_BROWSER_CHROMEOS_EXTENSIONS_FILE_MANAGER_EVENT_ROUTER_H_
 
 #include <map>
 #include <string>
 
 #include "base/basictypes.h"
 #include "base/compiler_specific.h"
 #include "base/files/file_path_watcher.h"
 #include "base/memory/scoped_ptr.h"
 #include "chrome/browser/chromeos/drive/drive_integration_service.h"
 #include "chrome/browser/chromeos/drive/file_system_observer.h"
 #include "chrome/browser/chromeos/drive/job_list.h"
 #include "chrome/browser/chromeos/drive/sync_client.h"
 #include "chrome/browser/chromeos/file_manager/file_watcher.h"
 #include "chrome/browser/chromeos/file_manager/fileapi_util.h"
 #include "chrome/browser/chromeos/file_manager/volume_manager.h"
 #include "chrome/browser/chromeos/file_manager/volume_manager_observer.h"
 #include "chrome/browser/drive/drive_service_interface.h"
 #include "chrome/common/extensions/api/file_manager_private.h"
 #include "chromeos/disks/disk_mount_manager.h"
 #include "chromeos/network/network_state_handler_observer.h"
+#include "components/keyed_service/core/keyed_service.h"
 #include "webkit/browser/fileapi/file_system_operation.h"
 
 class PrefChangeRegistrar;
 class Profile;
 
 using file_manager::util::EntryDefinition;
 
 namespace base {
 class ListValue;
 }
 
 namespace chromeos {
 class NetworkState;
 }
 
 namespace drive {
 class FileChange;
 }
 
 namespace file_manager {
 class DeviceEventRouter;
 
 // Monitors changes in disk mounts, network connection state and preferences
 // affecting File Manager. Dispatches appropriate File Browser events.
-class EventRouter : public chromeos::NetworkStateHandlerObserver,
+class EventRouter : public KeyedService,
+                    public chromeos::NetworkStateHandlerObserver,
                     public drive::FileSystemObserver,
                     public drive::JobListObserver,
                     public drive::DriveServiceObserver,
                     public VolumeManagerObserver {
  public:
   explicit EventRouter(Profile* profile);
   virtual ~EventRouter();
 
-  void Shutdown();
+  // KeyedService overrides.
+  virtual void Shutdown() OVERRIDE;
 
   // Starts observing file system change events.
   void ObserveEvents();

[mtomasz, 2014/09/10 07:29:30]

ObserveEvents() is now not called anymore. Maybe we should just call it from the
constructor?

[satorux1, 2014/09/11 04:12:51]

Good catch! Called it from the constructor and made it private.

 
   typedef base::Callback<void(bool success)> BoolCallback;
 
   // Adds a file watch at |local_path|, associated with |virtual_path|, for
   // an extension with |extension_id|.
   //
   // |callback| will be called with true on success, or false on failure.
   // |callback| must not be null.
   void AddFileWatch(const base::FilePath& local_path,
                     const base::FilePath& virtual_path,
@@ skipping 127 matching lines @@
 
   // Note: This should remain the last member so it'll be destroyed and
   // invalidate the weak pointers before any other members are destroyed.
   base::WeakPtrFactory<EventRouter> weak_factory_;
   DISALLOW_COPY_AND_ASSIGN(EventRouter);
 };
 
 }  // namespace file_manager
 
 #endif  // CHROME_BROWSER_CHROMEOS_EXTENSIONS_FILE_MANAGER_EVENT_ROUTER_H_