New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 604644 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Email to this user bounced
Closed: Apr 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 2
Type: Bug



Sign in to add a comment

Add HTMLHyperlinkElementUtils interface

Reported by sigbjo...@opera.com, Apr 19 2016

Issue description

The URLUtils interface is no longer being used in specs, the HTML spec having moved to defining HTMLHyperlinkElementUtils instead,

 https://html.spec.whatwg.org/#htmlhyperlinkelementutils

to cover the URL-related attributes that <a/> and <area/> must support.

Similarly, URL is now defined to expose the same set of attributes (+searchParams) directly rather than by way of a [NoInterfaceObject] URLUtils interface,

 https://url.spec.whatwg.org/#url

Sync our IDLs and implementation to track how things are organized in the specs.
 

Comment 1 by sigbjo...@opera.com, Apr 19 2016

Components: Blink>HTML Blink>Network
Project Member

Comment 2 by bugdroid1@chromium.org, Apr 19 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/40ebec288b976477aaa318b26899dc08f5b15426

commit 40ebec288b976477aaa318b26899dc08f5b15426
Author: sigbjornf <sigbjornf@opera.com>
Date: Tue Apr 19 15:20:22 2016

Rename URLUtils interface as HTMLHyperlinkElementUtils and update.

Follow the HTML spec for HTMLAnchorElement + HTMLAreaElement, and have
them implement the [NoInterfaceObject] HTMLHyperlinkElementUtils
interface, rather than the previous URLUtils which has now been
retired / divided up.

The other implementation of URLUtils, URL, now define the attributes
directly (with the addition of a searchParams getter.)  Update its IDL
also.

R=
BUG= 604644 

Review URL: https://codereview.chromium.org/1902683003

Cr-Commit-Position: refs/heads/master@{#388209}

[modify] https://crrev.com/40ebec288b976477aaa318b26899dc08f5b15426/third_party/WebKit/Source/core/core.gypi
[modify] https://crrev.com/40ebec288b976477aaa318b26899dc08f5b15426/third_party/WebKit/Source/core/dom/URL.idl
[delete] https://crrev.com/b89e8ac7021fcee56c7cc34fd077eac253c99d0b/third_party/WebKit/Source/core/dom/URLUtils.idl
[modify] https://crrev.com/40ebec288b976477aaa318b26899dc08f5b15426/third_party/WebKit/Source/core/html/HTMLAnchorElement.idl
[modify] https://crrev.com/40ebec288b976477aaa318b26899dc08f5b15426/third_party/WebKit/Source/core/html/HTMLAreaElement.idl
[add] https://crrev.com/40ebec288b976477aaa318b26899dc08f5b15426/third_party/WebKit/Source/core/html/HTMLHyperlinkElementUtils.idl

Comment 3 by sigbjo...@opera.com, Apr 19 2016

Status: Fixed (was: Assigned)

Sign in to add a comment