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

Side by Side Diff: chrome/browser/feedback/system_logs/log_sources/memory_details_log_source.cc

Issue 625113002: replace OVERRIDE and FINAL with override and final in chrome/browser/[a-i]* (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: fix newly added OVERRIDEs Created 6 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 unified diff | Download patch
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/feedback/system_logs/log_sources/memory_details_log_sou rce.h" 5 #include "chrome/browser/feedback/system_logs/log_sources/memory_details_log_sou rce.h"
6 6
7 #include "chrome/browser/memory_details.h" 7 #include "chrome/browser/memory_details.h"
8 #include "content/public/browser/browser_thread.h" 8 #include "content/public/browser/browser_thread.h"
9 9
10 namespace system_logs { 10 namespace system_logs {
11 11
12 // Reads Chrome memory usage. 12 // Reads Chrome memory usage.
13 class SystemLogsMemoryHandler : public MemoryDetails { 13 class SystemLogsMemoryHandler : public MemoryDetails {
14 public: 14 public:
15 explicit SystemLogsMemoryHandler(const SysLogsSourceCallback& callback) 15 explicit SystemLogsMemoryHandler(const SysLogsSourceCallback& callback)
16 : callback_(callback) {} 16 : callback_(callback) {}
17 17
18 // Sends the data to the callback. 18 // Sends the data to the callback.
19 // MemoryDetails override. 19 // MemoryDetails override.
20 virtual void OnDetailsAvailable() OVERRIDE { 20 virtual void OnDetailsAvailable() override {
21 DCHECK(content::BrowserThread::CurrentlyOn(content::BrowserThread::UI)); 21 DCHECK(content::BrowserThread::CurrentlyOn(content::BrowserThread::UI));
22 22
23 scoped_ptr<SystemLogsResponse> response(new SystemLogsResponse); 23 scoped_ptr<SystemLogsResponse> response(new SystemLogsResponse);
24 (*response)["mem_usage"] = ToLogString(); 24 (*response)["mem_usage"] = ToLogString();
25 callback_.Run(response.get()); 25 callback_.Run(response.get());
26 } 26 }
27 27
28 private: 28 private:
29 virtual ~SystemLogsMemoryHandler() {} 29 virtual ~SystemLogsMemoryHandler() {}
30 SysLogsSourceCallback callback_; 30 SysLogsSourceCallback callback_;
(...skipping 12 matching lines...) Expand all
43 DCHECK(content::BrowserThread::CurrentlyOn(content::BrowserThread::UI)); 43 DCHECK(content::BrowserThread::CurrentlyOn(content::BrowserThread::UI));
44 DCHECK(!callback.is_null()); 44 DCHECK(!callback.is_null());
45 45
46 scoped_refptr<SystemLogsMemoryHandler> 46 scoped_refptr<SystemLogsMemoryHandler>
47 handler(new SystemLogsMemoryHandler(callback)); 47 handler(new SystemLogsMemoryHandler(callback));
48 // TODO(jamescook): Maybe we don't need to update histograms here? 48 // TODO(jamescook): Maybe we don't need to update histograms here?
49 handler->StartFetch(MemoryDetails::UPDATE_USER_METRICS); 49 handler->StartFetch(MemoryDetails::UPDATE_USER_METRICS);
50 } 50 }
51 51
52 } // namespace system_logs 52 } // namespace system_logs
OLDNEW
« no previous file with comments | « chrome/browser/feedback/system_logs/log_sources/memory_details_log_source.h ('k') | chrome/browser/file_select_helper.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698