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

Issue 1752003: Roll Skia forward to r524. (Closed)

Created:
10 years, 8 months ago by Evan Martin
Modified:
9 years, 7 months ago
Reviewers:
Stephen White, agl
CC:
chromium-reviews, darin-cc_chromium.org, dpranke+watch_chromium.org, pam+watch_chromium.org
Visibility:
Public.

Description

Roll Skia forward to r524. My target is farther than that, but I'm doing this incrementally. Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=45223

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+5 lines, -1 line) Patch
M DEPS View 1 chunk +1 line, -1 line 0 comments Download
M webkit/tools/layout_tests/test_expectations.txt View 1 chunk +4 lines, -0 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
Evan Martin
10 years, 8 months ago (2010-04-21 19:56:14 UTC) #1
agl
LGTM although a voice in my head says that we might have moved test_expectations upstream.
10 years, 8 months ago (2010-04-21 19:59:04 UTC) #2
Evan Martin
On 2010/04/21 19:59:04, agl wrote: > LGTM although a voice in my head says that ...
10 years, 8 months ago (2010-04-21 20:00:05 UTC) #3
Stephen White
10 years, 8 months ago (2010-04-21 20:02:09 UTC) #4
I think this is ok -- we have a local copy that we can append tests for
temporary situations, like this one.

On Wed, Apr 21, 2010 at 3:59 PM, <agl@chromium.org> wrote:

> LGTM although a voice in my head says that we might have moved
> test_expectations
> upstream.
>
>
> http://codereview.chromium.org/1752003/show
>



-- 
All truth passes through three stages. First, it is ridiculed. Second, it is
violently opposed. Third, it is accepted as being self-evident. --
Schopenhauer

Powered by Google App Engine
This is Rietveld 408576698