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

Issue 259743003: Integrate fix for issue 18452 on the 1.3 branch. (Closed)

Created:
6 years, 7 months ago by ricow1
Modified:
6 years, 6 months ago
Reviewers:
zra, Ivan Posva
CC:
reviews_dartlang.org
Visibility:
Public.

Description

Integrate fix for issue 18452 on the 1.3 branch. This is effectively merging revision r35524 from bleeding edge to 1.3 branch

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+5 lines, -3 lines) Patch
M dart/runtime/vm/assembler_x64.cc View 2 chunks +5 lines, -3 lines 0 comments Download
M dart/runtime/vm/stub_code.h View 2 chunks +1 line, -1 line 0 comments Download
A + dart/tests/standalone/io/directory_list_sync_test.dart View 0 chunks +-1 lines, --1 lines 0 comments Download

Messages

Total messages: 5 (0 generated)
ricow1
6 years, 7 months ago (2014-04-29 17:25:32 UTC) #1
zra
lgtm
6 years, 7 months ago (2014-04-29 17:33:43 UTC) #2
Ivan Posva
LGTM -ip
6 years, 7 months ago (2014-04-30 00:42:49 UTC) #3
Ivan Posva
Is this still an active review? -Ivan
6 years, 6 months ago (2014-06-05 23:45:07 UTC) #4
Ivan Posva
6 years, 6 months ago (2014-06-09 20:50:02 UTC) #5
On 2014/06/05 23:45:07, Ivan Posva wrote:
> Is this still an active review?
> 
> -Ivan

Closing. -ip

Powered by Google App Engine
This is Rietveld 408576698