Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1375)

Unified Diff: chrome/android/java/src/org/chromium/chrome/browser/webapps/WebApkActivity.java

Issue 2705133002: android: Allow registering multiple CreationParams (Closed)
Patch Set: rebase Created 3 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/android/java/src/org/chromium/chrome/browser/webapps/WebApkActivity.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/webapps/WebApkActivity.java b/chrome/android/java/src/org/chromium/chrome/browser/webapps/WebApkActivity.java
index a5a65216bdf5bd5452dab9f44828af5b61b45989..de83e602556dd75d3101b5d4dc6d72d04b8c0eb1 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/webapps/WebApkActivity.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/webapps/WebApkActivity.java
@@ -8,7 +8,6 @@ import android.content.Intent;
import org.chromium.base.ContextUtils;
import org.chromium.base.library_loader.LibraryProcessType;
-import org.chromium.chrome.browser.ChromeApplication;
import org.chromium.chrome.browser.ShortcutHelper;
import org.chromium.chrome.browser.externalnav.ExternalNavigationParams;
import org.chromium.chrome.browser.metrics.WebApkUma;
@@ -34,6 +33,9 @@ public class WebApkActivity extends WebappActivity {
/** Indicates whether launching renderer in WebAPK process is enabled. */
private boolean mCanLaunchRendererInWebApkProcess;
+ private final ChildProcessCreationParams mDefaultParams =
+ ChildProcessCreationParams.getDefault();
+
@Override
protected void onNewIntent(Intent intent) {
super.onNewIntent(intent);
@@ -185,14 +187,13 @@ public class WebApkActivity extends WebappActivity {
*/
private void initializeChildProcessCreationParams(boolean isForWebApk) {
// TODO(hanxi): crbug.com/664530. WebAPKs shouldn't use a global ChildProcessCreationParams.
- ChromeApplication chrome = (ChromeApplication) ContextUtils.getApplicationContext();
- ChildProcessCreationParams params = chrome.getChildProcessCreationParams();
+ ChildProcessCreationParams params = mDefaultParams;
if (isForWebApk) {
boolean isExternalService = false;
params = new ChildProcessCreationParams(getWebappInfo().webApkPackageName(),
isExternalService, LibraryProcessType.PROCESS_CHILD);
}
- ChildProcessCreationParams.set(params);
+ ChildProcessCreationParams.registerDefault(params);
}
@Override

Powered by Google App Engine
This is Rietveld 408576698