Migrate to Styx thumbnails in Files App.

Migrate to Styx thumbnails in Files App.

Styx provides thumbnails using a persistent urls, in contrast to the previous urls which used to get stale with time. This patch moves to the persisten urls, to there is no need to handle stale urls anymore.

TEST=browser_tests
BUG=254025
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=226362

[mtomasz, 2013-10-01 05:53:07]

@yoshiki: PTAL at *.js.
@kinaba: PTAL at *.cc.

The thumbnailUrl is created in private_api_drive.cc, however I was also thinking
about creating it in Javascript, which would look cleaner. However, we would
need to pass the resource id instead of the thumbnail url to be able to generate
it.

Therefore, I chose to go this way for compatibility. We can clean this up, in
another CL. WDYT?

[kinaba, 2013-10-01 08:48:29]

On 2013/10/01 05:53:07, mtomasz wrote:
> @yoshiki: PTAL at *.js.
> @kinaba: PTAL at *.cc.
> 
> The thumbnailUrl is created in private_api_drive.cc, however I was also
thinking
> about creating it in Javascript, which would look cleaner. However, we would
> need to pass the resource id instead of the thumbnail url to be able to
generate
> it.
> 
> Therefore, I chose to go this way for compatibility. We can clean this up, in
> another CL. WDYT?

Sounds good. .cc lgtm.

[yoshiki, 2013-10-01 08:51:18]

.js lgtm with nit

https://codereview.chromium.org/25458002/diff/3001/chrome/browser/resources/i...
File chrome/browser/resources/image_loader/request.js (right):

https://codereview.chromium.org/25458002/diff/3001/chrome/browser/resources/i...
chrome/browser/resources/image_loader/request.js:264: token, url,
onInnerSuccess, onInnerFailure);
nit: indent

[commit-bot: I haz the power, 2013-10-01 09:22:49]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/mtomasz@chromium.org/25458002/9001

[commit-bot: I haz the power, 2013-10-01 11:29:12]

Step "update" is always a major failure.
Look at the try server FAQ for more details.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_chro...

[commit-bot: I haz the power, 2013-10-01 23:18:44]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/mtomasz@chromium.org/25458002/9001

[commit-bot: I haz the power, 2013-10-02 00:28:21]

Change committed as 226362