Turned on PbufferImageTransportSurface as default image transport for vista and win 7 (not XP).

Turned on PbufferImageTransportSurface as default image transport for vista and win 7 (not XP).

Also implemented PbufferImageTransportSurface::SetVisible by making the underlying surface 1x1 when it is not visible.

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=113761

[jonathan.backer, 2011-12-08 16:07:02]

LGTM

I'm assuming you've thoroughly tested (I haven't on my end).

http://codereview.chromium.org/8884001/diff/1/content/public/common/content_s...
File content/public/common/content_switches.h (right):

http://codereview.chromium.org/8884001/diff/1/content/public/common/content_s...
content/public/common/content_switches.h:49: extern const char
kDisableIndexedDatabase[];
Potential merge conflict?

[apatrick_chromium, 2011-12-08 21:20:06]

Done. I have tested it. Just waiting for you to review
http://codereview.chromium.org/8869007/ :)

http://codereview.chromium.org/8884001/diff/1/content/public/common/content_s...
File content/public/common/content_switches.h (right):

http://codereview.chromium.org/8884001/diff/1/content/public/common/content_s...
content/public/common/content_switches.h:49: extern const char
kDisableIndexedDatabase[];
On 2011/12/08 16:07:03, jonathan.backer wrote:
> Potential merge conflict?

Yep.

[jonathan.backer, 2011-12-08 21:26:39]

On 2011/12/08 21:20:06, apatrick_chromium wrote:
> Done. I have tested it. Just waiting for you to review
> http://codereview.chromium.org/8869007/ :)

Thanks for the ping. I looked at it earlier. I must have forgot to press the
magic lgtm button. Will do it now.

[apatrick_chromium, 2011-12-09 00:39:38]

+jam for content/public/common

[jam, 2011-12-09 01:57:11]

lgtm

btw there are now more owners in content/public :)
http://src.chromium.org/viewvc/chrome/trunk/src/content/public/OWNERS

[commit-bot: I haz the power, 2011-12-09 03:29:18]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/apatrick@chromium.org/8884001/6001

[commit-bot: I haz the power, 2011-12-09 04:44:31]

Change committed as 113761