Commit 79883706 authored by gogerald's avatar gogerald Committed by Commit bot

Do no reuse flag USE_FRE_FLOW_SEQUENCER to infer lightweight FRE

The flag was removed after generating the full FRE sequence to avoid regenerating the sequence if the activity restarts because of device configuration changes.

BUG=626993

Review-Url: https://codereview.chromium.org/2145393004
Cr-Commit-Position: refs/heads/master@{#405999}
parent d7cfe119
......@@ -48,6 +48,7 @@ public class FirstRunActivity extends AppCompatActivity implements FirstRunPageD
// Incoming parameters:
public static final String COMING_FROM_CHROME_ICON = "ComingFromChromeIcon";
public static final String USE_FRE_FLOW_SEQUENCER = "UseFreFlowSequencer";
public static final String START_LIGHTWEIGHT_FRE = "StartLightweightFRE";
static final String SHOW_WELCOME_PAGE = "ShowWelcome";
static final String SHOW_SIGNIN_PAGE = "ShowSignIn";
......@@ -141,9 +142,8 @@ public class FirstRunActivity extends AppCompatActivity implements FirstRunPageD
mFreProperties = new Bundle();
}
// Skip creating content view if it is a lightweight First Run Experience without using
// USE_FRE_FLOW_SEQUENCER.
if (!mFreProperties.getBoolean(FirstRunActivity.USE_FRE_FLOW_SEQUENCER)) {
// Skip creating content view if it is to start a lightweight First Run Experience.
if (mFreProperties.getBoolean(FirstRunActivity.START_LIGHTWEIGHT_FRE)) {
return;
}
......
......@@ -262,7 +262,7 @@ public abstract class FirstRunFlowSequencer {
Intent intent = new Intent();
intent.setClassName(context, LightweightFirstRunActivity.class.getName());
intent.putExtra(FirstRunActivity.COMING_FROM_CHROME_ICON, fromChromeIcon);
intent.putExtra(FirstRunActivity.USE_FRE_FLOW_SEQUENCER, false);
intent.putExtra(FirstRunActivity.START_LIGHTWEIGHT_FRE, true);
return intent;
}
......
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