Commit 4b1e4a6e authored by Tommy Steimel's avatar Tommy Steimel Committed by Commit Bot

[Media Intent Handler] Show entire content:// URI

This CL adds content:// to the list of schemes that the UrlBar should
display completely. This fixes an issue with the media intent handler
where only "content:" was shown as the url, which looked buggy.

Bug: 872489
Change-Id: I4af4bf01df1b62738df2023ebb633b4674562b86
Reviewed-on: https://chromium-review.googlesource.com/1168383Reviewed-by: default avatarTed Choc <tedchoc@chromium.org>
Reviewed-by: default avatarMounir Lamouri <mlamouri@chromium.org>
Commit-Queue: Tommy Steimel <steimel@chromium.org>
Cr-Commit-Position: refs/heads/master@{#581927}
parent c903aa21
...@@ -25,8 +25,9 @@ public class UrlBarData { ...@@ -25,8 +25,9 @@ public class UrlBarData {
/** /**
* The URL schemes that should be displayed complete with path. * The URL schemes that should be displayed complete with path.
*/ */
public static final HashSet<String> UNSUPPORTED_SCHEMES_TO_SPLIT = CollectionUtil.newHashSet( public static final HashSet<String> UNSUPPORTED_SCHEMES_TO_SPLIT =
UrlConstants.FILE_SCHEME, UrlConstants.JAVASCRIPT_SCHEME, UrlConstants.DATA_SCHEME); CollectionUtil.newHashSet(UrlConstants.FILE_SCHEME, UrlConstants.JAVASCRIPT_SCHEME,
UrlConstants.DATA_SCHEME, UrlConstants.CONTENT_SCHEME);
/** /**
* URI schemes that ContentView can handle. * URI schemes that ContentView can handle.
* *
......
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