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

Issues Search
    My Issues | Starred     Open | Closed | All

Issue 2438823002: Re-add null check for closure-data (Originally removed in 177dee8f1621eef43d5c820f1a0e9d91bce1dc6d) (Closed)

Created:
4 years, 2 months ago by Siggi Cherem (dart-lang)
Modified:
4 years, 2 months ago
Reviewers:
Johnni Winther
CC:
reviews_dartlang.org
Target Ref:
refs/heads/master
Visibility:
Public.

Description

Re-add null check for closure-data (Originally removed in 177dee8f1621eef43d5c820f1a0e9d91bce1dc6d) Committed: https://github.com/dart-lang/sdk/commit/61bf1c2dc0d57df22292ef5b85e3fb111a2eee14

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+7 lines, -5 lines) Patch
M pkg/compiler/lib/src/js_emitter/runtime_type_generator.dart View 1 chunk +7 lines, -5 lines 0 comments Download

Messages

Total messages: 5 (2 generated)
Siggi Cherem (dart-lang)
4 years, 2 months ago (2016-10-20 19:51:11 UTC) #2
Siggi Cherem (dart-lang)
Committed patchset #1 (id:1) manually as 61bf1c2dc0d57df22292ef5b85e3fb111a2eee14 (presubmit successful).
4 years, 2 months ago (2016-10-20 21:21:08 UTC) #4
Johnni Winther
4 years, 2 months ago (2016-10-21 07:47:03 UTC) #5
Message was sent while issue was closed. 
lgtm

Powered by Google App Engine
This is Rietveld 408576698