Always show distillation option in print preview

Always show distillation option in print preview

The accuracy of distillability test is enough for prompting reader mode,
but not for print preview. The user experience would be better if the
user can override whether to distill.

BUG=490809
Committed: https://crrev.com/89a9e430bdca57cf6543d431e001b0902abd29f1
Cr-Commit-Position: refs/heads/master@{#357476}

[wychen, 2015-11-01 19:31:16]

Description was changed from

==========
Always show distillation option in print preview

BUG=
==========

to

==========
Always show distillation option in print preview

The accuracy of distillability test is enough for prompting reader mode, but not
for print preview. The user experience would be better if the user can override
whether to distill.

BUG=
==========

[wychen, 2015-11-01 19:31:16]

wychen@chromium.org changed reviewers:
+ arjunpatel@hp.com, vitalybuka@chromium.org

[wychen, 2015-11-01 19:31:32]

Description was changed from

==========
Always show distillation option in print preview

The accuracy of distillability test is enough for prompting reader mode, but not
for print preview. The user experience would be better if the user can override
whether to distill.

BUG=
==========

to

==========
Always show distillation option in print preview

The accuracy of distillability test is enough for prompting reader mode,
but not for print preview. The user experience would be better if the
user can override whether to distill.

BUG=
==========

[wychen, 2015-11-01 19:32:41]

PTAL

[arjunpatel, 2015-11-02 22:09:35]

There might be overlap between this and and an existing CL
https://codereview.chromium.org/1419063002. PTAL

[wychen, 2015-11-02 23:01:42]

On 2015/11/02 22:09:35, arjunpatel wrote:
> There might be overlap between this and and an existing CL
> https://codereview.chromium.org/1419063002. PTAL

Thanks. It seems that CL already has everything I wanted.

[wychen, 2015-11-02 23:19:26]

On 2015/11/02 23:01:42, wychen wrote:
> On 2015/11/02 22:09:35, arjunpatel wrote:
> > There might be overlap between this and and an existing CL
> > https://codereview.chromium.org/1419063002. PTAL
> 
> Thanks. It seems that CL already has everything I wanted.

By the way, this CL has some other CLs depending on it. Just in
case if your CL is not landed before 11/4, would you mind if I
land this first? Thanks!

[wychen, 2015-11-02 23:19:44]

Description was changed from

==========
Always show distillation option in print preview

The accuracy of distillability test is enough for prompting reader mode,
but not for print preview. The user experience would be better if the
user can override whether to distill.

BUG=
==========

to

==========
Always show distillation option in print preview

The accuracy of distillability test is enough for prompting reader mode,
but not for print preview. The user experience would be better if the
user can override whether to distill.

BUG=490809
==========

[Vitaly Buka (NO REVIEWS), 2015-11-02 23:31:11]

lgtm

[Vitaly Buka (NO REVIEWS), 2015-11-02 23:31:46]

The CQ bit was checked by vitalybuka@chromium.org

[commit-bot: I haz the power, 2015-11-02 23:33:55]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1419363004/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1419363004/1

[wychen, 2015-11-02 23:58:07]

On 2015/11/02 23:31:11, Vitaly Buka wrote:
> lgtm

Thanks for your prompt reply!

[commit-bot: I haz the power, 2015-11-03 00:14:38]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-11-03 00:15:33]

Patchset 1 (id:??) landed as
https://crrev.com/89a9e430bdca57cf6543d431e001b0902abd29f1
Cr-Commit-Position: refs/heads/master@{#357476}