Commit 011e4ed9 authored by Michael Thiessen's avatar Michael Thiessen Committed by Commit Bot

Reland "VR: Remove VR Launch Intents flag"

This reverts commit 848f0ea0.

Reason for revert: Reland after reverting enums.xml change

Original change's description:
> Revert "VR: Remove VR Launch Intents flag"
> 
> This reverts commit c83231b7.
> 
> Reason for revert: Should not have removed flags from enums.xml
> 
> Original change's description:
> > VR: Remove VR Launch Intents flag
> > 
> > This flag was really only for development while we weren't sure how to
> > handle VR Intents. With Chrome VR browsing already shipped, there's no
> > reason not to allow Intents to launch it.
> > 
> > Change-Id: Ibbc7e5ea41668c073a4a04340ec45bfe40025b0f
> > Reviewed-on: https://chromium-review.googlesource.com/959051
> > Commit-Queue: Michael Thiessen <mthiesse@chromium.org>
> > Reviewed-by: Yaron Friedman <yfriedman@chromium.org>
> > Cr-Commit-Position: refs/heads/master@{#542508}
> 
> TBR=yfriedman@chromium.org,mthiesse@chromium.org
> 
> Change-Id: I6b92369c483f7f0d8a11290bb9a50f6cfd1ea766
> No-Presubmit: true
> No-Tree-Checks: true
> No-Try: true
> Reviewed-on: https://chromium-review.googlesource.com/958021
> Reviewed-by: Michael Thiessen <mthiesse@chromium.org>
> Commit-Queue: Michael Thiessen <mthiesse@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#542550}

TBR=yfriedman@chromium.org,mthiesse@chromium.org

Change-Id: Iadb349fe60ea6a3a843228f3a4dcf24c6b9f9589
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Reviewed-on: https://chromium-review.googlesource.com/958103
Commit-Queue: Michael Thiessen <mthiesse@chromium.org>
Reviewed-by: default avatarMichael Thiessen <mthiesse@chromium.org>
Cr-Commit-Position: refs/heads/master@{#542572}
parent dddc338d
......@@ -258,7 +258,6 @@ public abstract class ChromeFeatureList {
public static final String VR_BROWSING_IN_CUSTOM_TAB = "VrBrowsingInCustomTab";
public static final String VR_BROWSING_NATIVE_ANDROID_UI = "VrBrowsingNativeAndroidUi";
public static final String VR_ICON_IN_DAYDREAM_HOME = "VrIconInDaydreamHome";
public static final String VR_LAUNCH_INTENT = "VrLaunchIntents";
public static final String WEB_PAYMENTS = "WebPayments";
public static final String WEB_PAYMENTS_METHOD_SECTION_ORDER_V2 =
"WebPaymentsMethodSectionOrderV2";
......
......@@ -1070,10 +1070,7 @@ public class VrShellDelegate
return;
}
instance.onAutopresentIntent();
} else if (isVrShellEnabled(instance.mVrSupportLevel)
&& (ChromeFeatureList.isEnabled(ChromeFeatureList.VR_LAUNCH_INTENT)
|| ChromeFeatureList.isEnabled(
ChromeFeatureList.VR_ICON_IN_DAYDREAM_HOME))) {
} else if (isVrShellEnabled(instance.mVrSupportLevel)) {
if (DEBUG_LOGS) Log.i(TAG, "onNewIntentWithNative: vr");
instance.onVrIntent();
} else {
......
......@@ -148,7 +148,6 @@ public class VrShellTransitionTest {
@Test
@Restriction(RESTRICTION_TYPE_DEVICE_DAYDREAM)
@MediumTest
@CommandLineFlags.Add("enable-features=VrLaunchIntents")
public void testVrIntentStartsVrShell() {
// Send a VR intent, which will open the link in a CTA.
String url = VrTestFramework.getHtmlTestFile("test_navigation_2d_page");
......
......@@ -2250,9 +2250,6 @@ const FeatureEntry kFeatureEntries[] = {
{"vr-icon-in-daydream-home", flag_descriptions::kVrIconInDaydreamHomeName,
flag_descriptions::kVrIconInDaydreamHomeDescription, kOsAndroid,
FEATURE_VALUE_TYPE(chrome::android::kVrIconInDaydreamHome)},
{"vr-launch-intents", flag_descriptions::kVrLaunchIntentsName,
flag_descriptions::kVrLaunchIntentsDescription, kOsAndroid,
FEATURE_VALUE_TYPE(chrome::android::kVrLaunchIntents)},
{"webvr-autopresent-from-intent",
flag_descriptions::kWebVrAutopresentFromIntentName,
flag_descriptions::kWebVrAutopresentFromIntentDescription, kOsAndroid,
......
......@@ -128,7 +128,6 @@ const base::Feature* kFeaturesExposedToJava[] = {
&kVrBrowsingInCustomTab,
&kVrBrowsingNativeAndroidUi,
&kVrIconInDaydreamHome,
&kVrLaunchIntents,
&payments::features::kWebPaymentsMethodSectionOrderV2,
&payments::features::kWebPaymentsModifiers,
&kWebPaymentsSingleAppUiSkip,
......@@ -388,9 +387,6 @@ const base::Feature kVrBrowsingNativeAndroidUi{
const base::Feature kVrIconInDaydreamHome{"VrIconInDaydreamHome",
base::FEATURE_ENABLED_BY_DEFAULT};
const base::Feature kVrLaunchIntents{"VrLaunchIntents",
base::FEATURE_DISABLED_BY_DEFAULT};
const base::Feature kWebPaymentsSingleAppUiSkip{
"WebPaymentsSingleAppUiSkip", base::FEATURE_ENABLED_BY_DEFAULT};
......
......@@ -85,7 +85,6 @@ extern const base::Feature kVrBrowsingFeedback;
extern const base::Feature kVrBrowsingInCustomTab;
extern const base::Feature kVrBrowsingNativeAndroidUi;
extern const base::Feature kVrIconInDaydreamHome;
extern const base::Feature kVrLaunchIntents;
extern const base::Feature kWebPaymentsSingleAppUiSkip;
extern const base::Feature kWebVrAutopresentFromIntent;
extern const base::Feature kWebVrCardboardSupport;
......
......@@ -2916,10 +2916,6 @@ const char kVrIconInDaydreamHomeName[] = "Chrome icon in Daydream Home";
const char kVrIconInDaydreamHomeDescription[] =
"Adds an icon to Daydream Home that allows launching Chrome in VR.";
const char kVrLaunchIntentsName[] = "VR intents";
const char kVrLaunchIntentsDescription[] =
"Allow intents to launch Chrome in VR mode.";
const char kWebVrAutopresentFromIntentName[] =
"WebVR auto presentation from intents";
const char kWebVrAutopresentFromIntentDescription[] =
......
......@@ -1791,9 +1791,6 @@ extern const char kVrBrowsingInCustomTabDescription[];
extern const char kVrIconInDaydreamHomeName[];
extern const char kVrIconInDaydreamHomeDescription[];
extern const char kVrLaunchIntentsName[];
extern const char kVrLaunchIntentsDescription[];
extern const char kWebVrAutopresentFromIntentName[];
extern const char kWebVrAutopresentFromIntentDescription[];
......
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