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

Unified Diff: blimp/client/app/android/java/src/org/chromium/blimp/BrowserRestartActivity.java

Issue 2493333002: Move Java Blimp shell code to app subpackage (Closed)
Patch Set: Merge branch 'refs/heads/master' into blimp-shell-integration Created 4 years, 1 month 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: blimp/client/app/android/java/src/org/chromium/blimp/BrowserRestartActivity.java
diff --git a/blimp/client/app/android/java/src/org/chromium/blimp/BrowserRestartActivity.java b/blimp/client/app/android/java/src/org/chromium/blimp/BrowserRestartActivity.java
deleted file mode 100644
index ef7461ee7c12cca4e4bee0f301d0d4d0c7a682a0..0000000000000000000000000000000000000000
--- a/blimp/client/app/android/java/src/org/chromium/blimp/BrowserRestartActivity.java
+++ /dev/null
@@ -1,74 +0,0 @@
-// Copyright 2016 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.blimp;
-
-import android.app.Activity;
-import android.content.Context;
-import android.content.Intent;
-import android.os.Bundle;
-import android.os.Process;
-
-/**
- * Kills and (optionally) restarts the main Blimp process, then immediately kills itself.
- *
- * Starting this Activity requires passing in the process ID (the Intent should have the value of
- * Process#myPid() as an extra).
- *
- * This Activity runs on a separate process from the main Blimp browser and cannot see the main
- * process' Activities. It is an workaround for crbug.com/515919 which doesn't use AlarmManager.
- */
-public class BrowserRestartActivity extends Activity {
- private static final String EXTRA_MAIN_PID =
- "org.chromium.blimp.browser.BrowserRestartActivity.main_pid";
- private static final String EXTRA_RESTART =
- "org.chromium.blimp.browser.BrowserRestartActivity.restart";
-
- private static final String TAG = "BrowserRestartActivity";
-
- @Override
- public void onCreate(Bundle savedInstanceState) {
- super.onCreate(savedInstanceState);
- destroyProcess(getIntent());
- }
-
- @Override
- public void onNewIntent(Intent intent) {
- destroyProcess(getIntent());
- }
-
- private void destroyProcess(Intent intent) {
- // Kill the main Blimp process.
- int mainBrowserPid = intent.getIntExtra(BrowserRestartActivity.EXTRA_MAIN_PID, -1);
- assert mainBrowserPid != -1;
- Process.killProcess(mainBrowserPid);
-
- // Fire an Intent to restart Blimp.
- boolean restart = intent.getBooleanExtra(BrowserRestartActivity.EXTRA_RESTART, false);
- if (restart) {
- Intent restartIntent = new Intent(Intent.ACTION_MAIN);
- restartIntent.setPackage(getPackageName());
- restartIntent.setFlags(Intent.FLAG_ACTIVITY_NEW_TASK);
- startActivity(restartIntent);
- }
-
- // Kill this process.
- finish();
- Process.killProcess(Process.myPid());
- }
-
- /**
- * Helper method to create an Intent to restart the browser.
- * @param context The current android context
- * @return Intent to be fired
- */
- public static Intent createRestartIntent(Context context) {
- Intent intent = new Intent();
- intent.setClassName(context.getPackageName(), BrowserRestartActivity.class.getName());
- intent.setFlags(Intent.FLAG_ACTIVITY_NEW_TASK);
- intent.putExtra(BrowserRestartActivity.EXTRA_MAIN_PID, Process.myPid());
- intent.putExtra(BrowserRestartActivity.EXTRA_RESTART, true);
- return intent;
- }
-}

Powered by Google App Engine
This is Rietveld 408576698