New issue
Advanced search Search tips

Issue 644397 link

Starred by 3 users

Issue metadata

Status: Fixed
Owner:
Closed: Apr 2017
Cc:
EstimatedDays: ----
NextAction: ----
OS: Linux , Android , Chrome , Mac
Pri: 3
Type: Bug



Sign in to add a comment

Create MemoryManager classes for other operating systems.

Project Member Reported by bcwh...@chromium.org, Sep 6 2016

Issue description

Implement the MemoryMonitor class for operating systems other than Windows.

 
Project Member

Comment 1 by bugdroid1@chromium.org, Sep 9 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/8ceca0e580766a9f91cf655a52de1ebd97d8443d

commit 8ceca0e580766a9f91cf655a52de1ebd97d8443d
Author: bcwhite <bcwhite@chromium.org>
Date: Fri Sep 09 16:57:12 2016

Added MemoryMonitor for Linux (with test).

Extraction of common code also means some changes to the
Windows memory-monitor.

BUG= 644397 

Review-Url: https://codereview.chromium.org/2310193002
Cr-Commit-Position: refs/heads/master@{#417607}

[modify] https://crrev.com/8ceca0e580766a9f91cf655a52de1ebd97d8443d/components/memory_coordinator/browser/BUILD.gn
[add] https://crrev.com/8ceca0e580766a9f91cf655a52de1ebd97d8443d/components/memory_coordinator/browser/memory_monitor.cc
[modify] https://crrev.com/8ceca0e580766a9f91cf655a52de1ebd97d8443d/components/memory_coordinator/browser/memory_monitor.h
[add] https://crrev.com/8ceca0e580766a9f91cf655a52de1ebd97d8443d/components/memory_coordinator/browser/memory_monitor_linux.cc
[add] https://crrev.com/8ceca0e580766a9f91cf655a52de1ebd97d8443d/components/memory_coordinator/browser/memory_monitor_linux.h
[add] https://crrev.com/8ceca0e580766a9f91cf655a52de1ebd97d8443d/components/memory_coordinator/browser/memory_monitor_linux_unittest.cc
[modify] https://crrev.com/8ceca0e580766a9f91cf655a52de1ebd97d8443d/components/memory_coordinator/browser/memory_monitor_win.cc
[modify] https://crrev.com/8ceca0e580766a9f91cf655a52de1ebd97d8443d/components/memory_coordinator/browser/memory_monitor_win.h
[modify] https://crrev.com/8ceca0e580766a9f91cf655a52de1ebd97d8443d/components/memory_coordinator/browser/memory_monitor_win_unittest.cc
[add] https://crrev.com/8ceca0e580766a9f91cf655a52de1ebd97d8443d/components/memory_coordinator/browser/test_memory_monitor.cc
[add] https://crrev.com/8ceca0e580766a9f91cf655a52de1ebd97d8443d/components/memory_coordinator/browser/test_memory_monitor.h

Status: Fixed (was: Started)
My understanding is that this is now obsolete.  The uncommitted Mac one is here:
https://codereview.chromium.org/2360433002/  (closed)

Sign in to add a comment