Commit 069a4050 authored by Peter Kotwicz's avatar Peter Kotwicz Committed by Commit Bot

[Android Webapp Refactor] Remove redundant native check in onNewIntent()

This CL removes the redundant native check in
WebappActivity#onNewIntent(). The native check is redundant because
CustomTabIntentHandler#onNewIntent() delays the handling till the tab is
added to the tab model - WebappActivityTabController#initializeState().

BUG=1069311

Change-Id: I0e00c6a797469af87701ad0d4ca86932fc422a19
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2143114Reviewed-by: default avatarPeter Conn <peconn@chromium.org>
Commit-Queue: Peter Kotwicz <pkotwicz@chromium.org>
Cr-Commit-Position: refs/heads/master@{#758951}
parent 55ecd206
......@@ -76,7 +76,6 @@ public class WebappActivity extends BaseCustomTabActivity<WebappActivityComponen
private TabObserverRegistrar mTabObserverRegistrar;
private CustomTabDelegateFactory mDelegateFactory;
private boolean mIsInitialized;
private Integer mBrandColor;
private static Integer sOverrideCoreCountForTesting;
......@@ -146,7 +145,7 @@ public class WebappActivity extends BaseCustomTabActivity<WebappActivityComponen
if (newWebappInfo == null) {
Log.e(TAG, "Failed to parse new Intent: " + intent);
ApiCompatibilityUtils.finishAndRemoveTask(this);
} else if (newWebappInfo.shouldForceNavigation() && mIsInitialized) {
} else if (newWebappInfo.shouldForceNavigation()) {
mCustomTabIntentHandler.onNewIntent(newWebappInfo.getProvider());
}
}
......@@ -335,7 +334,6 @@ public class WebappActivity extends BaseCustomTabActivity<WebappActivityComponen
getFullscreenManager().setTab(getActivityTab());
super.finishNativeInitialization();
mIsInitialized = true;
}
@Override
......
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