MemoryCoordinatorClient's destructor should be protected-virtual

base/memory/memory_coordinator_client.h

Index: base/memory/memory_coordinator_client.h
diff --git a/base/memory/memory_coordinator_client.h b/base/memory/memory_coordinator_client.h
index 6e0ce6309eb1d52675cac7428db9cc77a6616f97..d63b17cc1f8b88779dfaa83b7a677529dbc37573 100644
--- a/base/memory/memory_coordinator_client.h
+++ b/base/memory/memory_coordinator_client.h
@@ -43,8 +43,6 @@ enum class MemoryState {
 // threading guarantees and ownership management.
 class BASE_EXPORT MemoryCoordinatorClient {
  public:
-  virtual ~MemoryCoordinatorClient() {}
-
   // Called when memory state has changed. Any transition can occur except for
   // UNKNOWN. General guidelines are:
   //  * NORMAL:    Restore the default settings for memory allocation/usage if
@@ -52,6 +50,9 @@ class BASE_EXPORT MemoryCoordinatorClient {
   //  * THROTTLED: Use smaller limits for memory allocations and caches.
   //  * SUSPENDED: Purge memory.
   virtual void OnMemoryStateChange(MemoryState state) = 0;
+
+protected:
+  virtual ~MemoryCoordinatorClient() {}
 };
 
 }  // namespace base