Document memory coordinator

content/browser/memory/memory_coordinator.h

 // Copyright 2016 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.
 
 #ifndef CONTENT_BROWSER_MEMORY_MEMORY_COORDINATOR_H_
 #define CONTENT_BROWSER_MEMORY_MEMORY_COORDINATOR_H_
 
 #include "base/memory/memory_coordinator_client_registry.h"
 #include "base/memory/singleton.h"
 #include "base/process/process_handle.h"
 #include "content/common/content_export.h"
 #include "content/common/memory_coordinator.mojom.h"
 #include "mojo/public/cpp/bindings/binding.h"
 
 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 seems that this overview is not
needed. Removed.

+// 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. Is this mechanism expected to
completely replace MemoryPressureMonitor/Listener?

[bashi, 2016/09/26 05:57:22]

Yes, we are planning to remove MemoryPressureMonitor/Listener.

+// 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
 // browser and child proceeses. It dispatches memory events to its clients and
 // child processes based on its best knowledge of the memory usage.
 class CONTENT_EXPORT MemoryCoordinator {
  public:
   ~MemoryCoordinator();
 
   // Singleton factory/accessor.
@@ skipping 55 matching lines @@
   // MemoryCoordinator and removed automatically when an underlying binding is
   // disconnected.
   ChildInfoMap children_;
 
   DISALLOW_COPY_AND_ASSIGN(MemoryCoordinator);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_MEMORY_MEMORY_COORDINATOR_H_