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

Unified Diff: chrome/browser/extensions/api/system_info_memory/memory_info_provider.cc

Issue 18290002: [SystemInfo API] Finish TODOs in SystemInfoProvider (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@dev_rewrite_storage_info_api
Patch Set: Update Created 7 years, 5 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/extensions/api/system_info_memory/memory_info_provider.cc
diff --git a/chrome/browser/extensions/api/system_info_memory/memory_info_provider.cc b/chrome/browser/extensions/api/system_info_memory/memory_info_provider.cc
index f1964bec086e2f0cee173029d73a34bb0226e09e..f951426a18b3ba5754b5fede6fbee2bdebf3a0c7 100644
--- a/chrome/browser/extensions/api/system_info_memory/memory_info_provider.cc
+++ b/chrome/browser/extensions/api/system_info_memory/memory_info_provider.cc
@@ -14,9 +14,13 @@ MemoryInfoProvider::MemoryInfoProvider() {}
MemoryInfoProvider::~MemoryInfoProvider() {}
-bool MemoryInfoProvider::QueryInfo(MemoryInfo* info) {
- info->capacity = static_cast<double>(base::SysInfo::AmountOfPhysicalMemory());
- info->available_capacity =
+const MemoryInfo& MemoryInfoProvider::memory_info() const {
+ return info_;
+}
+
+bool MemoryInfoProvider::QueryInfo() {
+ info_.capacity = static_cast<double>(base::SysInfo::AmountOfPhysicalMemory());
+ info_.available_capacity =
static_cast<double>(base::SysInfo::AmountOfAvailablePhysicalMemory());
return true;
}

Powered by Google App Engine
This is Rietveld 408576698