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

Issue 1733843007: Add tests for indexing. (Closed)

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

Description

Add tests for indexing. No tests yet for multi-level constructor redirection and qualified/unqualified references. R=paulberry@google.com BUG= Committed: https://github.com/dart-lang/sdk/commit/c106ad40331ae70acda02e8fdbd088f772e17524

Patch Set 1 #

Total comments: 3
Unified diffs Side-by-side diffs Delta from patch set Stats (+625 lines, -56 lines) Patch
M pkg/analyzer/lib/src/summary/format.dart View 10 chunks +45 lines, -2 lines 0 comments Download
M pkg/analyzer/lib/src/summary/idl.dart View 2 chunks +32 lines, -0 lines 1 comment Download
M pkg/analyzer/lib/src/summary/index_unit.dart View 13 chunks +45 lines, -33 lines 2 comments Download
M pkg/analyzer/test/src/summary/index_unit_test.dart View 8 chunks +503 lines, -21 lines 0 comments Download

Messages

Total messages: 4 (1 generated)
scheglov
4 years, 10 months ago (2016-02-26 18:52:14 UTC) #1
Paul Berry
lgtm https://codereview.chromium.org/1733843007/diff/1/pkg/analyzer/lib/src/summary/idl.dart File pkg/analyzer/lib/src/summary/idl.dart (right): https://codereview.chromium.org/1733843007/diff/1/pkg/analyzer/lib/src/summary/idl.dart#newcode138 pkg/analyzer/lib/src/summary/idl.dart:138: * The element itself. I'm confused. How can ...
4 years, 10 months ago (2016-02-26 19:02:05 UTC) #2
scheglov
4 years, 10 months ago (2016-02-26 19:19:06 UTC) #4
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
c106ad40331ae70acda02e8fdbd088f772e17524 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698