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

Issue 2874553004: Remove memory_pressure component. (Closed)

Created:
3 years, 7 months ago by chrisha
Modified:
3 years, 7 months ago
CC:
chromium-reviews, droger+watchlist_chromium.org, blundell+watchlist_chromium.org, lcwu+watch_chromium.org, sdefresne+watchlist_chromium.org, halliwell+watch_chromium.org, alokp+watch_chromium.org
Target Ref:
refs/heads/master
Project:
chromium
Visibility:
Public.

Description

Remove memory_pressure component. This was part of an aborted refactor, which was abandoned due to memory coordinator work (which has deprecated the memory pressure subsystem). When memory coordinator is finished all uses of memory pressure will be migrated to it. BUG= TBR=sdefresne@chromium.org, halliwell@chromium.org Review-Url: https://codereview.chromium.org/2874553004 Cr-Commit-Position: refs/heads/master@{#471027} Committed: https://chromium.googlesource.com/chromium/src/+/4d1357d65728d99aef9de10f4ab8a914cb09b96d

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+0 lines, -2892 lines) Patch
M chrome/chrome_watcher/BUILD.gn View 1 chunk +0 lines, -1 line 0 comments Download
M chrome/chrome_watcher/DEPS View 1 chunk +0 lines, -1 line 0 comments Download
M chromecast/browser/cast_memory_pressure_monitor.h View 1 chunk +0 lines, -2 lines 0 comments Download
M components/BUILD.gn View 1 chunk +0 lines, -1 line 0 comments Download
D components/memory_pressure/BUILD.gn View 1 chunk +0 lines, -46 lines 0 comments Download
D components/memory_pressure/DEPS View 1 chunk +0 lines, -2 lines 0 comments Download
D components/memory_pressure/OWNERS View 1 chunk +0 lines, -3 lines 0 comments Download
D components/memory_pressure/direct_memory_pressure_calculator.h View 1 chunk +0 lines, -14 lines 0 comments Download
D components/memory_pressure/direct_memory_pressure_calculator_linux.h View 1 chunk +0 lines, -111 lines 0 comments Download
D components/memory_pressure/direct_memory_pressure_calculator_linux.cc View 1 chunk +0 lines, -169 lines 0 comments Download
D components/memory_pressure/direct_memory_pressure_calculator_linux_unittest.cc View 1 chunk +0 lines, -244 lines 0 comments Download
D components/memory_pressure/direct_memory_pressure_calculator_win.h View 1 chunk +0 lines, -72 lines 0 comments Download
D components/memory_pressure/direct_memory_pressure_calculator_win.cc View 1 chunk +0 lines, -116 lines 0 comments Download
D components/memory_pressure/direct_memory_pressure_calculator_win_unittest.cc View 1 chunk +0 lines, -181 lines 0 comments Download
D components/memory_pressure/filtered_memory_pressure_calculator.h View 1 chunk +0 lines, -84 lines 0 comments Download
D components/memory_pressure/filtered_memory_pressure_calculator.cc View 1 chunk +0 lines, -79 lines 0 comments Download
D components/memory_pressure/filtered_memory_pressure_calculator_unittest.cc View 1 chunk +0 lines, -249 lines 0 comments Download
D components/memory_pressure/memory_pressure_calculator.h View 1 chunk +0 lines, -33 lines 0 comments Download
D components/memory_pressure/memory_pressure_listener.h View 1 chunk +0 lines, -92 lines 0 comments Download
D components/memory_pressure/memory_pressure_listener.cc View 1 chunk +0 lines, -61 lines 0 comments Download
D components/memory_pressure/memory_pressure_monitor.h View 1 chunk +0 lines, -210 lines 0 comments Download
D components/memory_pressure/memory_pressure_monitor.cc View 1 chunk +0 lines, -283 lines 0 comments Download
D components/memory_pressure/memory_pressure_monitor_unittest.cc View 1 chunk +0 lines, -389 lines 0 comments Download
D components/memory_pressure/memory_pressure_stats_collector.h View 1 chunk +0 lines, -94 lines 0 comments Download
D components/memory_pressure/memory_pressure_stats_collector.cc View 1 chunk +0 lines, -124 lines 0 comments Download
D components/memory_pressure/memory_pressure_stats_collector_unittest.cc View 1 chunk +0 lines, -130 lines 0 comments Download
D components/memory_pressure/test_memory_pressure_calculator.h View 1 chunk +0 lines, -55 lines 0 comments Download
D components/memory_pressure/test_memory_pressure_calculator.cc View 1 chunk +0 lines, -46 lines 0 comments Download

Messages

Total messages: 16 (10 generated)
chrisha
PTAL?
3 years, 7 months ago (2017-05-11 13:05:24 UTC) #6
brettw
lgtm, thanks
3 years, 7 months ago (2017-05-11 16:53:53 UTC) #7
chrisha
3 years, 7 months ago (2017-05-11 17:50:28 UTC) #9
Sigurður Ásgeirsson
chrome_watcher LGTM
3 years, 7 months ago (2017-05-11 17:54:12 UTC) #11
commit-bot: I haz the power
CQ is trying da patch. Follow status at: https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2874553004/1
3 years, 7 months ago (2017-05-11 17:58:03 UTC) #13
commit-bot: I haz the power
3 years, 7 months ago (2017-05-11 19:02:13 UTC) #16
Message was sent while issue was closed.
Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/4d1357d65728d99aef9de10f4ab8...

Powered by Google App Engine
This is Rietveld 408576698