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

Issue 1727003002: Replace the now missing ContextAttributes with our previously Firefox-only _TypedContextAttributes (Closed)

Created:
4 years, 10 months ago by Alan Knight
Modified:
4 years, 10 months ago
Reviewers:
terry, Jacob
CC:
reviews_dartlang.org
Base URL:
git@github.com:dart-lang/sdk.git@master
Target Ref:
refs/heads/master
Visibility:
Public.

Description

Replace the now missing ContextAttributes with our previously Firefox-only _TypedContextAttributes BUG= R=jacobr@google.com Committed: https://github.com/dart-lang/sdk/commit/78b0f6f353b5349a50c3e38ef5f44ded1f7e1ee4

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+4 lines, -4 lines) Patch
M sdk/lib/html/html_common/conversions.dart View 3 chunks +3 lines, -3 lines 0 comments Download
M tools/dom/scripts/dartmetadata.py View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 5 (2 generated)
Alan Knight
I think we can just rename _TypedContextAttributes to ContextAttributes and always use it. Actually, it ...
4 years, 10 months ago (2016-02-23 23:24:18 UTC) #2
Jacob
lgtm
4 years, 10 months ago (2016-02-23 23:26:26 UTC) #3
Alan Knight
4 years, 10 months ago (2016-02-24 00:33:36 UTC) #5
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
78b0f6f353b5349a50c3e38ef5f44ded1f7e1ee4.

Powered by Google App Engine
This is Rietveld 408576698