Commit 480d3c55 authored by Jeevan Shikaram's avatar Jeevan Shikaram Committed by Commit Bot

[Intent Picker] Navigation misinterpreted as back/forward button press.

PageTransitionGetQualifier is misinterpreting a user selecting a link
as a back/forward or typing in URL bar, disabling intent picker from
displaying when it should. We need to mask out the client redirect such
that the intent picker shows.

Bug: 1015296
Change-Id: I4081c3a6bffec350aafa05824ac56802ce89f518
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1924329Reviewed-by: default avatarDavid Jacobo <djacobo@chromium.org>
Commit-Queue: Jeevan Shikaram <jshikaram@chromium.org>
Cr-Commit-Position: refs/heads/master@{#716937}
parent aa1bd803
......@@ -494,7 +494,7 @@ AppsNavigationThrottle::HandleRequest() {
constexpr bool kAllowFormSubmit = false;
// Ignore navigations with the CLIENT_REDIRECT qualifier on.
constexpr bool kAllowClientRedirect = false;
constexpr bool kAllowClientRedirect = true;
ui::PageTransition page_transition = handle->GetPageTransition();
content::WebContents* web_contents = handle->GetWebContents();
......
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