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

Issue 2742343006: Formalizing the hacks letting the angular analyzer plugin run for now. (Closed)

Created:
3 years, 9 months ago by mfairhurst
Modified:
3 years, 9 months ago
Reviewers:
Brian Wilkerson
CC:
reviews_dartlang.org
Target Ref:
refs/heads/master
Visibility:
Public.

Description

Formalizing the hacks letting the angular analyzer plugin run for now. Mostly just making private methods public. But also add in some hooks (via streams and callbacks) that allow the angular analyzer to wrap analysis drivers in new angular drivers. This code will likely all be reverted at some point when a much much better plugin API is available. BUG= R=brianwilkerson@google.com Committed: https://github.com/dart-lang/sdk/commit/ebab20824a201dc061e80273c6e2be86e7af1efc

Patch Set 1 #

Total comments: 2

Patch Set 2 : Remove TODO, use correct event on file change #

Patch Set 3 : Rename variable not method #

Unified diffs Side-by-side diffs Delta from patch set Stats (+130 lines, -28 lines) Patch
M pkg/analysis_server/lib/src/analysis_server.dart View 1 2 6 chunks +39 lines, -3 lines 0 comments Download
M pkg/analysis_server/lib/src/context_manager.dart View 8 chunks +22 lines, -8 lines 0 comments Download
M pkg/analysis_server/lib/src/server/driver.dart View 3 chunks +9 lines, -3 lines 0 comments Download
M pkg/analysis_server/lib/starter.dart View 2 chunks +6 lines, -1 line 0 comments Download
M pkg/analyzer/lib/src/context/builder.dart View 3 chunks +15 lines, -1 line 0 comments Download
M pkg/analyzer/lib/src/dart/analysis/driver.dart View 13 chunks +39 lines, -12 lines 0 comments Download

Messages

Total messages: 5 (2 generated)
mfairhurst
3 years, 9 months ago (2017-03-13 20:24:39 UTC) #2
Brian Wilkerson
lgtm https://codereview.chromium.org/2742343006/diff/1/pkg/analysis_server/lib/src/analysis_server.dart File pkg/analysis_server/lib/src/analysis_server.dart (right): https://codereview.chromium.org/2742343006/diff/1/pkg/analysis_server/lib/src/analysis_server.dart#newcode386 pkg/analysis_server/lib/src/analysis_server.dart:386: // "new NotificationManager(channel, resourceProvider)" You can delete the ...
3 years, 9 months ago (2017-03-13 21:00:22 UTC) #3
mfairhurst
3 years, 9 months ago (2017-03-14 19:09:31 UTC) #5
Message was sent while issue was closed.
Committed patchset #3 (id:40001) manually as
ebab20824a201dc061e80273c6e2be86e7af1efc (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698