drive by: make sure you run the layout tests with pixel diffs turned on so ...
16 years, 11 months ago
(2009-06-22 15:28:32 UTC)
#1
drive by: make sure you run the layout tests with pixel diffs turned on so it
looks at the crcs. They crc is run over the raw image data, so it needs to be
in the right layout.
This is required for a Chromium patch ( http://codereview.chromium.org/141037 ). Not sure what the best ...
16 years, 11 months ago
(2009-06-22 16:56:57 UTC)
#2
This is required for a Chromium patch ( http://codereview.chromium.org/141037 ).
Not sure what the best way is to land this, as chrome won't compile if this
lands first, and chrome's tests will fail if the chrome patch lands first. How
is this usually done?
I changed the #error into a #warning, so that this doesn't break the chrome mac ...
16 years, 11 months ago
(2009-06-22 18:14:20 UTC)
#4
I changed the #error into a #warning, so that this doesn't break the chrome mac
build when this lands. This way, it's possible to commit this to skia, roll the
skia deps in chromium, and then land the chromium part of this CL.
This also has the advantage of not breaking the build of other projects that
might use this file with a non-default channel configuration (at the cost of not
really informing them of the broken cgimages they get from this function).
patch submitted to skia/trunk On 2009/06/22 18:14:20, thakis wrote: > I changed the #error into ...
16 years, 11 months ago
(2009-06-22 20:04:51 UTC)
#5
patch submitted to skia/trunk
On 2009/06/22 18:14:20, thakis wrote:
> I changed the #error into a #warning, so that this doesn't break the chrome
mac
> build when this lands. This way, it's possible to commit this to skia, roll
the
> skia deps in chromium, and then land the chromium part of this CL.
>
> This also has the advantage of not breaking the build of other projects that
> might use this file with a non-default channel configuration (at the cost of
not
> really informing them of the broken cgimages they get from this function).
Issue 84041: Make SkCreateCGImageRef() work with a non-default channel config
Created 16 years, 11 months ago by thakis
Modified 16 years, 11 months ago
Reviewers: thomasvl, reed, agl
Base URL: http://skia.googlecode.com/svn/trunk/
Comments: 0