rewrite spec links with http: to https: and fix known Javascript redirected links

rewrite spec links with http: to https: and fix known Javascript redirected links

BUG=460722
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel

Committed: https://crrev.com/089deec15a31886c0fd377bb7b7b294bbe90c8a4
Cr-Commit-Position: refs/heads/master@{#435941}

[Mark Dittmer, 2016-12-01 19:53:03]

Description was changed from

==========
rewrite spec links with http: to https: and fix known Javascript redirected
links

BUG=460722
==========

to

==========
rewrite spec links with http: to https: and fix known Javascript redirected
links

BUG=460722
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
==========

[Mark Dittmer, 2016-12-01 19:56:25]

markdittmer@chromium.org changed reviewers:
+ foolip@chromium.org, mfoltz@chromium.org, yukishiino@chromium.org

[Mark Dittmer, 2016-12-01 19:56:27]

mfoltz@chromium.org: Please review changes you own

yukishiino@chromium.org: Please review changes you own

foolip@chromium.org: Please review changes you own

[Marijn Kruisselbrink, 2016-12-01 19:58:32]

https://codereview.chromium.org/2549633002/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/html/HTMLLinkElement.idl (right):

https://codereview.chromium.org/2549633002/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/html/HTMLLinkElement.idl:57: //
https://mkruisselbrink.github.io/ServiceWorker/spec/service_worker/#link-elem...
If you're editing this anyway, might as well update this to
https://w3c.github.io/ServiceWorker/#link-element-interface-extensions

[Mark Dittmer, 2016-12-01 20:00:14]

markdittmer@chromium.org changed reviewers:
+ mlamouri@chromium.org

[Mark Dittmer, 2016-12-01 20:00:15]

mlamouri@chromium.org: Please review changes in budget, cachestorage, device_*,
encoding, encryptedmedia, fetch, payments, serviceworker, vr, web*

[Mark Dittmer, 2016-12-01 20:00:37]

The CQ bit was checked by markdittmer@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-12-01 20:00:59]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Mark Dittmer, 2016-12-01 20:25:12]

Fix up reference to cwilso web-midi-api

[Mark Dittmer, 2016-12-01 20:30:39]

Fix frame-timing link

[Yuki, 2016-12-02 06:34:10]

lgtm

[alancutter (OOO until 2018), 2016-12-02 07:17:13]

alancutter@chromium.org changed reviewers:
+ alancutter@chromium.org

[alancutter (OOO until 2018), 2016-12-02 07:17:14]

core/animation lgtm.

[haraken, 2016-12-02 07:24:38]

LGTM

[foolip, 2016-12-02 09:55:30]

everything lgtm % nits. If some of them are part of a pattern, feel free to
handle those in separate CLs.

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/core/css/CSSNamespaceRule.idl (right):

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/core/css/CSSNamespaceRule.idl:5: //
https://dev.w3.org/csswg/cssom-1/#the-cssnamespacerule-interface
https://drafts.csswg.org/cssom/#the-cssnamespacerule-interface

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/core/css/StyleMedia.idl (right):

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/core/css/StyleMedia.idl:30: //
https://web.archive.org/web/20100206142043/https://dev.w3.org/csswg/cssom-vie...
Nit: keep the second http, it probably wasn't available on https then and
following some links will bring you back to the http variant.

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/core/dom/Document.idl (right):

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/core/dom/Document.idl:158: //
https://rawgit.com/w3c/touch-events/v1-errata/touchevents.html#extensions-to-...
https://w3c.github.io/touch-events/#extensions-to-the-document-interface

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/core/dom/Element.idl (right):

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/core/dom/Element.idl:38: // PointerEvent
(https://www.w3.org/TR/pointerevents/#extensions-to-the-element-interface)
Can you reformat this to have the link on its own line as the other instances?
(Otherwise your URL parser has to skip a trailing ) I presume.)

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/core/editing/Selection.idl (right):

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/core/editing/Selection.idl:30: //
https://w3c.github.io/selection-api/#idl-def-Selection
https://w3c.github.io/selection-api/#selection-interface

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/core/events/InputEvent.idl (right):

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/core/events/InputEvent.idl:4: 
https://w3c.github.io/uievents/#interface-inputevent at the top and the

// Input Events
// https://w3c.github.io/input-events/#interface-InputEvent

