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

Unified Diff: chrome/browser/chromeos/memory/oom_memory_details.cc

Issue 1188823002: [MemoryPressure] Move chrome/browser/chromeos/* to chrome/browser/memory/*. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Addressed sky@ comments. Created 5 years, 6 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/memory/oom_memory_details.cc
diff --git a/chrome/browser/chromeos/memory/oom_memory_details.cc b/chrome/browser/chromeos/memory/oom_memory_details.cc
deleted file mode 100644
index effd3178b543157eb1fff05873a706bd430a1907..0000000000000000000000000000000000000000
--- a/chrome/browser/chromeos/memory/oom_memory_details.cc
+++ /dev/null
@@ -1,52 +0,0 @@
-// Copyright 2015 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.
-
-#include "chrome/browser/chromeos/memory/oom_memory_details.h"
-
-#include "base/logging.h"
-#include "base/process/process_metrics.h"
-#include "base/strings/utf_string_conversions.h"
-#include "chrome/browser/chromeos/memory/oom_memory_details.h"
-#include "ui/base/text/bytes_formatting.h"
-
-namespace chromeos {
-
-// static
-void OomMemoryDetails::Log(const std::string& title,
- const base::Closure& callback) {
- // Deletes itself upon completion.
- OomMemoryDetails* details = new OomMemoryDetails(title, callback);
- details->StartFetch(MemoryDetails::FROM_CHROME_ONLY);
-}
-
-OomMemoryDetails::OomMemoryDetails(const std::string& title,
- const base::Closure& callback)
- : title_(title), callback_(callback) {
- AddRef(); // Released in OnDetailsAvailable().
- start_time_ = base::TimeTicks::Now();
-}
-
-OomMemoryDetails::~OomMemoryDetails() {
-}
-
-void OomMemoryDetails::OnDetailsAvailable() {
- base::TimeDelta delta = base::TimeTicks::Now() - start_time_;
- // These logs are collected by user feedback reports. We want them to help
- // diagnose user-reported problems with frequently discarded tabs.
- std::string log_string = ToLogString();
- base::SystemMemoryInfoKB memory;
- if (base::GetSystemMemoryInfo(&memory) && memory.gem_size != -1) {
- log_string += "Graphics ";
- log_string += base::UTF16ToASCII(ui::FormatBytes(memory.gem_size));
- }
- LOG(WARNING) << title_ << " (" << delta.InMilliseconds() << " ms):\n"
- << log_string;
- if (!callback_.is_null())
- callback_.Run();
- // Delete ourselves so we don't have to worry about OomPriorityManager
- // deleting us when we're still working.
- Release();
-}
-
-} // namespace chromeos
« no previous file with comments | « chrome/browser/chromeos/memory/oom_memory_details.h ('k') | chrome/browser/chromeos/memory/oom_priority_manager.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698