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

Issue 23499007: HTML types that don't have [] do not implement JavaScriptIndexingBehavior, that is, the items canno… (Closed)

Created:
7 years, 3 months ago by ngeoffray
Modified:
7 years, 3 months ago
Reviewers:
karlklose, blois, kustermann
CC:
reviews_dartlang.org
Visibility:
Public.

Description

HTML types that don't have [] do not implement JavaScriptIndexingBehavior, that is, the items cannot be accessed through [] in JavaScript. R=kustermann@google.com Committed: https://code.google.com/p/dart/source/detail?r=26911

Patch Set 1 #

Patch Set 2 : #

Unified diffs Side-by-side diffs Delta from patch set Stats (+23 lines, -19 lines) Patch
M sdk/lib/html/dart2js/html_dart2js.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/svg/dart2js/svg_dart2js.dart View 1 7 chunks +7 lines, -7 lines 0 comments Download
M sdk/lib/web_sql/dart2js/web_sql_dart2js.dart View 1 2 chunks +2 lines, -2 lines 0 comments Download
M tools/dom/scripts/systemhtml.py View 1 3 chunks +11 lines, -7 lines 0 comments Download
M tools/dom/templates/html/dart2js/svg_dart2js.darttemplate View 1 1 chunk +1 line, -1 line 0 comments Download
M tools/dom/templates/html/dart2js/web_sql_dart2js.darttemplate View 1 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 4 (0 generated)
ngeoffray
7 years, 3 months ago (2013-08-30 08:22:21 UTC) #1
ngeoffray
+Karl, whose type inference CL is blocked by the bug fixed by this CL.
7 years, 3 months ago (2013-08-30 11:40:37 UTC) #2
kustermann
lgtm
7 years, 3 months ago (2013-08-30 11:59:57 UTC) #3
ngeoffray
7 years, 3 months ago (2013-08-30 13:27:36 UTC) #4
Message was sent while issue was closed.
Committed patchset #2 manually as r26911 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698