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

Issue 2986073002: Partial support for intelligent paste operation (Closed)

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

Description

Patch Set 1 #

Patch Set 2 : fix errors #

Total comments: 2
Unified diffs Side-by-side diffs Delta from patch set Stats (+667 lines, -76 lines) Patch
M pkg/analysis_server/lib/src/computer/import_elements_computer.dart View 1 chunk +379 lines, -25 lines 2 comments Download
M pkg/analysis_server/lib/src/edit/edit_domain.dart View 1 chunk +5 lines, -5 lines 0 comments Download
M pkg/analysis_server/test/src/computer/import_elements_computer_test.dart View 1 2 chunks +283 lines, -46 lines 0 comments Download

Messages

Total messages: 6 (2 generated)
Brian Wilkerson
I want to refactor the implementation to move a large portion of it into the ...
3 years, 4 months ago (2017-07-27 20:21:44 UTC) #2
scheglov
LGTM https://codereview.chromium.org/2986073002/diff/20001/pkg/analysis_server/lib/src/computer/import_elements_computer.dart File pkg/analysis_server/lib/src/computer/import_elements_computer.dart (right): https://codereview.chromium.org/2986073002/diff/20001/pkg/analysis_server/lib/src/computer/import_elements_computer.dart#newcode56 pkg/analysis_server/lib/src/computer/import_elements_computer.dart:56: SourceFactory sourceFactory = libraryElement.context.sourceFactory; It might be worth ...
3 years, 4 months ago (2017-07-27 20:54:15 UTC) #3
Brian Wilkerson
https://codereview.chromium.org/2986073002/diff/20001/pkg/analysis_server/lib/src/computer/import_elements_computer.dart File pkg/analysis_server/lib/src/computer/import_elements_computer.dart (right): https://codereview.chromium.org/2986073002/diff/20001/pkg/analysis_server/lib/src/computer/import_elements_computer.dart#newcode56 pkg/analysis_server/lib/src/computer/import_elements_computer.dart:56: SourceFactory sourceFactory = libraryElement.context.sourceFactory; I agree. I'll try to ...
3 years, 4 months ago (2017-07-27 20:55:33 UTC) #4
Brian Wilkerson
3 years, 4 months ago (2017-07-27 20:56:05 UTC) #6
Message was sent while issue was closed.
Committed patchset #2 (id:20001) manually as
9591a9449f28c9c39c68928d9576e518046e94cb (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698