Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(138)

Issue 2350933005: Android: In incognito, links never open in external apps if possible. (Closed)

Created:
4 years, 3 months ago by Matt Giuca
Modified:
4 years, 3 months ago
Reviewers:
Maria
CC:
chromium-reviews, chrome-apps-syd-reviews_chromium.org, Ted C
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Android: In incognito, links never open in external apps if possible. When clicking a link that can either be handled by the browser, or by an external app, in incognito mode we now automatically choose the browser, to keep the user within incognito mode as much as possible. Links that can only be handled externally (e.g., intent:// and other system schemes) are unchanged (prompt the user, then send an external intent). Added a test that an intent:// URL in incognito mode will still prompt. BUG=587306 Committed: https://crrev.com/7189731c3d87496bbb62aba9e125ce341296c66b Cr-Commit-Position: refs/heads/master@{#419984}

Patch Set 1 #

Patch Set 2 : Fix tests, and add a new one. #

Patch Set 3 : Added a comment. #

Patch Set 4 : Rebase. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+29 lines, -2 lines) Patch
M chrome/android/java/src/org/chromium/chrome/browser/externalnav/ExternalNavigationHandler.java View 1 2 3 1 chunk +6 lines, -0 lines 0 comments Download
M chrome/android/javatests/src/org/chromium/chrome/browser/externalnav/ExternalNavigationHandlerTest.java View 1 2 3 5 chunks +23 lines, -2 lines 0 comments Download

Messages

Total messages: 18 (12 generated)
Matt Giuca
Hi Maria, See discussion on https://crbug.com/587306, particularly #13, #22 and #27. This is Darin's proposal, ...
4 years, 3 months ago (2016-09-21 00:54:58 UTC) #7
Maria
lgtm
4 years, 3 months ago (2016-09-21 04:18:19 UTC) #12
Maria
4 years, 3 months ago (2016-09-21 04:18:55 UTC) #13
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2350933005/60001
4 years, 3 months ago (2016-09-21 05:44:19 UTC) #15
commit-bot: I haz the power
Committed patchset #4 (id:60001)
4 years, 3 months ago (2016-09-21 05:49:49 UTC) #16
commit-bot: I haz the power
4 years, 3 months ago (2016-09-21 05:52:08 UTC) #18
Message was sent while issue was closed.
Patchset 4 (id:??) landed as
https://crrev.com/7189731c3d87496bbb62aba9e125ce341296c66b
Cr-Commit-Position: refs/heads/master@{#419984}

Powered by Google App Engine
This is Rietveld 408576698