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

Issue 71463003: "Fix" another crash in DeferredCanvasTest on valgrind bot (Closed)

Created:
7 years, 1 month ago by robertphillips
Modified:
7 years, 1 month ago
Reviewers:
jvanverth1
CC:
skia-review_googlegroups.com
Visibility:
Public.

Description

This fixes the following issue on the valgrind bot: Process terminating with default action of signal 11 (SIGSEGV) Access not within mapped region at address 0x18 at 0x581A13: SkSurface::getCanvas() (SkSurface_Base.h:75) by 0x426CAC: getSurfacePixelPtr(SkSurface*, bool) (DeferredCanvasTest.cpp:690) by 0x42C62A: TestDeferredCanvasSetSurface(skiatest::Reporter*, GrContextFactory*) (DeferredCanvasTest.cpp:793) by 0x42D46E: skiatest::TestDeferredCanvasClass::onRun(skiatest::Reporter*) (DeferredCanvasTest.cpp:847) by 0x49670A: skiatest::Test::run() (Test.cpp:109) by 0x48FDA0: SkTestRunnable::run() (skia_test.cpp:155) by 0x48F746: tool_main(int, char**) (skia_test.cpp:246) by 0x48F922: main (skia_test.cpp:276)

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+3 lines, -0 lines) Patch
M tests/DeferredCanvasTest.cpp View 1 chunk +3 lines, -0 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
robertphillips
7 years, 1 month ago (2013-11-13 14:05:32 UTC) #1
jvanverth1
lgtm
7 years, 1 month ago (2013-11-13 14:24:33 UTC) #2
robertphillips
7 years, 1 month ago (2013-11-13 14:27:12 UTC) #3
Message was sent while issue was closed.
committed as r12265

Powered by Google App Engine
This is Rietveld 408576698