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

Issue 63843003: Mojo: Add support for attributes [name=value] on interfaces. (Closed)

Created:
7 years, 1 month ago by darin (slow to review)
Modified:
7 years, 1 month ago
Reviewers:
DaveMoore
CC:
chromium-reviews, Aaron Boodman, viettrungluu+watch_chromium.org, ben+mojo_chromium.org, abarth-chromium
Visibility:
Public.

Description

Mojo: Add support for attributes [name=value] on interfaces. This will be useful for defining peer relations between interfaces. R=davemoore@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=233652

Patch Set 1 #

Total comments: 2
Unified diffs Side-by-side diffs Delta from patch set Stats (+7 lines, -5 lines) Patch
M mojo/public/bindings/parser/mojo_parser.py View 2 chunks +4 lines, -3 lines 1 comment Download
M mojo/public/bindings/parser/mojo_translate.py View 2 chunks +3 lines, -2 lines 1 comment Download

Messages

Total messages: 3 (0 generated)
darin (slow to review)
https://codereview.chromium.org/63843003/diff/1/mojo/public/bindings/parser/mojo_parser.py File mojo/public/bindings/parser/mojo_parser.py (right): https://codereview.chromium.org/63843003/diff/1/mojo/public/bindings/parser/mojo_parser.py#newcode153 mojo/public/bindings/parser/mojo_parser.py:153: """attribute : NAME EQUALS NUMBER Previously, I had just ...
7 years, 1 month ago (2013-11-07 06:14:30 UTC) #1
DaveMoore
lgtm
7 years, 1 month ago (2013-11-07 19:02:41 UTC) #2
darin (slow to review)
7 years, 1 month ago (2013-11-07 19:09:11 UTC) #3
Message was sent while issue was closed.
Committed patchset #1 manually as r233652 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698