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

Issue 2066813003: Revert of Implement script-side of callback reactions for Custom Elements V1 (Closed)

Created:
4 years, 6 months ago by picksi1
Modified:
4 years, 6 months ago
CC:
blink-reviews, blink-reviews-bindings_chromium.org, chromium-reviews
Base URL:
https://chromium.googlesource.com/chromium/src.git@callback-ce
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Revert of Implement script-side of callback reactions for Custom Elements V1 (patchset #6 id:100001 of https://codereview.chromium.org/2060753002/ ) Reason for revert: Seems to have broken the Linux builder: https://build.chromium.org/p/chromium.perf/builders/Linux%20Builder/builds/44040 Original issue's description: > Implement script-side of callback reactions for Custom Elements V1 > > This patch implements script-side of callback reactions. Abstract > classes were implemented in a separate CL[1]. > > Also contains following refactoring: > 1. Callbacks are stored as v8::Function to make calling them easier. > 2. m_observedAttribute is moved to the super class > (CustomElementDefinition) to allow iteratiion. > 3. The order to Get("observedAttributes") was changed as per the spec > change[2]. > > The wpt test failure is a test issue, PR[3] is sent. > > [1] https://codereview.chromium.org/2058823002 > [2] https://github.com/whatwg/html/issues/1373 > [3] https://github.com/w3c/web-platform-tests/pull/3175 > > BUG=594918 > > Committed: https://crrev.com/4f581ae17af347201ede92b01dc47068ee0cf2d5 > Cr-Commit-Position: refs/heads/master@{#399859} TBR=dominicc@chromium.org,haraken@chromium.org,foolip@chromium.org,jl@opera.com,kojii@chromium.org # Skipping CQ checks because original CL landed less than 1 days ago. NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=594918

Patch Set 1 #

Messages

Total messages: 10 (2 generated)
picksi1
Created Revert of Implement script-side of callback reactions for Custom Elements V1
4 years, 6 months ago (2016-06-15 09:21:13 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/2066813003/1
4 years, 6 months ago (2016-06-15 09:21:26 UTC) #3
commit-bot: I haz the power
No L-G-T-M from a valid reviewer yet. CQ run can only be started by full ...
4 years, 6 months ago (2016-06-15 09:21:28 UTC) #5
haraken
Will https://codereview.chromium.org/2066793003/ fix the build issue?
4 years, 6 months ago (2016-06-15 09:22:03 UTC) #6
picksi1
On 2016/06/15 09:22:03, haraken wrote: > Will https://codereview.chromium.org/2066793003/ fix the build i It's currently being ...
4 years, 6 months ago (2016-06-15 09:25:30 UTC) #7
haraken
Stamping LGTM so that you can land this when you want.
4 years, 6 months ago (2016-06-15 09:32:36 UTC) #8
kojii
Looks like compile step has passed.
4 years, 6 months ago (2016-06-15 09:50:42 UTC) #9
picksi1
4 years, 6 months ago (2016-06-15 09:52:07 UTC) #10
Build has turned green! Thanks.

Powered by Google App Engine
This is Rietveld 408576698