mtklein [Wed, 3 Sep 2014 21:06:47 +0000 (14:06 -0700)]
SkThreadPool ~~> SkTaskGroup
SkTaskGroup is like SkThreadPool except the threads stay in
one global pool. Each SkTaskGroup itself is tiny (4 bytes)
and its wait() method applies only to tasks add()ed to that
instance, not the whole thread pool.
This means we don't need to bring up new thread pools when
tests themselves want to use multithreading (e.g. pathops,
quilt). We just create a new SkTaskGroup and wait for that
to complete. This should be more efficient, and allow us
to expand where we use threads to really latency sensitive
places. E.g. we can probably now use these in nanobench
for CPU .skp rendering.
Now that all threads are sharing the same pool, I think we
can remove most of the custom mechanism pathops tests use
to control threading. They'll just ride on the global pool
with all other tests now.
This (temporarily?) removes the GPU multithreading feature
from DM, which we don't use.
On my desktop, DM runs a little faster (57s -> 55s) in
Debug, and a lot faster in Release (36s -> 24s). The bots
show speedups of similar proportions, cutting more than a
minute off the N4/Release and Win7/Debug runtimes.
BUG=skia:
R=caryclark@google.com, bsalomon@google.com, bungeman@google.com, mtklein@google.com, reed@google.com
Author: mtklein@chromium.org
Review URL: https://codereview.chromium.org/
531653002
bsalomon [Wed, 3 Sep 2014 21:05:49 +0000 (14:05 -0700)]
Add reference base class to GrGpuResourcewith pending IO references.
BUG=skia:2889
R=robertphillips@google.com
Author: bsalomon@google.com
Review URL: https://codereview.chromium.org/
533343002
Mike Klein [Wed, 3 Sep 2014 21:02:04 +0000 (17:02 -0400)]
fix up GM suppressions
BUG=skia:2911
Review URL: https://codereview.chromium.org/
538723002
robertphillips [Wed, 3 Sep 2014 20:46:02 +0000 (13:46 -0700)]
Remove use of EXPERIMENTAL_getActiveOps from layer hoisting code
This is getting in the way of switching to the SkRecord backend and is of questionable value.
R=bsalomon@google.com
Author: robertphillips@google.com
Review URL: https://codereview.chromium.org/
539693002
mtklein [Wed, 3 Sep 2014 20:40:52 +0000 (13:40 -0700)]
Turn downsamplebitmap GMs vertical to fit in 2048x2048.
BUG=skia:2911
R=bsalomon@google.com, mtklein@google.com
Author: mtklein@chromium.org
Review URL: https://codereview.chromium.org/
536103002
djsollen [Wed, 3 Sep 2014 20:38:32 +0000 (13:38 -0700)]
Revert to enabling all of -O2 as it exposes a bug in the toolchain.
Instead as a workaround we will temporarily disable tiling the
few GMs that produce errors with the existing 64-bit ARM toolchain.
BUG=skia:2908
R=mtklein@google.com
Author: djsollen@google.com
Review URL: https://codereview.chromium.org/
537713002
mtklein [Wed, 3 Sep 2014 20:22:46 +0000 (13:22 -0700)]
On Android, skip tiling for all giantbitmap variants.
BUG=skia:
R=djsollen@google.com, mtklein@google.com
Author: mtklein@chromium.org
Review URL: https://codereview.chromium.org/
537693002
piotaixr [Wed, 3 Sep 2014 19:34:57 +0000 (12:34 -0700)]
Remove SkBitmapCache::Find/Add(_,width,height,_)
in favor of the versions having a SkIRect as input parameter
BUG=skia:2909
R=reed@google.com, junov@chromium.org
Author: piotaixr@chromium.org
Review URL: https://codereview.chromium.org/
539643002
reed [Wed, 3 Sep 2014 18:54:58 +0000 (11:54 -0700)]
Hide fields in SkImageInfo
R=rmistry@google.com
TBR=bsalomon
Author: reed@google.com
Review URL: https://codereview.chromium.org/
536003002
piotaixr [Wed, 3 Sep 2014 18:33:13 +0000 (11:33 -0700)]
The key for SkBitmapCache can now be genID+SkIRect
BUG=skia:2894
R=junov@chromium.org, reed@google.com
Author: piotaixr@chromium.org
Review URL: https://codereview.chromium.org/
518983002
robertphillips [Wed, 3 Sep 2014 18:04:30 +0000 (11:04 -0700)]
Replace SkPictureReplacementPlayback with GrRecordReplaceDraw
I think this is sufficiently specialized to keep it in Ganesh for the time being.
R=bsalomon@google.com, mtklein@google.com
Author: robertphillips@google.com
Review URL: https://codereview.chromium.org/
535953002
djsollen [Wed, 3 Sep 2014 17:44:00 +0000 (10:44 -0700)]
Disable -fexpensive-optimizations on 64-bit ARM.
BUG=skia:2908
R=mtklein@google.com
Author: djsollen@google.com
Review URL: https://codereview.chromium.org/
535113004
mtklein [Wed, 3 Sep 2014 15:57:02 +0000 (08:57 -0700)]
Revert of Enable highQualityFilter_SSE2 (patchset #1 id:1 of https://codereview.chromium.org/
525283002/)
Reason for revert:
Color order looks wrong on Macs:
Before: http://chromium-skia-gm.commondatastorage.googleapis.com/gm/bitmap-64bitMD5/filterbitmap_image_mandrill_16.png/
12823183142873462143.png
After: http://chromium-skia-gm.commondatastorage.googleapis.com/gm/bitmap-64bitMD5/filterbitmap_image_mandrill_16.png/
13683040204546320578.png
Original issue's description:
> Enable highQualityFilter_SSE2
>
> With SSE2, bitmap_BGRA_8888_A_scale_rotate_bicubic gains about 40%
> performance improvement on desktop i7-3770.
>
> BUG=skia:
>
> Committed: https://skia.googlesource.com/skia/+/
b381fa10d8079c58928058bb8a6db32b39f05e51
R=humper@google.com, qiankun.miao@intel.com
TBR=humper@google.com, qiankun.miao@intel.com
NOTREECHECKS=true
NOTRY=true
BUG=skia:
Author: mtklein@google.com
Review URL: https://codereview.chromium.org/
539523002
jvanverth [Wed, 3 Sep 2014 15:44:59 +0000 (08:44 -0700)]
Update fontcache GM to stress the font atlas.
Pulled out of https://codereview.chromium.org/
466363009/.
R=bsalomon@google.com
Author: jvanverth@google.com
Review URL: https://codereview.chromium.org/
534253003
qiankun.miao [Wed, 3 Sep 2014 15:24:12 +0000 (08:24 -0700)]
Enable highQualityFilter_SSE2
With SSE2, bitmap_BGRA_8888_A_scale_rotate_bicubic gains about 40%
performance improvement on desktop i7-3770.
BUG=skia:
R=mtklein@google.com, humper@google.com
Author: qiankun.miao@intel.com
Review URL: https://codereview.chromium.org/
525283002
bsalomon [Wed, 3 Sep 2014 14:19:50 +0000 (07:19 -0700)]
Remove GrDrawTarget::AutoRenderTargetRestore.
Pass GrRenderTarget in GrGpuG clear\bind methods.
BUG=skia:2889
R=egdaniel@google.com
Author: bsalomon@google.com
Review URL: https://codereview.chromium.org/
533883004
mtklein [Wed, 3 Sep 2014 14:02:07 +0000 (07:02 -0700)]
Disable SSE4 S32A_Opaque blit.
This code sometimes generates a build warning that bothers Chrome.
BUG=399842,skia:2906
R=reed@google.com, mtklein@google.com
Author: mtklein@chromium.org
Review URL: https://codereview.chromium.org/
538463003
reed [Wed, 3 Sep 2014 13:31:06 +0000 (06:31 -0700)]
remove flags that are already mirrored in chrome's SkUserConfig.h
NOTRY=True
TBR=fmalita@google.com
Author: reed@google.com
Review URL: https://codereview.chromium.org/
537563002
qiankun.miao [Wed, 3 Sep 2014 13:27:54 +0000 (06:27 -0700)]
Remove dead code in SkBitmapFilter_opts_SSE2.h/cpp
BUG=skia:
R=mtklein@google.com, humper@google.com
Author: qiankun.miao@intel.com
Review URL: https://codereview.chromium.org/
530673002
Derek Sollenberger [Wed, 3 Sep 2014 13:10:20 +0000 (09:10 -0400)]
Update failing testimagefilter GM with new baseline and bug number
Review URL: https://codereview.chromium.org/
535843003
Derek Sollenberger [Wed, 3 Sep 2014 13:01:29 +0000 (09:01 -0400)]
Rebase NEON image procs based on C++ implementation.
BUG=skia:2845
Review URL: https://codereview.chromium.org/
532193003
reed [Wed, 3 Sep 2014 12:48:56 +0000 (05:48 -0700)]
Revert of Add gamma/sRGB tag to SkImageInfo (patchset #1 id:1 of https://codereview.chromium.org/
527073003/)
Reason for revert:
virtual gpu failures in layouttests
http://build.chromium.org/p/tryserver.blink/builders/linux_blink_dbg/builds/23717
[6:6:0903/041147:
1700960503:INFO:SkBitmap.cpp(1003)] ../../third_party/skia/src/core/SkBitmap.cpp:1003: failed assertion "fPixelRef->info() == pixelRef->info()"
[6:6:0903/041147:
1700961002:FATAL:SkBitmap.cpp(1003)] SK_CRASH
#0 0x7f9867df2c1e base::debug::StackTrace::StackTrace()
#1 0x7f9867e89e05 logging::LogMessage::~LogMessage()
#2 0x7f98689c4970 SkDebugf_FileLine()
#3 0x7f986870ebc6 SkBitmap::deepCopyTo()
#4 0x7f98690a3b6d blink::deepSkBitmapCopy()
#5 0x7f98690a3abe blink::ImageBuffer::copyImage()
#6 0x7f985fd4479e blink::ImageBitmap::ImageBitmap()
#7 0x7f985fd45942 blink::ImageBitmap::create()
#8 0x7f985fd84fdc blink::ImageBitmapFactories::createImageBitmap()
#9 0x7f985fd84d11 blink::ImageBitmapFactories::createImageBitmap()
#10 0x7f985ec9a7f2 blink::LocalDOMWindowV8Internal::createImageBitmap13Method()
#11 0x7f985ec96b0c blink::LocalDOMWindowV8Internal::createImageBitmapMethod()
#12 0x7f985ec91954 blink::LocalDOMWindowV8Internal::createImageBitmapMethodCallback()
#13 0x7f9869955af0 v8::internal::FunctionCallbackArguments::Call()
#14 0x7f98699833bd v8::internal::Builtin_HandleApiCall()
Original issue's description:
> Add gamma/sRGB tag to SkImageInfo
>
> This reverts commit
1cbc68f9659f15206d920dacd434ddf4b658ad1f.
>
> requires this to land in blink https://codereview.chromium.org/
531883002/
>
> Committed: https://skia.googlesource.com/skia/+/
2f6abdecc5c2f21da13003c615903679abc73fc7
R=fmalita@google.com, reed@chromium.org
TBR=fmalita@google.com, reed@chromium.org
NOTREECHECKS=true
NOTRY=true
Author: reed@google.com
Review URL: https://codereview.chromium.org/
535113002
Derek Sollenberger [Wed, 3 Sep 2014 12:31:02 +0000 (08:31 -0400)]
Update GM baselines for Arm64 bot.
Review URL: https://codereview.chromium.org/
530423003
reed [Wed, 3 Sep 2014 04:56:40 +0000 (21:56 -0700)]
Add gamma/sRGB tag to SkImageInfo
This reverts commit
1cbc68f9659f15206d920dacd434ddf4b658ad1f.
requires this to land in blink https://codereview.chromium.org/
531883002/
R=fmalita@google.com, reed@chromium.org
Author: reed@google.com
Review URL: https://codereview.chromium.org/
527073003
mtklein [Tue, 2 Sep 2014 22:19:48 +0000 (15:19 -0700)]
Fix drawPosText() bounds bug.
We didn't catch this in our local tests because we tend to use default
kUTF8_TextEncoding with single-byte characters, which means N == byteLength.
BUG=409110
R=reed@google.com, mtklein@google.com
Author: mtklein@chromium.org
Review URL: https://codereview.chromium.org/
531933002
reed [Tue, 2 Sep 2014 20:50:39 +0000 (13:50 -0700)]
add legacy guard for SK_SUPPORT_LEGACY_ALLOCPIXELS_BOOL to android
NOTRY=True
BUG=skia:
R=djsollen@google.com
Author: reed@google.com
Review URL: https://codereview.chromium.org/
530163002
pkasting [Tue, 2 Sep 2014 20:40:48 +0000 (13:40 -0700)]
Address MSVC warnings about possible value truncation. In the process removes some apparently unused code.
BUG=81439
TEST=none
R=reed@google.com
Author: pkasting@chromium.org
Review URL: https://codereview.chromium.org/
517763002
djsollen [Tue, 2 Sep 2014 20:34:49 +0000 (13:34 -0700)]
Remove legacy (and also broken) SkPicture constructor.
R=robertphillips@google.com, reed@google.com
Author: djsollen@google.com
Review URL: https://codereview.chromium.org/
532703004
jvanverth [Tue, 2 Sep 2014 20:15:40 +0000 (13:15 -0700)]
Fix matrix similarity test on arm64
Addresses precision issue by using a simpler test.
Also fixes issues with IvMatrix::preservesRightAngles, and adds unit tests.
BUG=skia:2405
R=robertphillips@google.com, egdaniel@google.com, djsollen@google.com, reed@google.com
Author: jvanverth@google.com
Review URL: https://codereview.chromium.org/
520123002
egdaniel [Tue, 2 Sep 2014 19:53:12 +0000 (12:53 -0700)]
Rebaseline bezier cubics
TBR=bsalomon@google.com
NOTRY=True
BUG=skia:
Author: egdaniel@google.com
Review URL: https://codereview.chromium.org/
528033002
reed [Tue, 2 Sep 2014 19:50:45 +0000 (12:50 -0700)]
make allocPixels throw on failure
BUG=skia:
R=mtklein@google.com, fmalita@google.com, fmalita@chromium.org
Author: reed@google.com
Review URL: https://codereview.chromium.org/
510423005
mtklein [Tue, 2 Sep 2014 19:03:31 +0000 (12:03 -0700)]
SkRecordPartialDraw with less code duplication
BUG=skia:
R=robertphillips@google.com, mtklein@google.com
Author: mtklein@chromium.org
Review URL: https://codereview.chromium.org/
527423002
piotaixr [Tue, 2 Sep 2014 18:50:59 +0000 (11:50 -0700)]
Rename ScaledImageCache.cpp to SkResourceCacheTest.cpp
BUG=skia:
R=junov@chromium.org, reed@android.com
Author: piotaixr@chromium.org
Review URL: https://codereview.chromium.org/
518213003
piotaixr [Tue, 2 Sep 2014 18:27:11 +0000 (11:27 -0700)]
Added Doc for GetAllocator in SkResourceCache and SkBitmapCache
BUG=skia:2896
R=junov@chromium.org, reed@google.com
Author: piotaixr@chromium.org
Review URL: https://codereview.chromium.org/
533713003
robertphillips [Tue, 2 Sep 2014 17:20:50 +0000 (10:20 -0700)]
Reorganize Layer Hoisting code
With the new MultiPictureDraw API the GrContext will be performing the layer hoisting (instead of the SkGpuDevice). This CL being moving the layer hoisting functionality to GrLayerHoister rather then dumping it straight into GrContext.
R=bsalomon@google.com
Author: robertphillips@google.com
Review URL: https://codereview.chromium.org/
531733003
egdaniel [Tue, 2 Sep 2014 17:01:30 +0000 (10:01 -0700)]
When ignoring color input don't use vertex attribs on gpu
Also rename SolidWhite_ColorInput enum to be AllOnes_ColorInput to better reflect what it's use is
BUG=skia:
R=bsalomon@google.com
Author: egdaniel@google.com
Review URL: https://codereview.chromium.org/
525433002
djsollen [Tue, 2 Sep 2014 15:57:39 +0000 (08:57 -0700)]
Disable NEON procs for box blur as it produces invalid results
R=reed@google.com, mtklein@google.com, senorblanco@google.com
TBR=reed@google.com
BUG=skia:2845
Author: djsollen@google.com
Review URL: https://codereview.chromium.org/
527973002
egdaniel [Tue, 2 Sep 2014 15:02:38 +0000 (08:02 -0700)]
Add option to add precision to varyings in shader
Also use highp varying on bezier cubics to fix numerical issues for n10 and n4.
BUG=skia:2860
R=bsalomon@google.com
Author: egdaniel@google.com
Review URL: https://codereview.chromium.org/
522303004
bsalomon [Tue, 2 Sep 2014 14:54:47 +0000 (07:54 -0700)]
Use non-VBO vertex arrays on QC GPUs.
BUG=skia:2901
R=robertphillips@google.com
Author: bsalomon@google.com
Review URL: https://codereview.chromium.org/
529993003
djsollen [Tue, 2 Sep 2014 13:32:25 +0000 (06:32 -0700)]
Revert of Disable NEON procs for box blur as it produces invalid results (patchset #1 id:1 of https://codereview.chromium.org/
520963002/)
Reason for revert:
failing more GMs than expected.
Original issue's description:
> Disable NEON procs for box blur as it produces invalid results
>
> BUG=skia:2845
>
> Committed: https://skia.googlesource.com/skia/+/
4a1764688c990fb926aaeab538497dad52768d99
R=senorblanco@google.com, mtklein@google.com
TBR=mtklein@google.com, senorblanco@google.com
NOTREECHECKS=true
NOTRY=true
BUG=skia:2845
Author: djsollen@google.com
Review URL: https://codereview.chromium.org/
531023002
djsollen [Tue, 2 Sep 2014 12:32:23 +0000 (05:32 -0700)]
Disable NEON procs for box blur as it produces invalid results
BUG=skia:2845
R=senorblanco@google.com, mtklein@google.com
Author: djsollen@google.com
Review URL: https://codereview.chromium.org/
520963002
Mike Klein [Tue, 2 Sep 2014 12:04:43 +0000 (08:04 -0400)]
whitespace change
BUG=skia:
Review URL: https://codereview.chromium.org/
529973002
reed [Tue, 2 Sep 2014 11:14:37 +0000 (04:14 -0700)]
Revert of Revert of Revert of Add gamma/sRGB tag to SkImageInfo (patchset #1 id:1 of https://codereview.chromium.org/
532583002/)
Reason for revert:
linux_blink_rel
[ RUN ] DeferredImageDecoderTest.drawIntoSkPicture
../../third_party/WebKit/Source/platform/graphics/DeferredImageDecoderTest.cpp:172: Failure
Value of: canvasBitmap.getColor(0, 0)
Actual: 0
Expected: static_cast<SkColor>( (static_cast<U8CPU>(255) << 24) | (static_cast<U8CPU>(255) << 16) | (static_cast<U8CPU>(255) << 8) | (static_cast<U8CPU>(255) << 0))
Which is:
4294967295
[ FAILED ] DeferredImageDecoderTest.drawIntoSkPicture (0 ms)
[1457/1458] DeferredImageDecoderTest.drawIntoSkPicture (0 ms)
[ RUN ] DeferredImageDecoderTest.decodeOnOtherThread
../../third_party/WebKit/Source/platform/graphics/DeferredImageDecoderTest.cpp:233: Failure
Value of: canvasBitmap.getColor(0, 0)
Actual: 0
Expected: static_cast<SkColor>( (static_cast<U8CPU>(255) << 24) | (static_cast<U8CPU>(255) << 16) | (static_cast<U8CPU>(255) << 8) | (static_cast<U8CPU>(255) << 0))
Which is:
4294967295
[ FAILED ] DeferredImageDecoderTest.decodeOnOtherThread (1 ms)
[1458/1458] DeferredImageDecoderTest.decodeOnOtherThread (1 ms)
Retrying 2 tests (retry #2)
Original issue's description:
> Revert of Revert of Add gamma/sRGB tag to SkImageInfo (patchset #1 id:1 of https://codereview.chromium.org/
525113005/)
>
> Reason for revert:
> Experiment to see resulting failures
>
> Original issue's description:
> > Revert of Add gamma/sRGB tag to SkImageInfo (patchset #1 id:1 of https://codereview.chromium.org/
522813002/)
> >
> > Reason for revert:
> > seems to be breaking layout tests in roll
> >
> > Original issue's description:
> > > Add gamma/sRGB tag to SkImageInfo
> > >
> > > This reverts commit
64ba5fa1ff428858f803523257cd862f8b33423b.
> > >
> > > BUG=skia:
> > >
> > > Committed: https://skia.googlesource.com/skia/+/
c89aa509d6a094bc1b18d73135343819903a9cfb
> >
> > TBR=reed@google.com
> > NOTREECHECKS=true
> > NOTRY=true
> > BUG=skia:
> >
> > Committed: https://skia.googlesource.com/skia/+/
b44c1895afae516cb851cd1a0cea83343c354ee4
>
> TBR=reed@google.com
> NOTREECHECKS=true
> NOTRY=true
> BUG=skia:
>
> Committed: https://skia.googlesource.com/skia/+/
4581828014eb3d015e6ed55c9a5b6932b8751818
R=reed@google.com
TBR=reed@google.com
NOTREECHECKS=true
NOTRY=true
BUG=skia:
Author: reed@chromium.org
Review URL: https://codereview.chromium.org/
524593005
reed [Tue, 2 Sep 2014 01:36:24 +0000 (18:36 -0700)]
Revert of Revert of Add gamma/sRGB tag to SkImageInfo (patchset #1 id:1 of https://codereview.chromium.org/
525113005/)
Reason for revert:
Experiment to see resulting failures
Original issue's description:
> Revert of Add gamma/sRGB tag to SkImageInfo (patchset #1 id:1 of https://codereview.chromium.org/
522813002/)
>
> Reason for revert:
> seems to be breaking layout tests in roll
>
> Original issue's description:
> > Add gamma/sRGB tag to SkImageInfo
> >
> > This reverts commit
64ba5fa1ff428858f803523257cd862f8b33423b.
> >
> > BUG=skia:
> >
> > Committed: https://skia.googlesource.com/skia/+/
c89aa509d6a094bc1b18d73135343819903a9cfb
>
> TBR=reed@google.com
> NOTREECHECKS=true
> NOTRY=true
> BUG=skia:
>
> Committed: https://skia.googlesource.com/skia/+/
b44c1895afae516cb851cd1a0cea83343c354ee4
R=reed@google.com
TBR=reed@google.com
NOTREECHECKS=true
NOTRY=true
BUG=skia:
Author: reed@chromium.org
Review URL: https://codereview.chromium.org/
532583002
reed [Mon, 1 Sep 2014 21:28:32 +0000 (14:28 -0700)]
Revert of Add gamma/sRGB tag to SkImageInfo (patchset #1 id:1 of https://codereview.chromium.org/
522813002/)
Reason for revert:
seems to be breaking layout tests in roll
Original issue's description:
> Add gamma/sRGB tag to SkImageInfo
>
> This reverts commit
64ba5fa1ff428858f803523257cd862f8b33423b.
>
> BUG=skia:
>
> Committed: https://skia.googlesource.com/skia/+/
c89aa509d6a094bc1b18d73135343819903a9cfb
R=reed@google.com
TBR=reed@google.com
NOTREECHECKS=true
NOTRY=true
BUG=skia:
Author: reed@chromium.org
Review URL: https://codereview.chromium.org/
525113005
reed [Mon, 1 Sep 2014 19:14:29 +0000 (12:14 -0700)]
Add gamma/sRGB tag to SkImageInfo
This reverts commit
64ba5fa1ff428858f803523257cd862f8b33423b.
BUG=skia:
R=reed@chromium.org
Author: reed@google.com
Review URL: https://codereview.chromium.org/
522813002
borenet [Mon, 1 Sep 2014 03:37:00 +0000 (20:37 -0700)]
Update SKP version to 90
Automatic commit by the RecreateSKPs bot.
TBR=
Author: borenet@google.com
Review URL: https://codereview.chromium.org/
528673002
Mike Klein [Sun, 31 Aug 2014 15:51:21 +0000 (11:51 -0400)]
Force a build.
BUG=skia:
Review URL: https://codereview.chromium.org/
530553002
borenet [Sun, 31 Aug 2014 03:50:55 +0000 (20:50 -0700)]
Update SKP version to 89
Automatic commit by the RecreateSKPs bot.
TBR=
Author: borenet@google.com
Review URL: https://codereview.chromium.org/
529543002
mtklein [Sat, 30 Aug 2014 21:33:06 +0000 (14:33 -0700)]
Revert of Revert of Revert of Revert of Revert of Revert of Revert of moar ws (patchset #1 id:1 of https://codereview.chromium.org/
524133002/)
Reason for revert:
another build
Original issue's description:
> Revert of Revert of Revert of Revert of Revert of Revert of moar ws (patchset #1 id:1 of https://codereview.chromium.org/
519413002/)
>
> Reason for revert:
> dun dun dun dun dah dah-dah-dah DAH dah dah-dah-dah DAH dah dah-dah-dah-dah
>
> Original issue's description:
> > Revert of Revert of Revert of Revert of Revert of moar ws (patchset #1 id:1 of https://codereview.chromium.org/
523413002/)
> >
> > Reason for revert:
> > ws
> >
> > Original issue's description:
> > > Revert of Revert of Revert of Revert of moar ws (patchset #1 id:1 of https://codereview.chromium.org/
519303002/)
> > >
> > > Reason for revert:
> > > ws
> > >
> > > Original issue's description:
> > > > Revert of Revert of Revert of moar ws (patchset #1 id:1 of https://codereview.chromium.org/
520023003/)
> > > >
> > > > Reason for revert:
> > > > spring break forever
> > > >
> > > > Original issue's description:
> > > > > Revert of Revert of moar ws (patchset #1 id:1 of https://codereview.chromium.org/
518333002/)
> > > > >
> > > > > Reason for revert:
> > > > > nah
> > > > >
> > > > > Original issue's description:
> > > > > > Revert of moar ws (patchset #1 id:1 of https://codereview.chromium.org/
518233004/)
> > > > > >
> > > > > > Reason for revert:
> > > > > > i like the old file better
> > > > > >
> > > > > > Original issue's description:
> > > > > > > moar ws
> > > > > > >
> > > > > > > BUG=skia:
> > > > > > >
> > > > > > > Committed: https://skia.googlesource.com/skia/+/7a353e9
> > > > > >
> > > > > > TBR=mtklein@chromium.org
> > > > > > NOTREECHECKS=true
> > > > > > NOTRY=true
> > > > > > BUG=skia:
> > > > > >
> > > > > > Committed: https://skia.googlesource.com/skia/+/
4ee3e529e2756275a978ae5e4763955a703191f2
> > > > >
> > > > > TBR=mtklein@chromium.org
> > > > > NOTREECHECKS=true
> > > > > NOTRY=true
> > > > > BUG=skia:
> > > > >
> > > > > Committed: https://skia.googlesource.com/skia/+/
5355536c790831f05cdd37a0ac84206ad1c3dc56
> > > >
> > > > TBR=mtklein@chromium.org
> > > > NOTREECHECKS=true
> > > > NOTRY=true
> > > > BUG=skia:
> > > >
> > > > Committed: https://skia.googlesource.com/skia/+/
2fe2ce41e3e5647c7aead4d76d75f5500f573fde
> > >
> > > TBR=mtklein@chromium.org
> > > NOTREECHECKS=true
> > > NOTRY=true
> > > BUG=skia:
> > >
> > > Committed: https://skia.googlesource.com/skia/+/
c7bee661a1e2b2f3b367bdd2b4176d1921aa69d4
> >
> > TBR=mtklein@chromium.org
> > NOTREECHECKS=true
> > NOTRY=true
> > BUG=skia:
> >
> > Committed: https://skia.googlesource.com/skia/+/
968212f837f1fbac011f4feb1e33ff27f5acc863
>
> TBR=mtklein@chromium.org
> NOTREECHECKS=true
> NOTRY=true
> BUG=skia:
>
> Committed: https://skia.googlesource.com/skia/+/
7843a24ab2c2dc4fe77e82b9689cde953d6abd7c
R=mtklein@chromium.org
TBR=mtklein@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=skia:
Author: mtklein@google.com
Review URL: https://codereview.chromium.org/
525153003
mtklein [Sat, 30 Aug 2014 15:47:49 +0000 (08:47 -0700)]
Revert of Revert of Revert of Revert of Revert of Revert of moar ws (patchset #1 id:1 of https://codereview.chromium.org/
519413002/)
Reason for revert:
dun dun dun dun dah dah-dah-dah DAH dah dah-dah-dah DAH dah dah-dah-dah-dah
Original issue's description:
> Revert of Revert of Revert of Revert of Revert of moar ws (patchset #1 id:1 of https://codereview.chromium.org/
523413002/)
>
> Reason for revert:
> ws
>
> Original issue's description:
> > Revert of Revert of Revert of Revert of moar ws (patchset #1 id:1 of https://codereview.chromium.org/
519303002/)
> >
> > Reason for revert:
> > ws
> >
> > Original issue's description:
> > > Revert of Revert of Revert of moar ws (patchset #1 id:1 of https://codereview.chromium.org/
520023003/)
> > >
> > > Reason for revert:
> > > spring break forever
> > >
> > > Original issue's description:
> > > > Revert of Revert of moar ws (patchset #1 id:1 of https://codereview.chromium.org/
518333002/)
> > > >
> > > > Reason for revert:
> > > > nah
> > > >
> > > > Original issue's description:
> > > > > Revert of moar ws (patchset #1 id:1 of https://codereview.chromium.org/
518233004/)
> > > > >
> > > > > Reason for revert:
> > > > > i like the old file better
> > > > >
> > > > > Original issue's description:
> > > > > > moar ws
> > > > > >
> > > > > > BUG=skia:
> > > > > >
> > > > > > Committed: https://skia.googlesource.com/skia/+/7a353e9
> > > > >
> > > > > TBR=mtklein@chromium.org
> > > > > NOTREECHECKS=true
> > > > > NOTRY=true
> > > > > BUG=skia:
> > > > >
> > > > > Committed: https://skia.googlesource.com/skia/+/
4ee3e529e2756275a978ae5e4763955a703191f2
> > > >
> > > > TBR=mtklein@chromium.org
> > > > NOTREECHECKS=true
> > > > NOTRY=true
> > > > BUG=skia:
> > > >
> > > > Committed: https://skia.googlesource.com/skia/+/
5355536c790831f05cdd37a0ac84206ad1c3dc56
> > >
> > > TBR=mtklein@chromium.org
> > > NOTREECHECKS=true
> > > NOTRY=true
> > > BUG=skia:
> > >
> > > Committed: https://skia.googlesource.com/skia/+/
2fe2ce41e3e5647c7aead4d76d75f5500f573fde
> >
> > TBR=mtklein@chromium.org
> > NOTREECHECKS=true
> > NOTRY=true
> > BUG=skia:
> >
> > Committed: https://skia.googlesource.com/skia/+/
c7bee661a1e2b2f3b367bdd2b4176d1921aa69d4
>
> TBR=mtklein@chromium.org
> NOTREECHECKS=true
> NOTRY=true
> BUG=skia:
>
> Committed: https://skia.googlesource.com/skia/+/
968212f837f1fbac011f4feb1e33ff27f5acc863
R=mtklein@chromium.org
TBR=mtklein@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=skia:
Author: mtklein@google.com
Review URL: https://codereview.chromium.org/
524133002
mtklein [Sat, 30 Aug 2014 15:06:37 +0000 (08:06 -0700)]
Revert of Revert of Revert of Revert of Revert of moar ws (patchset #1 id:1 of https://codereview.chromium.org/
523413002/)
Reason for revert:
ws
Original issue's description:
> Revert of Revert of Revert of Revert of moar ws (patchset #1 id:1 of https://codereview.chromium.org/
519303002/)
>
> Reason for revert:
> ws
>
> Original issue's description:
> > Revert of Revert of Revert of moar ws (patchset #1 id:1 of https://codereview.chromium.org/
520023003/)
> >
> > Reason for revert:
> > spring break forever
> >
> > Original issue's description:
> > > Revert of Revert of moar ws (patchset #1 id:1 of https://codereview.chromium.org/
518333002/)
> > >
> > > Reason for revert:
> > > nah
> > >
> > > Original issue's description:
> > > > Revert of moar ws (patchset #1 id:1 of https://codereview.chromium.org/
518233004/)
> > > >
> > > > Reason for revert:
> > > > i like the old file better
> > > >
> > > > Original issue's description:
> > > > > moar ws
> > > > >
> > > > > BUG=skia:
> > > > >
> > > > > Committed: https://skia.googlesource.com/skia/+/7a353e9
> > > >
> > > > TBR=mtklein@chromium.org
> > > > NOTREECHECKS=true
> > > > NOTRY=true
> > > > BUG=skia:
> > > >
> > > > Committed: https://skia.googlesource.com/skia/+/
4ee3e529e2756275a978ae5e4763955a703191f2
> > >
> > > TBR=mtklein@chromium.org
> > > NOTREECHECKS=true
> > > NOTRY=true
> > > BUG=skia:
> > >
> > > Committed: https://skia.googlesource.com/skia/+/
5355536c790831f05cdd37a0ac84206ad1c3dc56
> >
> > TBR=mtklein@chromium.org
> > NOTREECHECKS=true
> > NOTRY=true
> > BUG=skia:
> >
> > Committed: https://skia.googlesource.com/skia/+/
2fe2ce41e3e5647c7aead4d76d75f5500f573fde
>
> TBR=mtklein@chromium.org
> NOTREECHECKS=true
> NOTRY=true
> BUG=skia:
>
> Committed: https://skia.googlesource.com/skia/+/
c7bee661a1e2b2f3b367bdd2b4176d1921aa69d4
R=mtklein@chromium.org
TBR=mtklein@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=skia:
Author: mtklein@google.com
Review URL: https://codereview.chromium.org/
519413002
mtklein [Sat, 30 Aug 2014 14:54:38 +0000 (07:54 -0700)]
Revert of Revert of Revert of Revert of moar ws (patchset #1 id:1 of https://codereview.chromium.org/
519303002/)
Reason for revert:
ws
Original issue's description:
> Revert of Revert of Revert of moar ws (patchset #1 id:1 of https://codereview.chromium.org/
520023003/)
>
> Reason for revert:
> spring break forever
>
> Original issue's description:
> > Revert of Revert of moar ws (patchset #1 id:1 of https://codereview.chromium.org/
518333002/)
> >
> > Reason for revert:
> > nah
> >
> > Original issue's description:
> > > Revert of moar ws (patchset #1 id:1 of https://codereview.chromium.org/
518233004/)
> > >
> > > Reason for revert:
> > > i like the old file better
> > >
> > > Original issue's description:
> > > > moar ws
> > > >
> > > > BUG=skia:
> > > >
> > > > Committed: https://skia.googlesource.com/skia/+/7a353e9
> > >
> > > TBR=mtklein@chromium.org
> > > NOTREECHECKS=true
> > > NOTRY=true
> > > BUG=skia:
> > >
> > > Committed: https://skia.googlesource.com/skia/+/
4ee3e529e2756275a978ae5e4763955a703191f2
> >
> > TBR=mtklein@chromium.org
> > NOTREECHECKS=true
> > NOTRY=true
> > BUG=skia:
> >
> > Committed: https://skia.googlesource.com/skia/+/
5355536c790831f05cdd37a0ac84206ad1c3dc56
>
> TBR=mtklein@chromium.org
> NOTREECHECKS=true
> NOTRY=true
> BUG=skia:
>
> Committed: https://skia.googlesource.com/skia/+/
2fe2ce41e3e5647c7aead4d76d75f5500f573fde
R=mtklein@chromium.org
TBR=mtklein@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=skia:
Author: mtklein@google.com
Review URL: https://codereview.chromium.org/
523413002
Mike Klein [Sat, 30 Aug 2014 14:33:22 +0000 (10:33 -0400)]
Rebaseline one N4 GM we must have missed.
BUG=skia:
Review URL: https://codereview.chromium.org/
524103002
borenet [Sat, 30 Aug 2014 04:18:01 +0000 (21:18 -0700)]
Update SKP version to 88
Automatic commit by the RecreateSKPs bot.
TBR=
Author: borenet@google.com
Review URL: https://codereview.chromium.org/
520323002
mtklein [Sat, 30 Aug 2014 01:40:23 +0000 (18:40 -0700)]
Revert of Revert of Revert of moar ws (patchset #1 id:1 of https://codereview.chromium.org/
520023003/)
Reason for revert:
spring break forever
Original issue's description:
> Revert of Revert of moar ws (patchset #1 id:1 of https://codereview.chromium.org/
518333002/)
>
> Reason for revert:
> nah
>
> Original issue's description:
> > Revert of moar ws (patchset #1 id:1 of https://codereview.chromium.org/
518233004/)
> >
> > Reason for revert:
> > i like the old file better
> >
> > Original issue's description:
> > > moar ws
> > >
> > > BUG=skia:
> > >
> > > Committed: https://skia.googlesource.com/skia/+/7a353e9
> >
> > TBR=mtklein@chromium.org
> > NOTREECHECKS=true
> > NOTRY=true
> > BUG=skia:
> >
> > Committed: https://skia.googlesource.com/skia/+/
4ee3e529e2756275a978ae5e4763955a703191f2
>
> TBR=mtklein@chromium.org
> NOTREECHECKS=true
> NOTRY=true
> BUG=skia:
>
> Committed: https://skia.googlesource.com/skia/+/
5355536c790831f05cdd37a0ac84206ad1c3dc56
R=mtklein@chromium.org
TBR=mtklein@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=skia:
Author: mtklein@google.com
Review URL: https://codereview.chromium.org/
519303002
mtklein [Sat, 30 Aug 2014 01:33:24 +0000 (18:33 -0700)]
Revert of Revert of moar ws (patchset #1 id:1 of https://codereview.chromium.org/
518333002/)
Reason for revert:
nah
Original issue's description:
> Revert of moar ws (patchset #1 id:1 of https://codereview.chromium.org/
518233004/)
>
> Reason for revert:
> i like the old file better
>
> Original issue's description:
> > moar ws
> >
> > BUG=skia:
> >
> > Committed: https://skia.googlesource.com/skia/+/7a353e9
>
> TBR=mtklein@chromium.org
> NOTREECHECKS=true
> NOTRY=true
> BUG=skia:
>
> Committed: https://skia.googlesource.com/skia/+/
4ee3e529e2756275a978ae5e4763955a703191f2
R=mtklein@chromium.org
TBR=mtklein@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=skia:
Author: mtklein@google.com
Review URL: https://codereview.chromium.org/
520023003
mtklein [Fri, 29 Aug 2014 23:03:35 +0000 (16:03 -0700)]
Revert of moar ws (patchset #1 id:1 of https://codereview.chromium.org/
518233004/)
Reason for revert:
i like the old file better
Original issue's description:
> moar ws
>
> BUG=skia:
>
> Committed: https://skia.googlesource.com/skia/+/7a353e9
R=mtklein@chromium.org
TBR=mtklein@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=skia:
Author: mtklein@google.com
Review URL: https://codereview.chromium.org/
518333002
Mike Klein [Fri, 29 Aug 2014 22:52:18 +0000 (18:52 -0400)]
moar ws
BUG=skia:
Review URL: https://codereview.chromium.org/
518233004
Mike Klein [Fri, 29 Aug 2014 22:15:59 +0000 (18:15 -0400)]
another ws
BUG=skia:
Review URL: https://codereview.chromium.org/
519583008
fmalita [Fri, 29 Aug 2014 22:08:20 +0000 (15:08 -0700)]
Debugger render support for SkDrawTextBlobCommand.
R=robertphillips@google.com
Author: fmalita@chromium.org
Review URL: https://codereview.chromium.org/
523833003
bsalomon [Fri, 29 Aug 2014 22:05:53 +0000 (15:05 -0700)]
Move MakeDivByTextureWHMatrix to GrCoordTransform
R=robertphillips@google.com
Author: bsalomon@google.com
Review URL: https://codereview.chromium.org/
522873005
Mike Klein [Fri, 29 Aug 2014 21:53:34 +0000 (17:53 -0400)]
ws2
BUG=skia:
Review URL: https://codereview.chromium.org/
524853004
Mike Klein [Fri, 29 Aug 2014 21:20:44 +0000 (17:20 -0400)]
ws
BUG=skia:
Review URL: https://codereview.chromium.org/
523133002
Derek Sollenberger [Fri, 29 Aug 2014 19:46:28 +0000 (15:46 -0400)]
Add baselines for all GMs that are currently labeled "no comparison".
R=mtklein@google.com
Review URL: https://codereview.chromium.org/
519063004
Greg Humphreys [Fri, 29 Aug 2014 19:39:03 +0000 (15:39 -0400)]
rebaseline and re-enable all perspective tests
BUG=skia:
Review URL: https://codereview.chromium.org/
518253002
Greg Humphreys [Fri, 29 Aug 2014 19:35:49 +0000 (15:35 -0400)]
fix names of suppresssed tests on win7
BUG=skia:
Review URL: https://codereview.chromium.org/
524643003
Greg Humphreys [Fri, 29 Aug 2014 19:29:38 +0000 (15:29 -0400)]
partial rebaseline for image scaling
BUG=skia:
Review URL: https://codereview.chromium.org/
520863002
cdalton [Fri, 29 Aug 2014 19:18:36 +0000 (12:18 -0700)]
Don't assert on empty GrGLPaths
Ensure asserts aren't fired when creating a GrGLPath path from an
empty SkPath. An empty path is valid and has practical use, for
example, as a template path for certain nvpr calls.
BUG=skia:2891
R=bsalomon@google.com
Author: cdalton@nvidia.com
Review URL: https://codereview.chromium.org/
522883002
egdaniel [Fri, 29 Aug 2014 18:12:30 +0000 (11:12 -0700)]
Fix to use high precision on cubics not conics.
NOTRY=True
TBR=bsalomon@google.com
BUG=skia:2860
Author: egdaniel@google.com
Review URL: https://codereview.chromium.org/
521623003
bungeman [Fri, 29 Aug 2014 17:38:48 +0000 (10:38 -0700)]
Update FontConfig valgrind suppressions.
FcPattern uses 'intptr_t elts_offsets' instead of 'FcPatternEltPtr elts'
to reference its elements. Therefore, valgrind suspects possible leaks
on all values, elements, and element lists in FontConfig.
BUG=skia:2879
R=djsollen@google.com, mtklein@google.com
Author: bungeman@google.com
Review URL: https://codereview.chromium.org/
518903002
reed [Fri, 29 Aug 2014 17:25:26 +0000 (10:25 -0700)]
harden requirements on SkBitmapCache
BUG=skia:
R=humper@google.com
Author: reed@google.com
Review URL: https://codereview.chromium.org/
521433002
egdaniel [Fri, 29 Aug 2014 17:08:36 +0000 (10:08 -0700)]
Use highp for gpu bezier effects
BUG=skia:2860
R=bsalomon@google.com
Author: egdaniel@google.com
Review URL: https://codereview.chromium.org/
519473003
fmalita [Fri, 29 Aug 2014 16:46:36 +0000 (09:46 -0700)]
Quick-reject draw text blob calls.
R=mtklein@google.com, reed@google.com, robertphillips@google.com
Author: fmalita@chromium.org
Review URL: https://codereview.chromium.org/
494763004
borenet [Fri, 29 Aug 2014 16:43:06 +0000 (09:43 -0700)]
Rebaseline N10 Recipes bot
BUG=skia:
R=djsollen@google.com
Author: borenet@google.com
Review URL: https://codereview.chromium.org/
522803002
Derek Sollenberger [Fri, 29 Aug 2014 16:29:28 +0000 (12:29 -0400)]
Add Nexus 5 baselines to Android.
Review URL: https://codereview.chromium.org/
519813002
Mike Klein [Fri, 29 Aug 2014 16:24:50 +0000 (12:24 -0400)]
Start N5 skimage expectations off with N4's expectations
BUG=skia:
Review URL: https://codereview.chromium.org/
520753002
Derek Sollenberger [Fri, 29 Aug 2014 16:00:09 +0000 (12:00 -0400)]
Massive rebase of Nexus 4 bots.
Review URL: https://codereview.chromium.org/
517183003
george [Fri, 29 Aug 2014 15:47:55 +0000 (08:47 -0700)]
Use even rounding for better results when converting from scalar to fdot6
Originally from https://bugzilla.mozilla.org/show_bug.cgi?id=996108, patch by Jeff Muizelaar
R=reed@google.com, reed1
BUG=skia:
Author: george@mozilla.com
Review URL: https://codereview.chromium.org/
270263005
reed [Fri, 29 Aug 2014 15:23:55 +0000 (08:23 -0700)]
use possible corrected info when creating pixelref
TBR=halcanary, scroggo
Author: reed@google.com
Review URL: https://codereview.chromium.org/
518103002
bsalomon [Fri, 29 Aug 2014 15:08:35 +0000 (08:08 -0700)]
Make SkGrPixelRef responsible for owning GrSurface's texture rather than RT.
R=robertphillips@google.com
Author: bsalomon@google.com
Review URL: https://codereview.chromium.org/
522733003
robertphillips [Fri, 29 Aug 2014 15:03:56 +0000 (08:03 -0700)]
Try out scalar picture sizes
This paves the way for removing the 'fTile' parameter from SkPictureShader (although that should be a different CL). If we like this we could also move to providing an entire cull SkRect.
R=reed@google.com, mtklein@google.com, fmalita@google.com, fmalita@chromium.org
Author: robertphillips@google.com
Review URL: https://codereview.chromium.org/
513983002
borenet [Fri, 29 Aug 2014 14:56:14 +0000 (07:56 -0700)]
Fix adb_pull_if_needed
BUG=skia:
R=djsollen@google.com
Author: borenet@google.com
Review URL: https://codereview.chromium.org/
519743002
mtklein [Fri, 29 Aug 2014 14:55:34 +0000 (07:55 -0700)]
Update LSAN suppressions.
Seems the obvious thing works now.
Ran nanobench,
-----------------------------------------------------
Suppressions used:
count bytes template
586 3530064 libGL.so
-----------------------------------------------------
and dm,
-----------------------------------------------------
Suppressions used:
count bytes template
340 2048160 libGL.so
-----------------------------------------------------
BUG=skia:
R=robertphillips@google.com, mtklein@google.com
Author: mtklein@chromium.org
Review URL: https://codereview.chromium.org/
517133002
reed [Fri, 29 Aug 2014 14:50:28 +0000 (07:50 -0700)]
Revert of Add gamma/sRGB tags to SkImageInfo (patchset #1 of https://codereview.chromium.org/
517123002/)
Reason for revert:
Seems to be triggering assert in blink
SSLUITest.TestRedirectHTTPToBadHTTPS (run #1):
[ RUN ] SSLUITest.TestRedirectHTTPToBadHTTPS
HTTP server started on http://127.0.0.1:58000...
sending server_data: {"host": "127.0.0.1", "port": 58000} (36 bytes)
HTTPS server started on https://127.0.0.1:58009...
sending server_data: {"host": "127.0.0.1", "port": 58009} (36 bytes)
ASSERTION FAILED: info.fAlphaType == m_imageInfo.fAlphaType
../../third_party/WebKit/Source/platform/graphics/DecodingImageGenerator.cpp(78) : virtual bool blink::DecodingImageGenerator::onGetPixels(const SkImageInfo &, void *, size_t, SkPMColor *, int *)
1 0x77eb0d3 blink::DecodingImageGenerator::onGetPixels(SkImageInfo const&, void*, unsigned long, unsigned int*, int*)
2 0x92edddc SkImageGenerator::getPixels(SkImageInfo const&, void*, unsigned long, unsigned int*, int*)
3 0x92adf78 SkDiscardablePixelRef::onNewLockPixels(SkPixelRef::LockRec*)
4 0x9369283 SkPixelRef::lockPixels(SkPixelRef::LockRec*)
5 0x9369433 SkPixelRef::lockPixels()
6 0x9213344 SkBitmap::lockPixels() const
7 0x921ca57 SkAutoLockPixels::SkAutoLockPixels(SkBitmap const&, bool)
8 0x921ad80 SkAutoLockPixels::SkAutoLockPixels(SkBitmap const&, bool)
9 0x92b7125 SkDraw::drawBitmap(SkBitmap const&, SkMatrix const&, SkPaint const&) const
10 0x921f4fb SkBitmapDevice::drawBitmap(SkDraw const&, SkBitmap const&, SkMatrix const&, SkPaint const&)
11 0x921f8c7 SkBitmapDevice::drawBitmapRect(SkDraw const&, SkBitmap const&, SkRect const*, SkRect const&, SkPaint const&, SkCanvas::DrawBitmapRectFlags)
12 0x9288e12 SkCanvas::internalDrawBitmapRect(SkBitmap const&, SkRect const*, SkRect const&, SkPaint const*, SkCanvas::DrawBitmapRectFlags)
13 0x9288ee9 SkCanvas::drawBitmapRectToRect(SkBitmap const&, SkRect const*, SkRect const&, SkPaint const*, SkCanvas::DrawBitmapRectFlags)
Original issue's description:
> Add gamma/sRGB tags to SkImageInfo
>
> requires this CL to land in chrome
> https://codereview.chromium.org/
517803002/
>
> BUG=skia:
>
> Committed: https://skia.googlesource.com/skia/+/
228b285ba14a6e9b6d1cc95ea1583caab30168a1
R=fmalita@google.com, fmalita@chromium.org, reed@chromium.org
TBR=fmalita@chromium.org, fmalita@google.com, reed@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=skia:
Author: reed@google.com
Review URL: https://codereview.chromium.org/
519583004
bsalomon [Fri, 29 Aug 2014 14:50:23 +0000 (07:50 -0700)]
Simplify SkGpuDevice construction
R=robertphillips@google.com, reed@google.com
Author: bsalomon@google.com
Review URL: https://codereview.chromium.org/
506413004
humper [Fri, 29 Aug 2014 14:45:23 +0000 (07:45 -0700)]
suppress one more filterbitmap GM
BUG=skia:
TBR=djsollen
Author: humper@google.com
Review URL: https://codereview.chromium.org/
522703002
humper [Fri, 29 Aug 2014 14:44:58 +0000 (07:44 -0700)]
more suppressions to handle differences on Win7 bot
BUG=skia:
TBR=djsollen
Author: humper@google.com
Review URL: https://codereview.chromium.org/
518833002
Derek Sollenberger [Fri, 29 Aug 2014 14:31:49 +0000 (10:31 -0400)]
Add empty expectations file for skimage on Nexus 5 (Debug).
Review URL: https://codereview.chromium.org/
519763002
Derek Sollenberger [Fri, 29 Aug 2014 14:27:35 +0000 (10:27 -0400)]
Add empty expectations file for skimage on Nexus 5.
Review URL: https://codereview.chromium.org/
519753002
Mike Klein [Fri, 29 Aug 2014 14:03:59 +0000 (10:03 -0400)]
Revert Jim's CL.
BUG=skia:
Review URL: https://codereview.chromium.org/
517123005
reed [Fri, 29 Aug 2014 11:08:43 +0000 (04:08 -0700)]
Add gamma/sRGB tags to SkImageInfo
requires this CL to land in chrome
https://codereview.chromium.org/
517803002/
BUG=skia:
R=fmalita@google.com, fmalita@chromium.org, reed@chromium.org
Author: reed@google.com
Review URL: https://codereview.chromium.org/
517123002
borenet [Fri, 29 Aug 2014 04:17:53 +0000 (21:17 -0700)]
Update SKP version to 87
Automatic commit by the RecreateSKPs bot.
TBR=
Author: borenet@google.com
Review URL: https://codereview.chromium.org/
523533002
reed [Thu, 28 Aug 2014 21:46:19 +0000 (14:46 -0700)]
Revert of Revert of Revert of add gamma/sRGB to SkImageInfo (patchset #1 of https://codereview.chromium.org/
511363002/)
Reason for revert:
android builder in chrome reveals the caller using { } initialization for SkImageInfo, which this CL now makes illegal. Will have to fix the call site before I can re-land this
Original issue's description:
> Revert of Revert of add gamma/sRGB to SkImageInfo (patchset #1 of https://codereview.chromium.org/
512243002/)
>
> Reason for revert:
> this revert was premature -- the chrome canary I looked at is old, and newer ones were green. So trying again...
>
> Original issue's description:
> > Revert of add gamma/sRGB to SkImageInfo (patchset #4 of https://codereview.chromium.org/
514753002/)
> >
> > Reason for revert:
> > breaks linker on chrome -- may need SK_API, but not sure.
> >
> > Original issue's description:
> > > add gamma/sRGB to SkImageInfo
> > >
> > > BUG=skia:
> > >
> > > Committed: https://skia.googlesource.com/skia/+/
615c369777258231054840a88cdb74c68c382485
> >
> > TBR=bungeman@google.com,bsalomon@google.com
> > NOTREECHECKS=true
> > NOTRY=true
> > BUG=skia:
> >
> > Committed: https://skia.googlesource.com/skia/+/
dc53701216c0e9f98eacf6c0cda769ebbd200b59
>
> TBR=bungeman@google.com,bsalomon@google.com
> NOTREECHECKS=true
> NOTRY=true
> BUG=skia:
>
> Committed: https://skia.googlesource.com/skia/+/
e759a2662273fdf52ebbb3822229945eda317e46
R=bungeman@google.com, bsalomon@google.com
TBR=bsalomon@google.com, bungeman@google.com
NOTREECHECKS=true
NOTRY=true
BUG=skia:
Author: reed@google.com
Review URL: https://codereview.chromium.org/
517113002
reed [Thu, 28 Aug 2014 21:44:23 +0000 (14:44 -0700)]
Revert of add SK_API to SkImageInfo (patchset #1 of https://codereview.chromium.org/
513333003/)
Reason for revert:
need to revert the big ImageInfo CL but since this was landed on top of it, I need to revert this guy first.
Original issue's description:
> add SK_API to SkImageInfo
>
> BUG=skia:
> TBR=bungeman
> NOTRY=True
>
> Committed: https://skia.googlesource.com/skia/+/
bedec8456ff0ea814ed1bba748a86678f458d063
TBR=
NOTREECHECKS=true
NOTRY=true
BUG=skia:
Author: reed@google.com
Review URL: https://codereview.chromium.org/
518543002
fmalita [Thu, 28 Aug 2014 21:32:24 +0000 (14:32 -0700)]
Expose drawTextBlob to devices.
Instead of unrolling blobs in SkCanvas, perform the equivalent ops in
SkBaseDevice.
This depends on https://codereview.chromium.org/
511783005/.
R=jvanverth@google.com, reed@google.com, robertphillips@google.com, bsalomon@google.com
Author: fmalita@chromium.org
Review URL: https://codereview.chromium.org/
517663003
humper [Thu, 28 Aug 2014 21:27:42 +0000 (14:27 -0700)]
Second attempt to land the integral image scaling change.
Scale all images to the nearest rounded integer, and if there's still
any scaling factor left over, pass it on to the subsequent bilerp code.
Should avoid artifacts when tiling scaled images.
Original CL received an LGTM from reed; new version disabled tiling
in the downsamplebitmap GM; I verified that this fixes the issue
we were seeing there on non-neon androids.
BUG=skia:2888
R=reed@android.com
TBR=reed
Author: humper@google.com
Review URL: https://codereview.chromium.org/
514383003
mtklein [Thu, 28 Aug 2014 21:10:05 +0000 (14:10 -0700)]
Align all SkRecord::alloc() calls up to at least a pointer size.
This should make the LSAN bots able to see all our pointers.
BUG=skia:
R=reed@google.com, robertphillips@google.com, mtklein@google.com
Author: mtklein@chromium.org
Review URL: https://codereview.chromium.org/
517073002
bsalomon [Thu, 28 Aug 2014 21:06:42 +0000 (14:06 -0700)]
Make SkGpuDevice constructors private.
R=robertphillips@google.com
Author: bsalomon@google.com
Review URL: https://codereview.chromium.org/
517033002
bsalomon [Thu, 28 Aug 2014 21:04:55 +0000 (14:04 -0700)]
Add dirty_after_edit to SkPath::moveTo. Add commented out test case that triggers assert.
BUG=skia:1460
R=reed@google.com
Author: bsalomon@google.com
Review URL: https://codereview.chromium.org/
517023003