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

Issue 1651193003: Revert of Change Event.deepPath to a method (Closed)

Created:
4 years, 10 months ago by kozy
Modified:
4 years, 10 months ago
Reviewers:
yuzuchan, hayato, kojii, samli, kochi
CC:
blink-reviews, chromium-reviews, falken, horo+watch_chromium.org, jsbell+serviceworker_chromium.org, kenjibaheux+watch_chromium.org, kinuko+serviceworker, michaeln, nhiroki, serviceworker-reviews, tzik
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Revert of Change Event.deepPath to a method (patchset #3 id:40001 of https://codereview.chromium.org/1637813002/ ) Reason for revert: This breaks new DevTools frontend while remote debugging old backends. Original issue's description: > Change Event.deepPath to a method > > In order to make clear that computation is performed, Event.deepPath has been changed to a method Event.deepPath(), that returns sequence<EventTarget>. > See the discussion here: https://github.com/w3c/webcomponents/issues/361 > Described in the spec here : http://w3c.github.io/webcomponents/spec/shadow/#widl-Event-deepPath-sequence-EventTarget > See the spec change here: https://github.com/w3c/webcomponents/commit/ed93413f9a37a6bfbd774cb2e09e14a76c87d56c > BUG=531990 > > Committed: https://crrev.com/b9279be7d61887c171d5c1cefa114292c84b9957 > Cr-Commit-Position: refs/heads/master@{#372596} TBR=samli@chromium.org,hayato@chromium.org,kochi@chromium.org,kojii@chromium.org,yuzus@chromium.org # Skipping CQ checks because original CL landed less than 1 days ago. NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=531990 Committed: https://crrev.com/d60c3cd683e0f6fd1585ed0e6ce68184980a6823 Cr-Commit-Position: refs/heads/master@{#372713}

Patch Set 1 #

Messages

Total messages: 5 (1 generated)
kozy
Created Revert of Change Event.deepPath to a method
4 years, 10 months ago (2016-02-01 19:06:18 UTC) #1
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1651193003/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1651193003/1
4 years, 10 months ago (2016-02-01 19:08:47 UTC) #2
commit-bot: I haz the power
Committed patchset #1 (id:1)
4 years, 10 months ago (2016-02-01 19:11:40 UTC) #3
commit-bot: I haz the power
4 years, 10 months ago (2016-02-01 19:13:59 UTC) #5
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/d60c3cd683e0f6fd1585ed0e6ce68184980a6823
Cr-Commit-Position: refs/heads/master@{#372713}

Powered by Google App Engine
This is Rietveld 408576698