Add documentation for MemoryCoordinator-related primitives

base/sys_info.h

 // Copyright (c) 2012 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 BASE_SYS_INFO_H_
 #define BASE_SYS_INFO_H_
 
 #include <stddef.h>
 #include <stdint.h>
 
@@ skipping 128 matching lines @@
   // Returns the Android build ID.
   static std::string GetAndroidBuildID();
 
   static int DalvikHeapSizeMB();
   static int DalvikHeapGrowthLimitMB();
 #endif  // defined(OS_ANDROID)
 
   // Returns true if this is a low-end device.
   // Low-end device refers to devices having less than 512M memory in the
   // current implementation.
+  // See a comment in MemoryCoordinator for how to use it.

[Ryan Sleevi, 2017/02/08 15:02:04]

So, I think this is a 'bad' comment, because it couples //base/sys_info to
//base/memory implementation details. Also, see my other remark about trying to
figure out what "in MemoryCoordinator" means

This is part of why I suggest a //base/memory/docs sort of guidance.

[haraken, 2017/02/09 02:37:31]

Done.

   static bool IsLowEndDevice();
 };
 
 }  // namespace base
 
 #endif  // BASE_SYS_INFO_H_