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

Issue 2539263002: Fix for constructor search. (Closed)

Created:
4 years ago by scheglov
Modified:
4 years ago
CC:
reviews_dartlang.org
Target Ref:
refs/heads/master
Visibility:
Public.

Description

Patch Set 1 #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+50 lines, -2 lines) Patch
M pkg/analyzer/lib/src/dart/analysis/search.dart View 1 chunk +3 lines, -0 lines 0 comments Download
M pkg/analyzer/test/src/dart/analysis/search_test.dart View 2 chunks +47 lines, -2 lines 1 comment Download

Messages

Total messages: 7 (2 generated)
scheglov
4 years ago (2016-11-30 21:05:07 UTC) #1
Brian Wilkerson
lgtm
4 years ago (2016-11-30 21:46:10 UTC) #2
scheglov
Committed patchset #1 (id:1) manually as a0fa2083d355d76b5e314b6534c7f73a8ee77902 (presubmit successful).
4 years ago (2016-11-30 22:00:03 UTC) #4
Paul Berry
https://codereview.chromium.org/2539263002/diff/1/pkg/analyzer/test/src/dart/analysis/search_test.dart File pkg/analyzer/test/src/dart/analysis/search_test.dart (right): https://codereview.chromium.org/2539263002/diff/1/pkg/analyzer/test/src/dart/analysis/search_test.dart#newcode162 pkg/analyzer/test/src/dart/analysis/search_test.dart:162: String other = _p('$testProject/other.dart'); This test fails on Windows. ...
4 years ago (2016-12-01 13:18:04 UTC) #6
scheglov
4 years ago (2016-12-01 16:38:41 UTC) #7
Message was sent while issue was closed.
On 2016/12/01 13:18:04, Paul Berry wrote:
>
https://codereview.chromium.org/2539263002/diff/1/pkg/analyzer/test/src/dart/...
> File pkg/analyzer/test/src/dart/analysis/search_test.dart (right):
> 
>
https://codereview.chromium.org/2539263002/diff/1/pkg/analyzer/test/src/dart/...
> pkg/analyzer/test/src/dart/analysis/search_test.dart:162: String other =
> _p('$testProject/other.dart');
> This test fails on Windows.  See
>
https://build.chromium.org/p/client.dart/builders/analyzer-win7-release-be/bu...
> 
>   AnalysisException: Cannot compute DART_ERRORS for C:\test\lib/other.dart
>   Caused by Element mismatch in C:\test\lib\other.dart at
C:\test\lib\other.dart
>   #0      DeclarationResolver.resolve
> (package:analyzer/src/generated/declaration_resolver.dart:50:7)
>   #1      BuildCompilationUnitElementTask.internalPerform
> (package:analyzer/src/task/dart.dart:1082:33)
> 
> Can you have a look?

https://codereview.chromium.org/2542793003 will fix the failure.

Powered by Google App Engine
This is Rietveld 408576698