Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(292)

Issue 84041: Make SkCreateCGImageRef() work with a non-default channel config

Can't Edit
Can't Publish+Mail
Start Review
Created:
16 years, 11 months ago by thakis
Modified:
16 years, 11 months ago
Reviewers:
reed, thomasvl, agl
Base URL:
http://skia.googlecode.com/svn/trunk/
Visibility:
Public.

Description

This is required for the chromium patch http://codereview.chromium.org/141037 .

Patch Set 1 #

Patch Set 2 : Make SkCreateCGImageRef() work with a non-default channel config #

Unified diffs Side-by-side diffs Delta from patch set Stats (+25 lines, -1 line) Patch
M src/utils/mac/SkCreateCGImageRef.cpp View 1 1 chunk +25 lines, -1 line 0 comments Download

Messages

Total messages: 5
thomasvl
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
thakis
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
jrg
Idea for landing: Land in Skia. Add DEPS file update to your other CL so ...
16 years, 11 months ago (2009-06-22 18:13:35 UTC) #3
thakis
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
reed
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).
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b