Commit c75c394a authored by David Jacobo's avatar David Jacobo Committed by Commit Bot

Fix comments on ArcNavigationThrottle

Documentation must reflect the fact that Chrome is not presented as an
option to the user, also fixing typo.

Bug: 678141
Test: Build.
Change-Id: I467fa39f89a146b1779d8958c4ad526e4d582c5e
Reviewed-on: https://chromium-review.googlesource.com/747132Reviewed-by: default avatarYusuke Sato <yusukes@chromium.org>
Commit-Queue: David Jacobo <djacobo@chromium.org>
Cr-Commit-Position: refs/heads/master@{#512975}
parent 92c55639
......@@ -214,8 +214,9 @@ GURL ArcNavigationThrottle::GetStartingGURL() const {
return navigation_handle()->GetStartingSiteInstance()->GetSiteURL();
}
// We received the array of app candidates to handle this URL (even the Chrome
// app is included).
// Receives the array of app candidates to handle this URL and decides whether a
// preferred app should be triggered right away or ask the browser to display
// the intent picker.
void ArcNavigationThrottle::OnAppCandidatesReceived(
std::vector<mojom::IntentHandlerInfoPtr> handlers) {
DCHECK_CURRENTLY_ON(content::BrowserThread::UI);
......@@ -223,7 +224,7 @@ void ArcNavigationThrottle::OnAppCandidatesReceived(
const GURL& url = handle->GetURL();
if (!IsAppAvailable(handlers)) {
// This scenario shouldn't be accessed as ArcNavigatinoThrottle is created
// This scenario shouldn't be accessed as ArcNavigationThrottle is created
// iff there are ARC apps which can actually handle the given URL.
DVLOG(1) << "There are no app candidates for this URL: " << url;
ui_displayed_ = false;
......
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