Fix acquisition of thumbnail images in Linux

Fix acquisition of thumbnail images on Linux

Contributed by jwillcox@litl.com    

BUG=48066
TEST=Load a page from new tab page, then ensure the thumbnail is updated
by viewing the new tab page again

[scherkus (not reviewing), 2010-07-13 21:23:17]

The one thing I'm confused about is the CL description vs. the change.

This security check bug was actually preventing thumbnail images from working?

http://codereview.chromium.org/2904010/diff/1/2
File chrome/browser/renderer_host/backing_store_x.cc (right):

http://codereview.chromium.org/2904010/diff/1/2#newcode329
chrome/browser/renderer_host/backing_store_x.cc:329:
(std::numeric_limits<size_t>::max() / image->bytes_per_line) <
ahh looks like this was wrong :)

could we flip the condition so it reads better?

[jwillcox, 2010-07-15 15:31:04]

http://codereview.chromium.org/2904010/diff/1/2
File chrome/browser/renderer_host/backing_store_x.cc (right):

http://codereview.chromium.org/2904010/diff/1/2#newcode329
chrome/browser/renderer_host/backing_store_x.cc:329:
(std::numeric_limits<size_t>::max() / image->bytes_per_line) <
On 2010/07/13 21:23:19, scherkus wrote:
> ahh looks like this was wrong :)
> 
> could we flip the condition so it reads better?

Done.

[jwillcox, 2010-07-15 19:21:07]

BTW, I don't have commit access, so if this is ok now could you please commit it
for me? Thanks.

[scherkus (not reviewing), 2010-07-26 20:36:58]

Whoops sorry for not recognizing this sooner!

Could we get your full name added to AUTHORS?

[jwillcox, 2010-07-26 20:56:03]

On 2010/07/26 20:36:58, scherkus wrote:
> Whoops sorry for not recognizing this sooner!

No worries :)

> 
> Could we get your full name added to AUTHORS?

Done.

[scherkus (not reviewing), 2010-07-27 00:21:57]

Committed as r53721.