android: Limit num renderer to service slots

content/browser/child_process_launcher_helper.h

 // Copyright 2017 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_CHILD_PROCESS_LAUNCHER_HELPER_H_
 #define CONTENT_BROWSER_CHILD_PROCESS_LAUNCHER_HELPER_H_
 
 #include <memory>
 
 #include "base/macros.h"
@@ skipping 152 matching lines @@
   static void SetRegisteredFilesForService(
       const std::string& service_name,
       catalog::RequiredFileMap required_files);
 
   static void ResetRegisteredFilesForTesting();
 
 #if defined(OS_ANDROID)
   void OnChildProcessStarted(JNIEnv* env,
                              const base::android::JavaParamRef<jobject>& obj,
                              jint handle);
+  static size_t GetNumberOfRendererSlots();

[Jay Civelli, 2017/04/03 16:19:48]

I am starting to get weary of adding more stuff in ifdef's here.
I tried hard when I refactored this to get rid of almost all of them.
I am OK with this one for now. My plan is at some point to revisit this ifdef
block and remove it entirely from here.

[boliu, 2017/04/03 16:33:03]

I suppose this doesn't have to go through CPL. RPHI can directly call into some
java thing. But that does involve adding more jni files and whatnot, so I
settled on this. I can add new files if you don't think it's worth it.

[Jay Civelli, 2017/04/03 16:40:50]

That's fine for now.

 #endif  // OS_ANDROID
 
  private:
   friend class base::RefCountedThreadSafe<ChildProcessLauncherHelper>;
 
   ~ChildProcessLauncherHelper();
 
   void LaunchOnLauncherThread();
 
   const mojo::edk::PlatformHandle& mojo_client_handle() const {
@@ skipping 20 matching lines @@
 #if defined(OS_ANDROID)
   base::android::ScopedJavaGlobalRef<jobject> java_peer_;
 #endif
 };
 
 }  // namespace internal
 
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_CHILD_PROCESS_LAUNCHER_HELPER_H_