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

Issue 452493004: Reapplication of revision 38996 (dart side edits to make CSS Properties (Closed)

Created:
6 years, 4 months ago by Emily Fortuna
Modified:
6 years, 4 months ago
Reviewers:
Leaf
CC:
reviews_dartlang.org, ricow1
Visibility:
Public.

Description

Reapplication of revision 38996 (dart side edits to make CSS Properties accessible with and without prefixes), with the addition of CSSPropertyNames.in which will be deleted once https://codereview.chromium.org/444453006/ is committed. BUG= Committed: https://code.google.com/p/dart/source/detail?r=38998

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+2570 lines, -2146 lines) Patch
M sdk/lib/_blink/dartium/_blink_dartium.dart View 2 chunks +3 lines, -1 line 0 comments Download
M sdk/lib/html/dart2js/html_dart2js.dart View 56 chunks +640 lines, -673 lines 0 comments Download
M sdk/lib/html/dartium/html_dartium.dart View 56 chunks +651 lines, -667 lines 0 comments Download
M tests/html/cssstyledeclaration_test.dart View 1 chunk +7 lines, -0 lines 0 comments Download
M tools/dom/dom.json View 2 chunks +6 lines, -0 lines 0 comments Download
M tools/dom/idl/dart/dart.idl View 1 chunk +5 lines, -0 lines 0 comments Download
A tools/dom/scripts/CSSPropertyNames.in View 1 chunk +438 lines, -0 lines 0 comments Download
M tools/dom/scripts/css_code_generator.py View 3 chunks +156 lines, -130 lines 0 comments Download
M tools/dom/scripts/dartdomgenerator.py View 3 chunks +8 lines, -0 lines 0 comments Download
M tools/dom/scripts/systemhtml.py View 1 chunk +1 line, -0 lines 0 comments Download
M tools/dom/templates/html/impl/impl_CSSStyleDeclaration.darttemplate View 56 chunks +655 lines, -675 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
Emily Fortuna
I'm waiting for Terry to add CSSProperties via deps. In the meanwhile, I'm checking in ...
6 years, 4 months ago (2014-08-07 17:20:45 UTC) #1
Emily Fortuna
Committed patchset #1 manually as 38998 (presubmit successful).
6 years, 4 months ago (2014-08-07 17:21:12 UTC) #2
Leaf
6 years, 4 months ago (2014-08-07 17:22:45 UTC) #3
Message was sent while issue was closed.
lgtm

Powered by Google App Engine
This is Rietveld 408576698