Add MemoryCoordinatorDelegate

content/public/browser/memory_coordinator_delegate.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_PUBLIC_BROWSER_MEMORY_COORDINATOR_DELEGATE_H_
+#define CONTENT_PUBLIC_BROWSER_MEMORY_COORDINATOR_DELEGATE_H_
+
+#include "base/memory/weak_ptr.h"
+#include "content/common/content_export.h"
+
+namespace content {
+
+// A delegate class which is used by the memory coordinator.
+class CONTENT_EXPORT MemoryCoordinatorDelegate {
+ public:
+  // Sets an instance of MemoryCoordinatorDelegate.
+  static void Set(const base::WeakPtr<MemoryCoordinatorDelegate>& delegate);

[jam, 2016/10/11 15:50:00]

nit: the general pattern used is for ContentBrowerClient to have a
CreateMemoryCoordinatorDelegate method, instead of a setter here. that way it's
clear to the embedder:
1) when it should create/set it's delegate (i.e. so it's not set too late)
2) to have one place where the embedder can see all the delegates it can create

with that simple change, lgtm

[bashi, 2016/10/13 01:40:56]

Done. I was looking for something like ContentBrowserClient. Thanks for the
information!

+
+  // Returns true when a given backgrounded renderer process can be suspended.
+  virtual bool CanSuspendBackgroundedRenderer(int render_process_id) = 0;
+
+ protected:
+  virtual ~MemoryCoordinatorDelegate() {}
+};
+
+}  // namespace content
+
+#endif  // CONTENT_PUBLIC_BROWSER_MEMORY_COORDINATOR_DELEGATE_H_