where the extensions start. I also filed
https://github.com/w3c/input-events/issues/46

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/core/events/InputEventInit.idl (right):

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/core/events/InputEventInit.idl:5: //
https://w3c.github.io/input-events/
Same changes here. Even though https://github.com/w3c/input-events/issues/46
isn't finished, just link to the section where it'll appear as in
InputEvent.idl.

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/core/frame/Window.idl (right):

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/core/frame/Window.idl:199: //
https://www.whatwg.org/specs/web-apps/current-work/#transferable-objects
Redirects to https://html.spec.whatwg.org/#transferable-objects

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/core/html/HTMLLinkElement.idl (right):

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/core/html/HTMLLinkElement.idl:53: //
https://w3c.github.io/webappsec/specs/subresourceintegrity/#htmllinkelement-1
https://w3c.github.io/webappsec-subresource-integrity/#HTMLLinkElement here and
https://w3c.github.io/webappsec-subresource-integrity/#HTMLScriptElement in
HTMLScriptElement.idl

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/core/html/HTMLLinkElement.idl:57: //
https://mkruisselbrink.github.io/ServiceWorker/spec/service_worker/#link-elem...
Should be https://w3c.github.io/ServiceWorker/#link-element-interface-extensions

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/core/html/track/vtt/VTTCue.idl (right):

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/core/html/track/vtt/VTTCue.idl:30: //
https://dev.w3.org/html5/webvtt/#the-vttcue-interface
https://w3c.github.io/webvtt/#the-vttcue-interface

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/core/html/track/vtt/VTTRegion.idl (right):

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/core/html/track/vtt/VTTRegion.idl:26: //
https://dev.w3.org/html5/webvtt/#the-vttregion-interface
https://w3c.github.io/webvtt/#the-vttregion-interface

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/core/svg/SVGElement.idl (right):

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/core/svg/SVGElement.idl:23: //
https://www.w3.org/TR/SVG2/types.html#InterfaceSVGElement
We'll have to do something with all the /TR/SV2/ links, but not in this CL.

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/core/timing/Performance.idl (right):

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/core/timing/Performance.idl:32: //
https://wicg.github.io/hr-time/#the-performance-interface
That's a 404, https://w3c.github.io/hr-time/#the-performance-interface is good.

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/core/timing/Performance.idl:73: //
https://wicg.github.io/frame-timing/#extensions-performance-interface
Not a 404, so I guess the above was accidental search/replace :)

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/modules/budget/BudgetService.idl (right):

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/modules/budget/BudgetService.idl:6: // TODO(harkness):
Link WICG spec when it exists.
Remove the TODO here and elsewhere.

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/modules/cachestorage/Cache.idl (right):

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/modules/cachestorage/Cache.idl:5: // See
https://w3c.github.io/ServiceWorker/#cache
https://w3c.github.io/ServiceWorker/#cache-interface is better, and I'd drop the
"See"

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/modules/cachestorage/CacheQueryOptions.idl
(right):

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/modules/cachestorage/CacheQueryOptions.idl:5: //
https://w3c.github.io/ServiceWorker/#cache
Also https://w3c.github.io/ServiceWorker/#cache-interface

