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

Issue 1596693002: simplify EmbedderUrlResolver (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 #

Patch Set 2 : cleanup imports #

Patch Set 3 : merge #

Unified diffs Side-by-side diffs Delta from patch set Stats (+43 lines, -109 lines) Patch
M pkg/analyzer/lib/source/embedder.dart View 5 chunks +12 lines, -94 lines 0 comments Download
M pkg/analyzer/test/source/embedder_test.dart View 1 4 chunks +31 lines, -15 lines 0 comments Download

Messages

Total messages: 5 (2 generated)
danrubel
4 years, 11 months ago (2016-01-15 23:07:27 UTC) #2
Brian Wilkerson
Wow! That's a really nice simplification! LGTM
4 years, 11 months ago (2016-01-15 23:29:20 UTC) #3
danrubel
4 years, 11 months ago (2016-01-16 02:22:32 UTC) #5
Message was sent while issue was closed.
Committed patchset #3 (id:40001) manually as
1ef8fc8cec4cb8e77d6fb8da71c5fbb644651477 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698