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

Issue 9310119: Add ImplementedBy attribute for operations merged from [Supplemental=XXX] interfaces. (Closed)

Created:
8 years, 10 months ago by podivilov
Modified:
8 years, 10 months ago
Reviewers:
sra1
CC:
reviews_dartlang.org
Visibility:
Public.

Description

Add ImplementedBy attribute for operations merged from [Supplemental=XXX] interfaces. C++ implementations need to know webcore factory name. R=sra@google.com Committed: https://code.google.com/p/dart/source/detail?r=4209

Patch Set 1 #

Total comments: 1

Patch Set 2 : Move new code to _merge_interfaces #

Unified diffs Side-by-side diffs Delta from patch set Stats (+5 lines, -0 lines) Patch
M client/dom/scripts/databasebuilder.py View 1 1 chunk +5 lines, -0 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
podivilov
8 years, 10 months ago (2012-02-06 13:41:57 UTC) #1
sra1
https://chromiumcodereview.appspot.com/9310119/diff/1/client/dom/scripts/databasebuilder.py File client/dom/scripts/databasebuilder.py (right): https://chromiumcodereview.appspot.com/9310119/diff/1/client/dom/scripts/databasebuilder.py#newcode471 client/dom/scripts/databasebuilder.py:471: if operation.id in operations: Is it possible that foo(int) ...
8 years, 10 months ago (2012-02-06 18:14:36 UTC) #2
podivilov
On 2012/02/06 18:14:36, sra1 wrote: > https://chromiumcodereview.appspot.com/9310119/diff/1/client/dom/scripts/databasebuilder.py > File client/dom/scripts/databasebuilder.py (right): > > https://chromiumcodereview.appspot.com/9310119/diff/1/client/dom/scripts/databasebuilder.py#newcode471 > ...
8 years, 10 months ago (2012-02-10 16:00:09 UTC) #3
sra1
8 years, 10 months ago (2012-02-13 19:12:50 UTC) #4
lgtm

Powered by Google App Engine
This is Rietveld 408576698