(I like to link to the sections, since otherwise the extended attributes above
an interface can end up just outside the viewport.)

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/modules/cachestorage/CacheStorage.idl (right):

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/modules/cachestorage/CacheStorage.idl:5: // See
https://w3c.github.io/ServiceWorker/#cache-storage
https://w3c.github.io/ServiceWorker/#cachestorage-interface

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/modules/device_light/DeviceLightEvent.idl
(right):

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/modules/device_light/DeviceLightEvent.idl:5: //
https://w3c.github.io/ambient-light/#idl-def-DeviceLightEvent
It's not there, has to be
https://www.w3.org/TR/2015/WD-ambient-light-20150903/#devicelightevent-interface
:(

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/modules/device_light/DeviceLightEventInit.idl
(right):

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/modules/device_light/DeviceLightEventInit.idl:5: //
https://w3c.github.io/ambient-light/#idl-def-DeviceLightEvent
https://www.w3.org/TR/2015/WD-ambient-light-20150903/#devicelightevent-interface

(IIRC /TR/ is not on the whitelist, and I'd suggest not adding this specifically
because it is a problem.)

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/modules/fetch/Headers.idl (right):

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/modules/fetch/Headers.idl:5: //
https://fetch.spec.whatwg.org/#headers-class
Blank line after

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/modules/nfc/NavigatorNFC.idl (right):

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/modules/nfc/NavigatorNFC.idl:5: //
https://w3c.github.io/web-nfc/
https://w3c.github.io/web-nfc/#extensions-to-the-navigator-interface

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/modules/permissions/PushPermissionDescriptor.idl
(right):

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/modules/permissions/PushPermissionDescriptor.idl:6: //
https://w3c.github.io/push-api/
https://w3c.github.io/push-api/#pushsubscriptionoptions-interface-and-dictionary
above the dictionary and a TODO to rename it.

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/modules/vibration/NavigatorVibration.idl (right):

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/modules/vibration/NavigatorVibration.idl:20: //
https://dev.w3.org/2009/dap/vibration/#idl-def-Navigator
https://w3c.github.io/vibration/#vibration-interface

(Not sure about the current state, but think that only specific specs under
dev.w3.org should be whitelisted, and the above pattern not be included.)

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/modules/vr/VRDisplay.idl (right):

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/modules/vr/VRDisplay.idl:10: //
https://w3c.github.io/webvr/#vrdisplay
https://w3c.github.io/webvr/#interface-vrdisplay

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/modules/vr/VRDisplayCapabilities.idl (right):

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/modules/vr/VRDisplayCapabilities.idl:5: //
https://w3c.github.io/webvr/#vrdisplaycapabilities
https://w3c.github.io/webvr/#interface-vrdisplaycapabilities

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/modules/vr/VREyeParameters.idl (right):

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/modules/vr/VREyeParameters.idl:5: //
https://w3c.github.io/webvr/#vreyeparameters
https://w3c.github.io/webvr/#interface-vreyeparameters

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/modules/vr/VRFieldOfView.idl (right):

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/modules/vr/VRFieldOfView.idl:5: //
https://w3c.github.io/webvr/#vrfieldofview
https://w3c.github.io/webvr/#interface-vrfieldofview

Filed https://github.com/w3c/webvr/pull/157 to make it eventually correct :)

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/modules/vr/VRLayer.idl (right):

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/modules/vr/VRLayer.idl:7: //
https://w3c.github.io/webvr/#vrlayer
https://w3c.github.io/webvr/#interface-vrlayer

Curious that the spec has an interface by this name but we have a dictionary.
Something your tooling should eventually spot :)

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/modules/vr/VRPose.idl (right):

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/modules/vr/VRPose.idl:5: //
https://w3c.github.io/webvr/#vrpose
https://w3c.github.io/webvr/#interface-vrpose

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/modules/vr/VRStageParameters.idl (right):

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/modules/vr/VRStageParameters.idl:5: //
https://w3c.github.io/webvr/#vrstageparameters
https://w3c.github.io/webvr/#interface-vrstageparameters

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/modules/webaudio/AudioNode.idl (right):

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/modules/webaudio/AudioNode.idl:26: // See
https://webaudio.github.io/web-audio-api/#idl-def-AudioNode
https://webaudio.github.io/web-audio-api/#the-audionode-interface, no "See" and
blank line after.

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/modules/webmidi/MIDIConnectionEvent.idl (right):

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/modules/webmidi/MIDIConnectionEvent.idl:31: //
https://webaudio.github.io/web-midi-api/#idl-def-MIDIConnectionEvent
https://webaudio.github.io/web-midi-api/#MIDIConnectionEvent

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/modules/webmidi/MIDIConnectionEventInit.idl
(right):

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/modules/webmidi/MIDIConnectionEventInit.idl:5: //
https://webaudio.github.io/web-midi-api/#idl-def-MIDIConnectionEventInit
https://webaudio.github.io/web-midi-api/#MIDIConnectionEventInit

