Chromium Code Reviews

Unified Diff: content/browser/memory/memory_coordinator.h

Issue 2363353002: Document memory coordinator (Closed)
Patch Set: Created 4 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Index: content/browser/memory/memory_coordinator.h
diff --git a/content/browser/memory/memory_coordinator.h b/content/browser/memory/memory_coordinator.h
index c870c91f0316712fb0a4420a9bb42503c2616d82..a9e0095d32e9a4324dafd809e5a9072aaa4bb523 100644
--- a/content/browser/memory/memory_coordinator.h
+++ b/content/browser/memory/memory_coordinator.h
@@ -14,6 +14,15 @@
namespace content {
+// OVERVIEW:
+// Memory coordinator is a content-aware and stateful replacement for
dcheng 2016/09/26 04:55:07 I'm not entirely sure what content-aware means.
bashi 2016/09/26 05:57:22 I borrowed this sentence from the design doc. It s
+// MemoryPressureMonitor/MemoryPressureListener. The memory coordinator
+// system consists of several parts and some of them are located in base/ to
dcheng 2016/09/26 04:55:07 I think it's OK to leave out the second sentence.
bashi 2016/09/26 05:57:22 Yes, we are planning to remove MemoryPressureMonit
+// work around dependency issues.
+//
+// NOTE: Memory coordinator is under development and not fully working.
+// TODO(bashi): Add more explanations when we implement memory coordinator V0.
+
class MemoryCoordinatorHandleImpl;
// MemoryCoordinator is responsible for the whole memory management accross the

Powered by Google App Engine