From cb3d4fc7d6fa8e303983b5a0c185737a5ba6bc02 Mon Sep 17 00:00:00 2001 From: mtklein Date: Fri, 26 Feb 2016 16:57:33 -0800 Subject: Revert of Enable RAW codec for Windows (patchset #8 id:140001 of https://codereview.chromium.org/1738913002/ ) Reason for revert: I don't think there's anything wrong with this per-se, but the 32-bit Windows bots are running out of memory while running these tests now. (You'll see something like c:\0\build\slave\workdir\build\skia\include\core\skbitmap.h:247: fatal error: ""sk_throw"" in the log.) We run these tests in parallel, and sometimes these 32-bit processes try to use more than the 2-3G RAM they can allocate. Seems like this is a particularly memory-intense process? If we reland this, we might want to blacklist these tests on the 32-bit Windows bots. The 64-bit bots should have access to tons and tons of RAM and let us keep testing for Windows. Original issue's description: > Enable RAW codec for Windows > > * Fix the exception catching > * Set preprocessor differently for MSVC > > BUG=skia:4889(b/26958348) > GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1738913002 > > Committed: https://skia.googlesource.com/skia/+/474e4c3dd28b67f590851321f15d9983ef7fd031 TBR=scroggo@google.com,msarett@google.com,yujieqin@google.com # Skipping CQ checks because original CL landed less than 1 days ago. NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=skia:4889(b/26958348) Review URL: https://codereview.chromium.org/1747443003 --- src/codec/SkRawCodec.cpp | 43 ++++++++++++------------------------------- 1 file changed, 12 insertions(+), 31 deletions(-) (limited to 'src') diff --git a/src/codec/SkRawCodec.cpp b/src/codec/SkRawCodec.cpp index 9d0fb1cb25..a006e50837 100644 --- a/src/codec/SkRawCodec.cpp +++ b/src/codec/SkRawCodec.cpp @@ -10,20 +10,17 @@ #include "SkColorPriv.h" #include "SkData.h" #include "SkJpegCodec.h" -#include "SkMutex.h" #include "SkRawCodec.h" #include "SkRefCnt.h" #include "SkStream.h" #include "SkStreamPriv.h" #include "SkSwizzler.h" -#include "SkTArray.h" #include "SkTaskGroup.h" #include "SkTemplates.h" #include "SkTypes.h" #include "dng_area_task.h" #include "dng_color_space.h" -#include "dng_errors.h" #include "dng_exceptions.h" #include "dng_host.h" #include "dng_info.h" @@ -108,30 +105,15 @@ public: const std::vector taskAreas = compute_task_areas(maxTasks, area, tileSize); const int numTasks = static_cast(taskAreas.size()); - SkMutex mutex; - SkTArray exceptions; task.Start(numTasks, tileSize, &Allocator(), Sniffer()); for (int taskIndex = 0; taskIndex < numTasks; ++taskIndex) { - taskGroup.add([&mutex, &exceptions, &task, this, taskIndex, taskAreas, tileSize] { - try { - task.ProcessOnThread(taskIndex, taskAreas[taskIndex], tileSize, this->Sniffer()); - } catch (dng_exception& exception) { - SkAutoMutexAcquire lock(mutex); - exceptions.push_back(exception); - } catch (...) { - SkAutoMutexAcquire lock(mutex); - exceptions.push_back(dng_exception(dng_error_unknown)); - } + taskGroup.add([&task, this, taskIndex, taskAreas, tileSize] { + task.ProcessOnThread(taskIndex, taskAreas[taskIndex], tileSize, this->Sniffer()); }); } taskGroup.wait(); task.Finish(numTasks); - - // Currently we only re-throw the first catched exception. - if (!exceptions.empty()) { - Throw_dng_error(exceptions.front().ErrorCode(), nullptr, nullptr); - } } uint32 PerformAreaTaskThreads() override { @@ -446,15 +428,15 @@ public: } } + // render() takes ownership of fHost, fInfo, fNegative and fDngStream when available. + SkAutoTDelete host(fHost.release()); + SkAutoTDelete info(fInfo.release()); + SkAutoTDelete negative(fNegative.release()); + SkAutoTDelete dngStream(fDngStream.release()); + // DNG SDK preserves the aspect ratio, so it only needs to know the longer dimension. const int preferredSize = SkTMax(width, height); try { - // render() takes ownership of fHost, fInfo, fNegative and fDngStream when available. - SkAutoTDelete host(fHost.release()); - SkAutoTDelete info(fInfo.release()); - SkAutoTDelete negative(fNegative.release()); - SkAutoTDelete dngStream(fDngStream.release()); - host->SetPreferredSize(preferredSize); host->ValidateSizes(); @@ -524,12 +506,11 @@ private: } bool readDng() { + // Due to the limit of DNG SDK, we need to reset host and info. + fHost.reset(new SkDngHost(&fAllocator)); + fInfo.reset(new dng_info); + fDngStream.reset(new SkDngStream(fStream)); try { - // Due to the limit of DNG SDK, we need to reset host and info. - fHost.reset(new SkDngHost(&fAllocator)); - fInfo.reset(new dng_info); - fDngStream.reset(new SkDngStream(fStream)); - fHost->ValidateSizes(); fInfo->Parse(*fHost, *fDngStream); fInfo->PostParse(*fHost); -- cgit v1.2.3