Filed https://github.com/WebAudio/web-midi-api/pull/171 to fix the headings.

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/modules/webmidi/MIDIMessageEvent.idl (right):

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/modules/webmidi/MIDIMessageEvent.idl:31: //
https://webaudio.github.io/web-midi-api/#idl-def-MIDIMessageEvent
https://webaudio.github.io/web-midi-api/#MIDIMessageEvent and blank line after

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/modules/webmidi/MIDIMessageEventInit.idl (right):

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/modules/webmidi/MIDIMessageEventInit.idl:5: //
https://webaudio.github.io/web-midi-api/#idl-def-MIDIMessageEventInit
https://webaudio.github.io/web-midi-api/#MIDIMessageEventInit

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/modules/webmidi/MIDIOutput.idl (right):

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/modules/webmidi/MIDIOutput.idl:31: //
https://www.w3.org/TR/webmidi/#idl-def-MIDIOutput
Link to only https://webaudio.github.io/web-midi-api/#MIDIOutput

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/modules/webusb/NavigatorUSB.idl (right):

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/modules/webusb/NavigatorUSB.idl:5: //
https://wicg.github.io/webusb/#device-enumeration
https://wicg.github.io/webusb/#enumeration

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/modules/webusb/USB.idl (right):

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/modules/webusb/USB.idl:5: //
https://wicg.github.io/webusb/#device-enumeration
https://wicg.github.io/webusb/#enumeration

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/modules/webusb/USBDevice.idl (right):

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/modules/webusb/USBDevice.idl:5: //
https://wicg.github.io/webusb/#idl-def-usbtransferstatus
https://wicg.github.io/webusb/#transfers

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/modules/webusb/USBDeviceFilter.idl (right):

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/modules/webusb/USBDeviceFilter.idl:5: //
https://wicg.github.io/webusb/#device-enumeration
https://wicg.github.io/webusb/#enumeration

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/modules/webusb/USBDeviceRequestOptions.idl
(right):

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/modules/webusb/USBDeviceRequestOptions.idl:5: //
https://wicg.github.io/webusb/#device-enumeration
https://wicg.github.io/webusb/#enumeration

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/modules/webusb/USBInTransferResult.idl (right):

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/modules/webusb/USBInTransferResult.idl:5: //
https://wicg.github.io/webusb/#idl-def-usbintransferresult
https://wicg.github.io/webusb/#transfers

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/modules/webusb/USBOutTransferResult.idl (right):

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/modules/webusb/USBOutTransferResult.idl:5: //
https://wicg.github.io/webusb/#idl-def-usbouttransferresult
https://wicg.github.io/webusb/#transfers

[mlamouri (slow - plz ping), 2016-12-02 10:40:23]

mlamouri@chromium.org changed reviewers:
+ haraken@chromium.org

[mlamouri (slow - plz ping), 2016-12-02 10:40:25]

It looks like foolip@ had an exhaustive look so rs lgtm for whatever you need :)

[Takashi Toyoshima, 2016-12-02 13:04:37]

modules/webmidi/MIDIOptions still has a http link, and there are some idl files
that do not have a link to the spec there.
But changes in modules/webmidi LGTM.

I can make a follow up CL for Web MIDI.

Thanks!

[Mark Dittmer, 2016-12-02 14:05:06]

Address all code review comments; mostly rewrite incorrectly-anchored links

[Mark Dittmer, 2016-12-02 14:05:46]

Addressed all comments. foolip@chromium.org: Please not "RSVPs". I think they
can just be gathered and we can discuss tooling at next sync.

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/core/css/CSSNamespaceRule.idl (right):

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/core/css/CSSNamespaceRule.idl:5: //
https://dev.w3.org/csswg/cssom-1/#the-cssnamespacerule-interface
On 2016/12/02 09:55:28, foolip wrote:
> https://drafts.csswg.org/cssom/#the-cssnamespacerule-interface

Done.

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/core/css/StyleMedia.idl (right):

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/core/css/StyleMedia.idl:30: //
https://web.archive.org/web/20100206142043/https://dev.w3.org/csswg/cssom-vie...
On 2016/12/02 09:55:28, foolip wrote:
> Nit: keep the second http, it probably wasn't available on https then and
> following some links will bring you back to the http variant.

Done.

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/core/dom/Document.idl (right):

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/core/dom/Document.idl:158: //
https://rawgit.com/w3c/touch-events/v1-errata/touchevents.html#extensions-to-...
On 2016/12/02 09:55:28, foolip wrote:
> https://w3c.github.io/touch-events/#extensions-to-the-document-interface

Done.

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/core/dom/Element.idl (right):

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/core/dom/Element.idl:38: // PointerEvent
(https://www.w3.org/TR/pointerevents/#extensions-to-the-element-interface)
On 2016/12/02 09:55:28, foolip wrote:
> Can you reformat this to have the link on its own line as the other instances?
> (Otherwise your URL parser has to skip a trailing ) I presume.)

I'll do the reformat. It turns out that because I drop anchors, this isn't a
problem link for the scraper.

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/core/editing/Selection.idl (right):

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/core/editing/Selection.idl:30: //
https://w3c.github.io/selection-api/#idl-def-Selection
On 2016/12/02 09:55:28, foolip wrote:
> https://w3c.github.io/selection-api/#selection-interface

Done.

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/core/events/InputEvent.idl (right):

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/core/events/InputEvent.idl:4: 
On 2016/12/02 09:55:28, foolip wrote:
> https://w3c.github.io/uievents/#interface-inputevent at the top and the
> 
> // Input Events
> // https://w3c.github.io/input-events/#interface-InputEvent
> 
> where the extensions start. I also filed
> https://github.com/w3c/input-events/issues/46

