Set gamma correction settings to match platform.

Set gamma correction settings to match platform.

BUG=144809, 145317, 141425

[bungeman-chromium, 2012-08-28 22:06:33]

See http://code.google.com/p/chromium/issues/detail?id=145317

This change states what we're currently doing on Windows and Mac, and makes
Linux mimick what Chrome text looked like M21.

[kerz_google, 2012-08-28 22:13:16]

lgtm

[Nico, 2012-08-28 22:28:20]

Thanks! Can you add a BUG= line to your CL description?

http://codereview.chromium.org/10871091/diff/1/skia/skia.gyp
File skia/skia.gyp (right):

http://codereview.chromium.org/10871091/diff/1/skia/skia.gyp#newcode253
skia/skia.gyp:253: [ 'OS == "android" or OS == "linux"', {
You might want to check with the android folks if they want to match linux's
text rendering.

[Daniel Erat, 2012-08-29 01:23:03]

Drive-by comment: If this is the cause of
http://code.google.com/p/chromium/issues/detail?id=141425 and
http://code.google.com/p/chromium/issues/detail?id=144809, please add a:

  BUG=141425,141425

line to the changelist description so the bugs will be automatically updated
when this is committed.

[Daniel Erat, 2012-08-29 01:23:28]

On 2012/08/29 01:23:03, Daniel Erat wrote:
> Drive-by comment: If this is the cause of
> http://code.google.com/p/chromium/issues/detail?id=141425 and
> http://code.google.com/p/chromium/issues/detail?id=144809, please add a:
> 
>   BUG=141425,141425
> 
> line to the changelist description so the bugs will be automatically updated
> when this is committed.

(sorry for spam; should've read existing comments first)

[Nico, 2012-08-29 18:21:13]

ping

[bungeman-skia, 2012-08-29 19:31:16]

http://codereview.chromium.org/10871091/diff/1/skia/skia.gyp
File skia/skia.gyp (right):

http://codereview.chromium.org/10871091/diff/1/skia/skia.gyp#newcode253
skia/skia.gyp:253: [ 'OS == "android" or OS == "linux"', {
On 2012/08/28 22:28:20, Nico wrote:
> You might want to check with the android folks if they want to match linux's
> text rendering.

After checking around, it looks like android is willing to see what it looks
like, and can change this to whatever they like later.

[Nico, 2012-08-29 19:34:02]

On 2012/08/29 19:31:16, bungeman1 wrote:
> http://codereview.chromium.org/10871091/diff/1/skia/skia.gyp
> File skia/skia.gyp (right):
> 
> http://codereview.chromium.org/10871091/diff/1/skia/skia.gyp#newcode253
> skia/skia.gyp:253: [ 'OS == "android" or OS == "linux"', {
> On 2012/08/28 22:28:20, Nico wrote:
> > You might want to check with the android folks if they want to match linux's
> > text rendering.
> 
> After checking around, it looks like android is willing to see what it looks
> like, and can change this to whatever they like later.

Cool. Can you add a BUG= line to the CL description? Then this is ready to go.

[bungeman-skia, 2012-08-29 19:47:42]

> Cool. Can you add a BUG= line to the CL description? Then this is ready to go.

BUG= added. I think these are all of the relevant ones.

[Nico, 2012-08-29 20:01:47]

lgtm

[bungeman-skia, 2012-08-29 21:34:50]

Committed revision 153958.