[Merge m58] Abstracting parameters passed to child processes on Android.

content/public/android/java/src/org/chromium/content/app/ChromiumLinkerParams.java

 // Copyright 2014 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.
 
 package org.chromium.content.app;
 
-import android.content.Intent;
+import android.os.Parcel;
+import android.os.Parcelable;
 
 import java.util.Locale;
 
+import javax.annotation.concurrent.Immutable;
+
 /**
  * A class to hold information passed from the browser process to each
  * service one when using the chromium linker. For more information, read the
  * technical notes in Linker.java.
  */
-public class ChromiumLinkerParams {
+@Immutable
+public class ChromiumLinkerParams implements Parcelable {
     // Use this base address to load native shared libraries. If 0, ignore other members.
     public final long mBaseLoadAddress;
 
     // If true, wait for a shared RELRO Bundle just after loading the libraries.
     public final boolean mWaitForSharedRelro;
 
     // If not empty, name of Linker.TestRunner implementation that needs to be
     // registered in the service process.
     public final String mTestRunnerClassNameForTesting;
 
     // If mTestRunnerClassNameForTesting is not empty, the Linker implementation
     // to force for testing.
     public final int mLinkerImplementationForTesting;
 
-    private static final String EXTRA_LINKER_PARAMS_BASE_LOAD_ADDRESS =
-            "org.chromium.content.common.linker_params.base_load_address";
-
-    private static final String EXTRA_LINKER_PARAMS_WAIT_FOR_SHARED_RELRO =
-            "org.chromium.content.common.linker_params.wait_for_shared_relro";
-
-    private static final String EXTRA_LINKER_PARAMS_TEST_RUNNER_CLASS_NAME =
-            "org.chromium.content.common.linker_params.test_runner_class_name";
-
-    private static final String EXTRA_LINKER_PARAMS_LINKER_IMPLEMENTATION =
-            "org.chromium.content.common.linker_params.linker_implementation";
-
-    public ChromiumLinkerParams(long baseLoadAddress,
-                                boolean waitForSharedRelro) {
+    public ChromiumLinkerParams(long baseLoadAddress, boolean waitForSharedRelro) {
         mBaseLoadAddress = baseLoadAddress;
         mWaitForSharedRelro = waitForSharedRelro;
         mTestRunnerClassNameForTesting = null;
         mLinkerImplementationForTesting = 0;
     }
 
     /**
      * Use this constructor to create a LinkerParams instance for testing.
      */
     public ChromiumLinkerParams(long baseLoadAddress,
                                 boolean waitForSharedRelro,
                                 String testRunnerClassName,
                                 int linkerImplementation) {
         mBaseLoadAddress = baseLoadAddress;
         mWaitForSharedRelro = waitForSharedRelro;
         mTestRunnerClassNameForTesting = testRunnerClassName;
         mLinkerImplementationForTesting = linkerImplementation;
     }
 
-    /**
-     * Use this constructor to recreate a LinkerParams instance from an Intent.
-     *
-     * @param intent An Intent, its content must have been populated by a previous
-     * call to addIntentExtras().
-     */
-    public ChromiumLinkerParams(Intent intent) {
-        mBaseLoadAddress =
-                intent.getLongExtra(EXTRA_LINKER_PARAMS_BASE_LOAD_ADDRESS, 0);
-        mWaitForSharedRelro =
-                intent.getBooleanExtra(EXTRA_LINKER_PARAMS_WAIT_FOR_SHARED_RELRO, false);
-        mTestRunnerClassNameForTesting =
-                intent.getStringExtra(EXTRA_LINKER_PARAMS_TEST_RUNNER_CLASS_NAME);
-        mLinkerImplementationForTesting =
-                intent.getIntExtra(EXTRA_LINKER_PARAMS_LINKER_IMPLEMENTATION, 0);
+    ChromiumLinkerParams(Parcel in) {
+        mBaseLoadAddress = in.readLong();
+        mWaitForSharedRelro = in.readInt() != 0;
+        mTestRunnerClassNameForTesting = in.readString();
+        mLinkerImplementationForTesting = in.readInt();
     }
 
-    /**
-     * Ensure this LinkerParams instance is sent to a service process by adding
-     * it to an intent's extras.
-     *
-     * @param intent An Intent use to start or connect to the child service process.
-     */
-    public void addIntentExtras(Intent intent) {
-        intent.putExtra(EXTRA_LINKER_PARAMS_BASE_LOAD_ADDRESS, mBaseLoadAddress);
-        intent.putExtra(EXTRA_LINKER_PARAMS_WAIT_FOR_SHARED_RELRO, mWaitForSharedRelro);
-        intent.putExtra(EXTRA_LINKER_PARAMS_TEST_RUNNER_CLASS_NAME, mTestRunnerClassNameForTesting);
-        intent.putExtra(EXTRA_LINKER_PARAMS_LINKER_IMPLEMENTATION, mLinkerImplementationForTesting);
+    @Override
+    public int describeContents() {
+        return 0;
     }
 
+    @Override
+    public void writeToParcel(Parcel dest, int flags) {
+        dest.writeLong(mBaseLoadAddress);
+        dest.writeInt(mWaitForSharedRelro ? 1 : 0);
+        dest.writeString(mTestRunnerClassNameForTesting);
+        dest.writeInt(mLinkerImplementationForTesting);
+    }
+
+    public static final Parcelable.Creator<ChromiumLinkerParams> CREATOR =
+            new Parcelable.Creator<ChromiumLinkerParams>() {
+                @Override
+                public ChromiumLinkerParams createFromParcel(Parcel in) {
+                    return new ChromiumLinkerParams(in);
+                }
+
+                @Override
+                public ChromiumLinkerParams[] newArray(int size) {
+                    return new ChromiumLinkerParams[size];
+                }
+            };
+
     // For debugging traces only.
     @Override
     public String toString() {
         return String.format(Locale.US,
                 "LinkerParams(baseLoadAddress:0x%x, waitForSharedRelro:%s, "
                         + "testRunnerClassName:%s, linkerImplementation:%d",
-                mBaseLoadAddress,
-                mWaitForSharedRelro ? "true" : "false",
-                mTestRunnerClassNameForTesting,
-                mLinkerImplementationForTesting);
+                mBaseLoadAddress, Boolean.toString(mWaitForSharedRelro),
+                mTestRunnerClassNameForTesting, mLinkerImplementationForTesting);
     }
 }