Quick View: Support pdf and text preview.

Quick View: Support pdf and text preview.

This CL allows user to open files which are browsable on Chrome
on Quick View. Browsable files include pdf and text.
Screencast: https://bugs.chromium.org/p/chromium/issues/detail?id=640696#c11

BUG=642713, 640696
TEST=manually tested the following:
- For pdf and text, preview with 24px of vertical margin appears.
- For image, video and audio, preview with 15% of vertical margin
appears as before.
- For .docx, 'no preview available' is shown as expected, though the
file type is 'document' which is the same as pdf.

CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Committed: https://crrev.com/885a00341b184c0daed52a7a7fb234f6f14638f6
Cr-Commit-Position: refs/heads/master@{#438744}

[oka, 2016-11-30 12:48:18]

Description was changed from

==========
Revive Quick View test.

BUG=668142
==========

to

==========
Revive Quick View test.

BUG=668142
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[oka, 2016-11-30 12:49:24]

Description was changed from

==========
Revive Quick View test.

BUG=668142
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
WIP: POC: Support pdf and text preview.

BUG=642713,640696
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[oka, 2016-12-09 10:34:38]

Description was changed from

==========
WIP: POC: Support pdf and text preview.

BUG=642713,640696
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
Quick View: Support pdf and text preview.

BUG=642713,640696
TEST=manually tested the following:
- For pdf and text, preview with 24px of horizontal margin appears.
- For image, video and audio, preview with 15% of horizontal margin
appears as before.
- For .docx, 'no preview available' is shown as expected, though the
file type is 'document' which is the same as pdf.

BUG=642713,640696
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[oka, 2016-12-09 10:35:24]

Description was changed from

==========
Quick View: Support pdf and text preview.

BUG=642713,640696
TEST=manually tested the following:
- For pdf and text, preview with 24px of horizontal margin appears.
- For image, video and audio, preview with 15% of horizontal margin
appears as before.
- For .docx, 'no preview available' is shown as expected, though the
file type is 'document' which is the same as pdf.

BUG=642713,640696
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
Quick View: Support pdf and text preview.

BUG=642713,640696
TEST=manually tested the following:
- For pdf and text, preview with 24px of horizontal margin appears.
- For image, video and audio, preview with 15% of horizontal margin
appears as before.
- For .docx, 'no preview available' is shown as expected, though the
file type is 'document' which is the same as pdf.

CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[oka, 2016-12-09 10:35:40]

oka@chromium.org changed reviewers:
+ fukino@chromium.org

[oka, 2016-12-09 10:35:41]

PTAL.

[fukino, 2016-12-09 11:15:47]

lgtm assuming that the design got approval.

[oka, 2016-12-09 11:38:38]

Description was changed from

==========
Quick View: Support pdf and text preview.

BUG=642713,640696
TEST=manually tested the following:
- For pdf and text, preview with 24px of horizontal margin appears.
- For image, video and audio, preview with 15% of horizontal margin
appears as before.
- For .docx, 'no preview available' is shown as expected, though the
file type is 'document' which is the same as pdf.

CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
Quick View: Support pdf and text preview.

BUG=642713,640696
TEST=manually tested the following:
- For pdf and text, preview with 24px of vertical margin appears.
- For image, video and audio, preview with 15% of vertical margin
appears as before.
- For .docx, 'no preview available' is shown as expected, though the
file type is 'document' which is the same as pdf.

CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[fukino, 2016-12-12 02:06:49]

On 2016/12/09 11:15:47, fukino wrote:
> lgtm assuming that the design got approval.

BTW, please write CL description between the title line and BUG= line. (Please
refer other CLs regarding the format)

[oka, 2016-12-15 03:57:01]

Description was changed from

==========
Quick View: Support pdf and text preview.

BUG=642713,640696
TEST=manually tested the following:
- For pdf and text, preview with 24px of vertical margin appears.
- For image, video and audio, preview with 15% of vertical margin
appears as before.
- For .docx, 'no preview available' is shown as expected, though the
file type is 'document' which is the same as pdf.

CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
Quick View: Support pdf and text preview.

This CL allows user to open files which are browsable on Chrome
on Quick View. Browsable files include pdf and text.
Screencast: https://bugs.chromium.org/p/chromium/issues/detail?id=640696#c11

BUG=642713,640696
TEST=manually tested the following:
- For pdf and text, preview with 24px of vertical margin appears.
- For image, video and audio, preview with 15% of vertical margin
appears as before.
- For .docx, 'no preview available' is shown as expected, though the
file type is 'document' which is the same as pdf.

CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[oka, 2016-12-15 03:57:42]

On 2016/12/12 02:06:49, fukino wrote:
> On 2016/12/09 11:15:47, fukino wrote:
> > lgtm assuming that the design got approval.
> 
> BTW, please write CL description between the title line and BUG= line. (Please
> refer other CLs regarding the format)

Updated the description, and got LGTM from sgabriel@.

[oka, 2016-12-15 03:57:48]

The CQ bit was checked by oka@chromium.org

[commit-bot: I haz the power, 2016-12-15 03:58:16]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-12-15 04:38:52]