Done.

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/core/events/InputEventInit.idl (right):

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/core/events/InputEventInit.idl:5: //
https://w3c.github.io/input-events/
On 2016/12/02 09:55:28, foolip wrote:
> Same changes here. Even though https://github.com/w3c/input-events/issues/46
> isn't finished, just link to the section where it'll appear as in
> InputEvent.idl.

Done.

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/core/frame/Window.idl (right):

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/core/frame/Window.idl:199: //
https://www.whatwg.org/specs/web-apps/current-work/#transferable-objects
On 2016/12/02 09:55:28, foolip wrote:
> Redirects to https://html.spec.whatwg.org/#transferable-objects

Done.

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/core/html/HTMLLinkElement.idl (right):

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/core/html/HTMLLinkElement.idl:53: //
https://w3c.github.io/webappsec/specs/subresourceintegrity/#htmllinkelement-1
On 2016/12/02 09:55:28, foolip wrote:
> https://w3c.github.io/webappsec-subresource-integrity/#HTMLLinkElement here
and
> https://w3c.github.io/webappsec-subresource-integrity/#HTMLScriptElement in
> HTMLScriptElement.idl

Done.

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/core/html/HTMLLinkElement.idl:57: //
https://mkruisselbrink.github.io/ServiceWorker/spec/service_worker/#link-elem...
On 2016/12/02 09:55:28, foolip wrote:
> Should be
https://w3c.github.io/ServiceWorker/#link-element-interface-extensions

Done.

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/core/html/track/vtt/VTTCue.idl (right):

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/core/html/track/vtt/VTTCue.idl:30: //
https://dev.w3.org/html5/webvtt/#the-vttcue-interface
On 2016/12/02 09:55:28, foolip wrote:
> https://w3c.github.io/webvtt/#the-vttcue-interface

Done.

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/core/html/track/vtt/VTTRegion.idl (right):

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/core/html/track/vtt/VTTRegion.idl:26: //
https://dev.w3.org/html5/webvtt/#the-vttregion-interface
On 2016/12/02 09:55:28, foolip wrote:
> https://w3c.github.io/webvtt/#the-vttregion-interface

Done.

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/core/svg/SVGElement.idl (right):

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/core/svg/SVGElement.idl:23: //
https://www.w3.org/TR/SVG2/types.html#InterfaceSVGElement
On 2016/12/02 09:55:29, foolip wrote:
> We'll have to do something with all the /TR/SV2/ links, but not in this CL.

+foolip@chromium.org: RSVP (here or via email) as in, we should be scraping them
in tooling, or the need to be rewritten to point elsewhere?

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/core/timing/Performance.idl (right):

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/core/timing/Performance.idl:32: //
https://wicg.github.io/hr-time/#the-performance-interface
On 2016/12/02 09:55:29, foolip wrote:
> That's a 404, https://w3c.github.io/hr-time/#the-performance-interface is
good.

Done.

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/core/timing/Performance.idl:73: //
https://wicg.github.io/frame-timing/#extensions-performance-interface
On 2016/12/02 09:55:29, foolip wrote:
> Not a 404, so I guess the above was accidental search/replace :)

Done.

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/modules/budget/BudgetService.idl (right):

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/modules/budget/BudgetService.idl:6: // TODO(harkness):
Link WICG spec when it exists.
On 2016/12/02 09:55:29, foolip wrote:
> Remove the TODO here and elsewhere.

Done here and elsewhere.

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/modules/cachestorage/Cache.idl (right):

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/modules/cachestorage/Cache.idl:5: // See
https://w3c.github.io/ServiceWorker/#cache
On 2016/12/02 09:55:29, foolip wrote:
> https://w3c.github.io/ServiceWorker/#cache-interface is better, and I'd drop
the
> "See"

Done.

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/modules/cachestorage/CacheQueryOptions.idl
(right):

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/modules/cachestorage/CacheQueryOptions.idl:5: //
https://w3c.github.io/ServiceWorker/#cache
On 2016/12/02 09:55:29, foolip wrote:
> Also https://w3c.github.io/ServiceWorker/#cache-interface
> 
> (I like to link to the sections, since otherwise the extended attributes above
> an interface can end up just outside the viewport.)

Done.

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/modules/cachestorage/CacheStorage.idl (right):

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/modules/cachestorage/CacheStorage.idl:5: // See
https://w3c.github.io/ServiceWorker/#cache-storage
On 2016/12/02 09:55:29, foolip wrote:
> https://w3c.github.io/ServiceWorker/#cachestorage-interface

