[Custom Scheme] Remove isProtocolHandlerRegistered in the NavigatorContentUtils of blink #8431
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…ntentUtils of blink isProtocolHandlerRegistered has been existed in blink. However, latest custom scheme whatwg specfication removed it. - whatwg: https://html.spec.whatwg.org/multipage/system-state.html#custom-handlers - whatwg issue: whatwg/html#630 - commit to remove it: whatwg/html@b143dbc The reason to remove it was that no browsers have supported the function. So it would be good if we remove it from blink in order to reduce our maintanance overhead. Bug: 121825 Change-Id: I3e99dd5996e55dedcd8b5d47a7103410acfd24eb Reviewed-on: https://chromium-review.googlesource.com/784653 Commit-Queue: Gyuyoung Kim <gyuyoung.kim@lge.com> Reviewed-by: Mike Lawther <mikelawther@chromium.org> Reviewed-by: Kentaro Hara <haraken@chromium.org> Cr-Commit-Position: refs/heads/master@{#518981}
wpt-pr-bot
approved these changes
Nov 24, 2017
Build PASSEDStarted: 2017-11-24 18:32:11 View more information about this build on: |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
isProtocolHandlerRegistered has been existed in blink. However, latest custom scheme whatwg
specfication removed it.
The reason to remove it was that no browsers have supported the function. So it would be good
if we remove it from blink in order to reduce our maintanance overhead.
Bug: 121825
Change-Id: I3e99dd5996e55dedcd8b5d47a7103410acfd24eb
Reviewed-on: https://chromium-review.googlesource.com/784653
Commit-Queue: Gyuyoung Kim gyuyoung.kim@lge.com
Reviewed-by: Mike Lawther mikelawther@chromium.org
Reviewed-by: Kentaro Hara haraken@chromium.org
Cr-Commit-Position: refs/heads/master@{#518981}
This change is