Commit 48125e93 authored by Maggie Cai's avatar Maggie Cai Committed by Commit Bot

[IntentHandling] Fix common apps navigation throttle error.

This CL fixes an error in common apps navigation throttle. After load
app icons, we should return true to indicate we need to defer the
navigation. Otherwise it will cause DCHECK error when resume navigation.

BUG=853604

Change-Id: Ic618686070ab7eb237adf7b194de61cdceafcfd4
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2155764Reviewed-by: default avatarDominick Ng <dominickn@chromium.org>
Commit-Queue: Maggie Cai <mxcai@chromium.org>
Cr-Commit-Position: refs/heads/master@{#760404}
parent df2c15a6
......@@ -239,7 +239,7 @@ bool CommonAppsNavigationThrottle::ShouldDeferNavigation(
base::BindOnce(
&CommonAppsNavigationThrottle::OnDeferredNavigationProcessed,
weak_factory_.GetWeakPtr()));
return false;
return true;
}
void CommonAppsNavigationThrottle::OnDeferredNavigationProcessed(
......
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