Done.

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/modules/device_light/DeviceLightEvent.idl
(right):

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/modules/device_light/DeviceLightEvent.idl:5: //
https://w3c.github.io/ambient-light/#idl-def-DeviceLightEvent
On 2016/12/02 09:55:29, foolip wrote:
> It's not there, has to be
>
https://www.w3.org/TR/2015/WD-ambient-light-20150903/#devicelightevent-interface
> :(

foolip@chromium.org: RSVP here or via email: Do we need a whitelist exception in
the scrape tooling for this?

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/modules/device_light/DeviceLightEvent.idl:5: //
https://w3c.github.io/ambient-light/#idl-def-DeviceLightEvent
On 2016/12/02 09:55:29, foolip wrote:
> It's not there, has to be
>
https://www.w3.org/TR/2015/WD-ambient-light-20150903/#devicelightevent-interface
> :(

Done.

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/modules/device_light/DeviceLightEventInit.idl
(right):

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/modules/device_light/DeviceLightEventInit.idl:5: //
https://w3c.github.io/ambient-light/#idl-def-DeviceLightEvent
On 2016/12/02 09:55:29, foolip wrote:
>
https://www.w3.org/TR/2015/WD-ambient-light-20150903/#devicelightevent-interface
> 
> (IIRC /TR/ is not on the whitelist, and I'd suggest not adding this
specifically
> because it is a problem.)

foolip@chromium.org: Okay. We should sync up after this lands about what (if
any) whitelist changes we want to make for scrape tooling.

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/modules/device_light/DeviceLightEventInit.idl:5: //
https://w3c.github.io/ambient-light/#idl-def-DeviceLightEvent
On 2016/12/02 09:55:29, foolip wrote:
>
https://www.w3.org/TR/2015/WD-ambient-light-20150903/#devicelightevent-interface
> 
> (IIRC /TR/ is not on the whitelist, and I'd suggest not adding this
specifically
> because it is a problem.)

Done.

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/modules/fetch/Headers.idl (right):

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/modules/fetch/Headers.idl:5: //
https://fetch.spec.whatwg.org/#headers-class
On 2016/12/02 09:55:29, foolip wrote:
> Blank line after

Done.

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/modules/nfc/NavigatorNFC.idl (right):

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/modules/nfc/NavigatorNFC.idl:5: //
https://w3c.github.io/web-nfc/
On 2016/12/02 09:55:29, foolip wrote:
> https://w3c.github.io/web-nfc/#extensions-to-the-navigator-interface

Done.

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/modules/permissions/PushPermissionDescriptor.idl
(right):

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/modules/permissions/PushPermissionDescriptor.idl:6: //
https://w3c.github.io/push-api/
On 2016/12/02 09:55:29, foolip wrote:
>
https://w3c.github.io/push-api/#pushsubscriptionoptions-interface-and-dictionary
> above the dictionary and a TODO to rename it.

Done.

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/modules/vibration/NavigatorVibration.idl (right):

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/modules/vibration/NavigatorVibration.idl:20: //
https://dev.w3.org/2009/dap/vibration/#idl-def-Navigator
On 2016/12/02 09:55:29, foolip wrote:
> https://w3c.github.io/vibration/#vibration-interface
> 
> (Not sure about the current state, but think that only specific specs under
> http://dev.w3.org should be whitelisted, and the above pattern not be
included.)

foolip@chromium.org: Yeah. Let's sync up on this.

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/modules/vibration/NavigatorVibration.idl:20: //
https://dev.w3.org/2009/dap/vibration/#idl-def-Navigator
On 2016/12/02 09:55:29, foolip wrote:
> https://w3c.github.io/vibration/#vibration-interface
> 
> (Not sure about the current state, but think that only specific specs under
> http://dev.w3.org should be whitelisted, and the above pattern not be
included.)

Done.

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/modules/vr/VRDisplay.idl (right):

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/modules/vr/VRDisplay.idl:10: //
https://w3c.github.io/webvr/#vrdisplay
On 2016/12/02 09:55:29, foolip wrote:
> https://w3c.github.io/webvr/#interface-vrdisplay

Done.

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/modules/vr/VRDisplayCapabilities.idl (right):

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/modules/vr/VRDisplayCapabilities.idl:5: //
https://w3c.github.io/webvr/#vrdisplaycapabilities
On 2016/12/02 09:55:29, foolip wrote:
> https://w3c.github.io/webvr/#interface-vrdisplaycapabilities

Done.

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/modules/vr/VREyeParameters.idl (right):

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/modules/vr/VREyeParameters.idl:5: //
https://w3c.github.io/webvr/#vreyeparameters
On 2016/12/02 09:55:29, foolip wrote:
> https://w3c.github.io/webvr/#interface-vreyeparameters

Done.

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/modules/vr/VRFieldOfView.idl (right):

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/modules/vr/VRFieldOfView.idl:5: //
https://w3c.github.io/webvr/#vrfieldofview
On 2016/12/02 09:55:29, foolip wrote:
> https://w3c.github.io/webvr/#interface-vrfieldofview
> 
> Filed https://github.com/w3c/webvr/pull/157 to make it eventually correct :)

Done.

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/modules/vr/VRLayer.idl (right):

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/modules/vr/VRLayer.idl:7: //
https://w3c.github.io/webvr/#vrlayer
On 2016/12/02 09:55:29, foolip wrote:
> https://w3c.github.io/webvr/#interface-vrlayer
> 
> Curious that the spec has an interface by this name but we have a dictionary.
> Something your tooling should eventually spot :)

Done.

Indeed :)

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/modules/vr/VRPose.idl (right):

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/modules/vr/VRPose.idl:5: //
https://w3c.github.io/webvr/#vrpose
On 2016/12/02 09:55:29, foolip wrote:
> https://w3c.github.io/webvr/#interface-vrpose

