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

Issue 142593004: Revert of IDL compiler: sync Python to r165262 (Closed)

Created:
6 years, 11 months ago by Nils Barth (inactive)
Modified:
6 years, 11 months ago
Reviewers:
haraken
CC:
blink-reviews, Nils Barth (inactive), kojih, arv+blink, jsbell+bindings_chromium.org, sof, abarth-chromium, marja+watch_chromium.org, adamk+blink_chromium.org, Nate Chapin, Inactive, kouhei (in TOK)
Visibility:
Public.

Description

Revert of IDL compiler: sync Python to r165262 (https://codereview.chromium.org/138033017/) Reason for revert: Underlying Perl change reverted; sync Python Perl revert in r165372 Revert of Add use counters for using Attrs as Nodes. https://codereview.chromium.org/130703003/ Original issue's description: > IDL compiler: sync Python to r165262 > > Python side to: > Add use counters for using Attrs as Nodes. > https://codereview.chromium.org/127723002 > > There are other cases (for interface Node itself), which I'll handle > when finalizing Node.idl. > > BUG=239771 > TBR=haraken > NOTRY=true > > Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=165266 TBR=haraken@chromium.org NOTREECHECKS=true NOTRY=true BUG=239771 Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=165373

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+0 lines, -14 lines) Patch
M Source/bindings/scripts/unstable/v8_attributes.py View 1 chunk +0 lines, -4 lines 0 comments Download
M Source/bindings/scripts/unstable/v8_methods.py View 1 chunk +0 lines, -2 lines 0 comments Download
M Source/bindings/templates/attributes.cpp View 1 chunk +0 lines, -4 lines 0 comments Download
M Source/bindings/templates/methods.cpp View 1 chunk +0 lines, -4 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
Nils Barth (inactive)
Created Revert of IDL compiler: sync Python to r165262
6 years, 11 months ago (2014-01-20 01:25:06 UTC) #1
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/nbarth@chromium.org/142593004/1
6 years, 11 months ago (2014-01-20 01:25:21 UTC) #2
commit-bot: I haz the power
Change committed as 165373
6 years, 11 months ago (2014-01-20 01:25:35 UTC) #3
haraken
6 years, 11 months ago (2014-01-20 01:33:57 UTC) #4
Message was sent while issue was closed.
LGTM

Powered by Google App Engine
This is Rietveld 408576698