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

Unified Diff: chrome/browser/ui/webui/chromeos/drive_internals_ui.cc

Issue 11106009: drive: Add the local largest changestamp to chrome:drive-internals (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: address comments Created 8 years, 2 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/ui/webui/chromeos/drive_internals_ui.cc
diff --git a/chrome/browser/ui/webui/chromeos/drive_internals_ui.cc b/chrome/browser/ui/webui/chromeos/drive_internals_ui.cc
index 047c7abd58cae3e117741a458ee9e38a8087cbbb..a0c6b15a597a6d22ef40d318d4bb62110da79d8c 100644
--- a/chrome/browser/ui/webui/chromeos/drive_internals_ui.cc
+++ b/chrome/browser/ui/webui/chromeos/drive_internals_ui.cc
@@ -10,12 +10,12 @@
#include "base/memory/scoped_vector.h"
#include "base/memory/weak_ptr.h"
#include "base/path_service.h"
-#include "base/string_number_conversions.h"
#include "base/stringprintf.h"
#include "base/sys_info.h"
#include "chrome/browser/chromeos/drive/drive.pb.h"
#include "chrome/browser/chromeos/drive/drive_cache.h"
#include "chrome/browser/chromeos/drive/drive_file_system_interface.h"
+#include "chrome/browser/chromeos/drive/drive_resource_metadata.h"
#include "chrome/browser/chromeos/drive/drive_system_service.h"
#include "chrome/browser/google_apis/auth_service.h"
#include "chrome/browser/google_apis/drive_api_parser.h"
@@ -237,9 +237,8 @@ void DriveInternalsWebUIHandler::OnGetAccountMetadata(
about_resource->quota_bytes_total());
account_metadata.SetDouble("account-quota-used",
about_resource->quota_bytes_used());
- account_metadata.SetString(
- "account-largest-changestamp",
- base::Int64ToString(about_resource->largest_change_id()));
+ account_metadata.SetDouble("account-largest-changestamp-remote",
+ about_resource->largest_change_id());
// TODO(haruki): Fill installed Drive apps for Drive API.
// http://crbug.com/154241
@@ -251,9 +250,8 @@ void DriveInternalsWebUIHandler::OnGetAccountMetadata(
account_metadata.SetDouble("account-quota-total",
feed->quota_bytes_total());
account_metadata.SetDouble("account-quota-used", feed->quota_bytes_used());
- account_metadata.SetString(
- "account-largest-changestamp",
- base::Int64ToString(feed->largest_changestamp()));
+ account_metadata.SetDouble("account-largest-changestamp-remote",
+ feed->largest_changestamp());
base::ListValue* installed_apps = new base::ListValue();
for (size_t i = 0; i < feed->installed_apps().size(); ++i) {
@@ -269,6 +267,12 @@ void DriveInternalsWebUIHandler::OnGetAccountMetadata(
account_metadata.Set("installed-apps", installed_apps);
}
+ // Add the local largest chargestamp.
+ const drive::DriveFileSystemMetadata metadata =
+ GetSystemService()->file_system()->GetMetadata();
+ account_metadata.SetDouble("account-largest-changestamp-local",
+ metadata.largest_changestamp);
+
web_ui()->CallJavascriptFunction("updateAccountMetadata", account_metadata);
}

Powered by Google App Engine
This is Rietveld 408576698