Done.

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/modules/vr/VRStageParameters.idl (right):

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/modules/vr/VRStageParameters.idl:5: //
https://w3c.github.io/webvr/#vrstageparameters
On 2016/12/02 09:55:29, foolip wrote:
> https://w3c.github.io/webvr/#interface-vrstageparameters

Done.

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/modules/webaudio/AudioNode.idl (right):

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/modules/webaudio/AudioNode.idl:26: // See
https://webaudio.github.io/web-audio-api/#idl-def-AudioNode
On 2016/12/02 09:55:29, foolip wrote:
> https://webaudio.github.io/web-audio-api/#the-audionode-interface, no "See"
and
> blank line after.

Done.

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/modules/webmidi/MIDIMessageEvent.idl (right):

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/modules/webmidi/MIDIMessageEvent.idl:31: //
https://webaudio.github.io/web-midi-api/#idl-def-MIDIMessageEvent
On 2016/12/02 09:55:29, foolip wrote:
> https://webaudio.github.io/web-midi-api/#MIDIMessageEvent and blank line after

Done.

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/modules/webmidi/MIDIMessageEventInit.idl (right):

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/modules/webmidi/MIDIMessageEventInit.idl:5: //
https://webaudio.github.io/web-midi-api/#idl-def-MIDIMessageEventInit
On 2016/12/02 09:55:30, foolip wrote:
> https://webaudio.github.io/web-midi-api/#MIDIMessageEventInit

Done.

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/modules/webmidi/MIDIOutput.idl (right):

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/modules/webmidi/MIDIOutput.idl:31: //
https://www.w3.org/TR/webmidi/#idl-def-MIDIOutput
On 2016/12/02 09:55:30, foolip wrote:
> Link to only https://webaudio.github.io/web-midi-api/#MIDIOutput

Done.

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/modules/webusb/NavigatorUSB.idl (right):

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/modules/webusb/NavigatorUSB.idl:5: //
https://wicg.github.io/webusb/#device-enumeration
On 2016/12/02 09:55:30, foolip wrote:
> https://wicg.github.io/webusb/#enumeration

Done.

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/modules/webusb/USB.idl (right):

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/modules/webusb/USB.idl:5: //
https://wicg.github.io/webusb/#device-enumeration
On 2016/12/02 09:55:30, foolip wrote:
> https://wicg.github.io/webusb/#enumeration

Done.

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/modules/webusb/USBDevice.idl (right):

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/modules/webusb/USBDevice.idl:5: //
https://wicg.github.io/webusb/#idl-def-usbtransferstatus
On 2016/12/02 09:55:30, foolip wrote:
> https://wicg.github.io/webusb/#transfers

Done.

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/modules/webusb/USBDeviceRequestOptions.idl
(right):

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/modules/webusb/USBDeviceRequestOptions.idl:5: //
https://wicg.github.io/webusb/#device-enumeration
On 2016/12/02 09:55:30, foolip wrote:
> https://wicg.github.io/webusb/#enumeration

Done.

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/modules/webusb/USBInTransferResult.idl (right):

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/modules/webusb/USBInTransferResult.idl:5: //
https://wicg.github.io/webusb/#idl-def-usbintransferresult
On 2016/12/02 09:55:30, foolip wrote:
> https://wicg.github.io/webusb/#transfers

Done.

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/modules/webusb/USBOutTransferResult.idl (right):

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/modules/webusb/USBOutTransferResult.idl:5: //
https://wicg.github.io/webusb/#idl-def-usbouttransferresult
On 2016/12/02 09:55:30, foolip wrote:
> https://wicg.github.io/webusb/#transfers