CQ is committing da patch.

Bot data: {"patchset_id": 100001, "attempt_start_ts": 1481774268113030,
"parent_rev": "a86c7423bfc9fd13a66b61250810f3bd7025fae9", "commit_rev":
"3354028521dec80c9e0a943fdb2d8f9458bf908a"}

[commit-bot: I haz the power, 2016-12-15 04:39:18]

Description was changed from

==========
Quick View: Support pdf and text preview.

This CL allows user to open files which are browsable on Chrome
on Quick View. Browsable files include pdf and text.
Screencast: https://bugs.chromium.org/p/chromium/issues/detail?id=640696#c11

BUG=642713,640696
TEST=manually tested the following:
- For pdf and text, preview with 24px of vertical margin appears.
- For image, video and audio, preview with 15% of vertical margin
appears as before.
- For .docx, 'no preview available' is shown as expected, though the
file type is 'document' which is the same as pdf.

CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
Quick View: Support pdf and text preview.

This CL allows user to open files which are browsable on Chrome
on Quick View. Browsable files include pdf and text.
Screencast: https://bugs.chromium.org/p/chromium/issues/detail?id=640696#c11

BUG=642713,640696
TEST=manually tested the following:
- For pdf and text, preview with 24px of vertical margin appears.
- For image, video and audio, preview with 15% of vertical margin
appears as before.
- For .docx, 'no preview available' is shown as expected, though the
file type is 'document' which is the same as pdf.

CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2540863004
==========

[commit-bot: I haz the power, 2016-12-15 04:39:19]

Committed patchset #6 (id:100001)

[commit-bot: I haz the power, 2016-12-15 04:41:27]

Description was changed from

==========
Quick View: Support pdf and text preview.

This CL allows user to open files which are browsable on Chrome
on Quick View. Browsable files include pdf and text.
Screencast: https://bugs.chromium.org/p/chromium/issues/detail?id=640696#c11

BUG=642713,640696
TEST=manually tested the following:
- For pdf and text, preview with 24px of vertical margin appears.
- For image, video and audio, preview with 15% of vertical margin
appears as before.
- For .docx, 'no preview available' is shown as expected, though the
file type is 'document' which is the same as pdf.

CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2540863004
==========

to

==========
Quick View: Support pdf and text preview.

This CL allows user to open files which are browsable on Chrome
on Quick View. Browsable files include pdf and text.
Screencast: https://bugs.chromium.org/p/chromium/issues/detail?id=640696#c11

BUG=642713,640696
TEST=manually tested the following:
- For pdf and text, preview with 24px of vertical margin appears.
- For image, video and audio, preview with 15% of vertical margin
appears as before.
- For .docx, 'no preview available' is shown as expected, though the
file type is 'document' which is the same as pdf.

CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Committed: https://crrev.com/885a00341b184c0daed52a7a7fb234f6f14638f6
Cr-Commit-Position: refs/heads/master@{#438744}
==========

[commit-bot: I haz the power, 2016-12-15 04:41:29]

Patchset 6 (id:??) landed as
https://crrev.com/885a00341b184c0daed52a7a7fb234f6f14638f6
Cr-Commit-Position: refs/heads/master@{#438744}