Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1549)

Unified Diff: chrome/browser/chromeos/extensions/file_manager/event_router.h

Issue 507293002: Enrich fileBrowserPrivate.onFileTransfersUpdated event to support displaying total number of jobs. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/chromeos/extensions/file_manager/event_router.h
diff --git a/chrome/browser/chromeos/extensions/file_manager/event_router.h b/chrome/browser/chromeos/extensions/file_manager/event_router.h
index 7e094d6a1926f7a9a24149e389bd7ba7a530c60b..65e875e5cd2b9460d04988c595d3d3c44f09c83e 100644
--- a/chrome/browser/chromeos/extensions/file_manager/event_router.h
+++ b/chrome/browser/chromeos/extensions/file_manager/event_router.h
@@ -181,11 +181,6 @@ class EventRouter
void ShowRemovableDeviceInFileManager(VolumeType type,
const base::FilePath& mount_path);
- // Sends onFileTranferUpdated to extensions if needed. If |always| is true,
- // it sends the event always. Otherwise, it sends the event if enough time has
- // passed from the previous event so as not to make extension busy.
- void SendDriveFileTransferEvent(bool always);
-
// Manages the list of currently active Drive file transfer jobs.
struct DriveJobInfoWithStatus {
DriveJobInfoWithStatus();
@@ -194,9 +189,20 @@ class EventRouter
drive::JobInfo job_info;
std::string status;
};
+
+ // Schedule onFileTransferUpdated event.
+ void ScheduleDriveFileTransferEvent(const drive::JobInfo& job_info,
+ const std::string& status,
+ bool immediate);
+
+ // Sends onFileTranferUpdated to extensions if after nextSendTime_.
+ void SendDriveFileTransferEvent();
+
std::map<drive::JobID, DriveJobInfoWithStatus> drive_jobs_;
+ scoped_ptr<DriveJobInfoWithStatus> drive_job_info_with_status_;
base::Time last_file_transfer_event_;
base::Time last_copy_progress_event_;
+ base::Time next_send_file_transfer_event_;
WatcherMap file_watchers_;
scoped_ptr<PrefChangeRegistrar> pref_change_registrar_;

Powered by Google App Engine
This is Rietveld 408576698