Done.

[Mark Dittmer, 2016-12-02 14:07:16]

The CQ bit was checked by markdittmer@chromium.org

[Mark Dittmer, 2016-12-02 14:07:17]

The patchset sent to the CQ was uploaded after l-g-t-m from
alancutter@chromium.org, mlamouri@chromium.org, yukishiino@chromium.org,
foolip@chromium.org, toyoshim@chromium.org, haraken@chromium.org
Link to the patchset: https://codereview.chromium.org/2549633002/#ps60001
(title: "Address all code review comments; mostly rewrite incorrectly-anchored
links")

[commit-bot: I haz the power, 2016-12-02 14:07:30]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[foolip, 2016-12-02 14:45:33]

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/core/svg/SVGElement.idl (right):

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/core/svg/SVGElement.idl:23: //
https://www.w3.org/TR/SVG2/types.html#InterfaceSVGElement
On 2016/12/02 14:05:45, Mark Dittmer wrote:
> On 2016/12/02 09:55:29, foolip wrote:
> > We'll have to do something with all the /TR/SV2/ links, but not in this CL.
> 
> mailto:+foolip@chromium.org: RSVP (here or via email) as in, we should be
scraping them
> in tooling, or the need to be rewritten to point elsewhere?

This one should be https://svgwg.org/svg2-draft/types.html#InterfaceSVGElement I
think, but asking SVG folks (pdr@, fs@opera.com) would be good.

There are some /TR/SVG11/ links, but none to svgwg.org AFAICT. In any event,
linking to just the spec that's maintained is the idea, but no need to
prioritize until we see that SVG data isn't right, or if there's conflicting
data from different copies.

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/modules/device_light/DeviceLightEvent.idl
(right):

https://codereview.chromium.org/2549633002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/modules/device_light/DeviceLightEvent.idl:5: //
https://w3c.github.io/ambient-light/#idl-def-DeviceLightEvent
On 2016/12/02 14:05:45, Mark Dittmer wrote:
> On 2016/12/02 09:55:29, foolip wrote:
> > It's not there, has to be
> >
>
https://www.w3.org/TR/2015/WD-ambient-light-20150903/#devicelightevent-interface
> > :(
> 
> mailto:foolip@chromium.org: RSVP here or via email: Do we need a whitelist
exception in
> the scrape tooling for this?

Good question. It is abandoned standards-wise, so not whitelisting would but it
in the bucket of things to fix. I say let's start there and then see if things
like this start to drown out the never-spec'd stuff and adapt. I think we'll
just do a sweep of the problematic stuff and file bugs, so once we know about it
there would be some value in preventing accidental divergence from the
(obsolete) specs.

[foolip, 2016-12-02 14:46:59]

https://codereview.chromium.org/2549633002/diff/60001/third_party/WebKit/Sour...
File third_party/WebKit/Source/core/css/CSSNamespaceRule.idl (left):

https://codereview.chromium.org/2549633002/diff/60001/third_party/WebKit/Sour...
third_party/WebKit/Source/core/css/CSSNamespaceRule.idl:5: //
http://dev.w3.org/csswg/cssom-1/#the-cssnamespacerule-interface
Might be worth doing a sweep to see if there are any dev.w3.org links left after
this, and whitelisting just those to make sure we don't add new ones.

[commit-bot: I haz the power, 2016-12-02 15:41:13]

CQ is committing da patch.

Bot data: {"patchset_id": 60001, "attempt_start_ts": 1480687636780670,
"parent_rev": "fcba1e15fb0b1a884ed9c5e00a1796d35725336c", "commit_rev":
"939a5726cef1c11b7e471c4e1b06958c81c3a9b8"}

[commit-bot: I haz the power, 2016-12-02 15:41:47]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2016-12-02 15:43:20]

Description was changed from

==========
rewrite spec links with http: to https: and fix known Javascript redirected
links

BUG=460722
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
==========

to

==========
rewrite spec links with http: to https: and fix known Javascript redirected
links

BUG=460722
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel

Committed: https://crrev.com/089deec15a31886c0fd377bb7b7b294bbe90c8a4
Cr-Commit-Position: refs/heads/master@{#435941}
==========

[commit-bot: I haz the power, 2016-12-02 15:43:22]

Patchset 4 (id:??) landed as
https://crrev.com/089deec15a31886c0fd377bb7b7b294bbe90c8a4
Cr-Commit-Position: refs/heads/master@{#435941}