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

Issue 131123006: roll skia DEPS to 13107 (Closed)

Created:
6 years, 11 months ago by robertphillips
Modified:
6 years, 11 months ago
Reviewers:
reed, bsalomon, rmistry, hal.canary
CC:
chromium-reviews, skia-team_google.com
Visibility:
Public.

Description

roll skia DEPS to 13107 Chromium base revision: 244998 / a10bed42 Old Skia revision: 13077 New Skia revision: 13107 Control CL: https://codereview.chromium.org/140193002 This CL was created by Skia's roll_deps.py script. R=bsalomon@google.com Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=245246

Patch Set 1 #

Patch Set 2 : Roll Skia to r13107 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+2 lines, -2 lines) Patch
M DEPS View 1 1 chunk +2 lines, -2 lines 0 comments Download

Messages

Total messages: 6 (0 generated)
robertphillips
This roll is blocked on webkit_unit_test failures: android_dbg roll: slave_steps webkit_unit_tests Currently running: ImageDecodingStoreTest.insertGenerationCollisionAfterMemoryDiscarded ImageDecodingStoreTest.lockCacheFailedAfterMemoryDiscarded: ...
6 years, 11 months ago (2014-01-16 12:42:34 UTC) #1
f(malita)
I'm also hitting that assert with a debug ToT Chromium build this morning: [14043:14043:0116/140941:FATAL:SkBitmap.cpp(1604)] ../../third_party/skia/src/core/SkBitmap.cpp:1604: ...
6 years, 11 months ago (2014-01-16 14:10:50 UTC) #2
robertphillips
Here are the CLs in this roll: https://code.google.com/p/skia/source/list?num=26&start=13103 The proximal cause is: r13093 - add ...
6 years, 11 months ago (2014-01-16 14:49:56 UTC) #3
robertphillips
This seems good enough to roll.
6 years, 11 months ago (2014-01-16 17:16:02 UTC) #4
bsalomon
On 2014/01/16 17:16:02, robertphillips wrote: > This seems good enough to roll. lgtm
6 years, 11 months ago (2014-01-16 18:02:27 UTC) #5
robertphillips
6 years, 11 months ago (2014-01-16 18:03:46 UTC) #6
Message was sent while issue was closed.
Committed patchset #2 manually as r245246 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698