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

Issue 1584663006: Add EmbedderSdk to EmbedderUriResolver (Closed)

Created:
4 years, 11 months ago by danrubel
Modified:
4 years, 11 months ago
CC:
reviews_dartlang.org, Cutch
Base URL:
git@github.com:dart-lang/sdk.git@master
Target Ref:
refs/heads/master
Visibility:
Public.

Description

Patch Set 1 #

Total comments: 1

Patch Set 2 : merge #

Unified diffs Side-by-side diffs Delta from patch set Stats (+193 lines, -8 lines) Patch
M pkg/analyzer/lib/source/embedder.dart View 6 chunks +142 lines, -8 lines 0 comments Download
M pkg/analyzer/test/source/embedder_test.dart View 2 chunks +51 lines, -0 lines 0 comments Download

Messages

Total messages: 8 (3 generated)
danrubel
4 years, 11 months ago (2016-01-14 13:56:15 UTC) #2
Brian Wilkerson
LGTM https://codereview.chromium.org/1584663006/diff/1/pkg/analyzer/lib/source/embedder.dart File pkg/analyzer/lib/source/embedder.dart (right): https://codereview.chromium.org/1584663006/diff/1/pkg/analyzer/lib/source/embedder.dart#newcode22 pkg/analyzer/lib/source/embedder.dart:22: const String _DART_COLON_PREFIX = 'dart:'; nit: Personally, I ...
4 years, 11 months ago (2016-01-14 16:47:18 UTC) #3
pquitslund
LGTM!
4 years, 11 months ago (2016-01-14 17:22:47 UTC) #4
Cutch
LGTM Thanks!
4 years, 11 months ago (2016-01-14 17:32:41 UTC) #6
danrubel
4 years, 11 months ago (2016-01-14 19:51:46 UTC) #8
Message was sent while issue was closed.
Committed patchset #2 (id:20001) manually as
ebd1736c58d6d249e13eacf926c7136ea52f5105 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698