Commit 17743601 authored by Ella Ge's avatar Ella Ge Committed by Chromium LUCI CQ

Remove ChromeActivity usage in CustomTabIncognitoManager

Bug: 1155353
Change-Id: Icfc48b9f991dbccf8c2fbc96a6bdf3ceab0c62ca
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2630225Reviewed-by: default avatarRohit Agarwal <roagarwal@chromium.org>
Reviewed-by: default avatarPeter Conn <peconn@chromium.org>
Reviewed-by: default avatarMichael Thiessen <mthiesse@chromium.org>
Commit-Queue: Ella Ge <eirage@chromium.org>
Cr-Commit-Position: refs/heads/master@{#844535}
parent 6a288cd2
...@@ -115,9 +115,6 @@ specific_include_rules = { ...@@ -115,9 +115,6 @@ specific_include_rules = {
"CustomTabDelegateFactory\.java": [ "CustomTabDelegateFactory\.java": [
"+chrome/android/java/src/org/chromium/chrome/browser/app/ChromeActivity.java", "+chrome/android/java/src/org/chromium/chrome/browser/app/ChromeActivity.java",
], ],
"CustomTabIncognitoManager\.java": [
"+chrome/android/java/src/org/chromium/chrome/browser/app/ChromeActivity.java",
],
"CustomTabTabPersistencePolicy\.java": [ "CustomTabTabPersistencePolicy\.java": [
"+chrome/android/java/src/org/chromium/chrome/browser/app/ChromeActivity.java", "+chrome/android/java/src/org/chromium/chrome/browser/app/ChromeActivity.java",
], ],
......
...@@ -4,6 +4,8 @@ ...@@ -4,6 +4,8 @@
package org.chromium.chrome.browser.customtabs; package org.chromium.chrome.browser.customtabs;
import android.annotation.SuppressLint;
import android.app.Activity;
import android.view.WindowManager; import android.view.WindowManager;
import androidx.annotation.Nullable; import androidx.annotation.Nullable;
...@@ -13,7 +15,6 @@ import org.chromium.base.CommandLine; ...@@ -13,7 +15,6 @@ import org.chromium.base.CommandLine;
import org.chromium.base.UnownedUserData; import org.chromium.base.UnownedUserData;
import org.chromium.base.UnownedUserDataKey; import org.chromium.base.UnownedUserDataKey;
import org.chromium.base.annotations.CheckDiscard; import org.chromium.base.annotations.CheckDiscard;
import org.chromium.chrome.browser.app.ChromeActivity;
import org.chromium.chrome.browser.browserservices.BrowserServicesIntentDataProvider; import org.chromium.chrome.browser.browserservices.BrowserServicesIntentDataProvider;
import org.chromium.chrome.browser.customtabs.content.CustomTabActivityNavigationController; import org.chromium.chrome.browser.customtabs.content.CustomTabActivityNavigationController;
import org.chromium.chrome.browser.customtabs.content.CustomTabActivityTabProvider; import org.chromium.chrome.browser.customtabs.content.CustomTabActivityTabProvider;
...@@ -36,6 +37,7 @@ import javax.inject.Inject; ...@@ -36,6 +37,7 @@ import javax.inject.Inject;
*/ */
@ActivityScope @ActivityScope
public class CustomTabIncognitoManager implements NativeInitObserver, Destroyable, UnownedUserData { public class CustomTabIncognitoManager implements NativeInitObserver, Destroyable, UnownedUserData {
@SuppressLint("StaticFieldLeak") // This is for test only.
private static CustomTabIncognitoManager sCustomTabIncognitoManagerUsedForTesting; private static CustomTabIncognitoManager sCustomTabIncognitoManagerUsedForTesting;
private static final String TAG = "CctIncognito"; private static final String TAG = "CctIncognito";
...@@ -43,19 +45,19 @@ public class CustomTabIncognitoManager implements NativeInitObserver, Destroyabl ...@@ -43,19 +45,19 @@ public class CustomTabIncognitoManager implements NativeInitObserver, Destroyabl
private static final UnownedUserDataKey<CustomTabIncognitoManager> KEY = private static final UnownedUserDataKey<CustomTabIncognitoManager> KEY =
new UnownedUserDataKey<>(CustomTabIncognitoManager.class); new UnownedUserDataKey<>(CustomTabIncognitoManager.class);
private final ChromeActivity<?> mChromeActivity; private final Activity mActivity;
private final BrowserServicesIntentDataProvider mIntentDataProvider; private final BrowserServicesIntentDataProvider mIntentDataProvider;
private final WindowAndroid mWindowAndroid; private final WindowAndroid mWindowAndroid;
private OTRProfileID mOTRProfileID; private OTRProfileID mOTRProfileID;
@Inject @Inject
public CustomTabIncognitoManager(ChromeActivity<?> customTabActivity, public CustomTabIncognitoManager(Activity activity, WindowAndroid windowAndroid,
WindowAndroid windowAndroid, BrowserServicesIntentDataProvider intentDataProvider, BrowserServicesIntentDataProvider intentDataProvider,
CustomTabActivityNavigationController navigationController, CustomTabActivityNavigationController navigationController,
CustomTabActivityTabProvider tabProvider, CustomTabActivityTabProvider tabProvider,
ActivityLifecycleDispatcher lifecycleDispatcher) { ActivityLifecycleDispatcher lifecycleDispatcher) {
mChromeActivity = customTabActivity; mActivity = activity;
mWindowAndroid = windowAndroid; mWindowAndroid = windowAndroid;
mIntentDataProvider = intentDataProvider; mIntentDataProvider = intentDataProvider;
...@@ -135,7 +137,7 @@ public class CustomTabIncognitoManager implements NativeInitObserver, Destroyabl ...@@ -135,7 +137,7 @@ public class CustomTabIncognitoManager implements NativeInitObserver, Destroyabl
if (!CommandLine.getInstance().hasSwitch( if (!CommandLine.getInstance().hasSwitch(
ChromeSwitches.ENABLE_INCOGNITO_SNAPSHOTS_IN_ANDROID_RECENTS)) { ChromeSwitches.ENABLE_INCOGNITO_SNAPSHOTS_IN_ANDROID_RECENTS)) {
// Disable taking screenshots and seeing snapshots in recents. // Disable taking screenshots and seeing snapshots in recents.
mChromeActivity.getWindow().addFlags(WindowManager.LayoutParams.FLAG_SECURE); mActivity.getWindow().addFlags(WindowManager.LayoutParams.FLAG_SECURE);
} }
} }
} }
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment