Show open task button only when applicable.

Show open task button only when applicable.

In Quick View, the open task button should be shown only when
1. The dialog is full page mode, and
2. There is a task to open the file.
In this CL, hasTask property is added to files-quick-view element
and quick view controller sets it true only if 1 and 2 are satisfied.

BUG=658448
TEST=manually checked the following cases:
file type | volume   | has open button 
--------------------------------------
image     | Download | yes
image     | Drive    | yes
folder    | Download | no
folder    | Drive    | no

CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2634053002
Cr-Commit-Position: refs/heads/master@{#444011}
Committed: https://chromium.googlesource.com/chromium/src/+/a9273bfae3e059a8b3d7448db387b3e8b3d7ff2d

[oka, 2017-01-16 10:24:32]

Description was changed from

==========
Show open task button only when applicable.

In Quick View, the open task button should be shown only when
1. The dialog is full page mode, and
2. There is a task to open the file.

BUG=658448
TEST=manually.
==========

to

==========
Show open task button only when applicable.

In Quick View, the open task button should be shown only when
1. The dialog is full page mode, and
2. There is a task to open the file.

BUG=658448
TEST=manually.
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[oka, 2017-01-16 10:24:38]

The CQ bit was checked by oka@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-01-16 10:24:43]

Dry run: 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, 2017-01-16 10:35:47]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-01-16 10:35:48]

Dry run: Try jobs failed on following builders:
  closure_compilation on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/closure_compila...)

[oka, 2017-01-16 12:02:29]

.

[oka, 2017-01-16 12:02:42]

The CQ bit was checked by oka@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-01-16 12:02:49]

Dry run: 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, 2017-01-16 12:26:49]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-01-16 12:26:49]

Dry run: This issue passed the CQ dry run.

[oka, 2017-01-16 12:41:42]

Description was changed from

==========
Show open task button only when applicable.

In Quick View, the open task button should be shown only when
1. The dialog is full page mode, and
2. There is a task to open the file.

BUG=658448
TEST=manually.
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
Show open task button only when applicable.

In Quick View, the open task button should be shown only when
1. The dialog is full page mode, and
2. There is a task to open the file.
In this CL, hasTask property is added to files-quick-view element
and quick view controller sets it true only if 1 and 2 are satisfied.

BUG=658448
TEST=manually checked:
file type | volume   | has open button 
image     | Download | yes
image     | Drive    | yes
folder    | Download | no
folder    | Drive    | no

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

[oka, 2017-01-16 12:42:03]

Description was changed from

==========
Show open task button only when applicable.

In Quick View, the open task button should be shown only when
1. The dialog is full page mode, and
2. There is a task to open the file.
In this CL, hasTask property is added to files-quick-view element
and quick view controller sets it true only if 1 and 2 are satisfied.

BUG=658448
TEST=manually checked:
file type | volume   | has open button 
image     | Download | yes
image     | Drive    | yes
folder    | Download | no
folder    | Drive    | no

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

to

==========
Show open task button only when applicable.

In Quick View, the open task button should be shown only when
1. The dialog is full page mode, and
2. There is a task to open the file.
In this CL, hasTask property is added to files-quick-view element
and quick view controller sets it true only if 1 and 2 are satisfied.

BUG=658448
TEST=manually checked the following cases:
file type | volume   | has open button 
image     | Download | yes
image     | Drive    | yes
folder    | Download | no
folder    | Drive    | no

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

[oka, 2017-01-16 12:42:18]

Description was changed from

==========
Show open task button only when applicable.

In Quick View, the open task button should be shown only when
1. The dialog is full page mode, and
2. There is a task to open the file.
In this CL, hasTask property is added to files-quick-view element
and quick view controller sets it true only if 1 and 2 are satisfied.

BUG=658448
TEST=manually checked the following cases:
file type | volume   | has open button 
image     | Download | yes
image     | Drive    | yes
folder    | Download | no
folder    | Drive    | no

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

to

==========
Show open task button only when applicable.

In Quick View, the open task button should be shown only when
1. The dialog is full page mode, and
2. There is a task to open the file.
In this CL, hasTask property is added to files-quick-view element
and quick view controller sets it true only if 1 and 2 are satisfied.

BUG=658448
TEST=manually checked the following cases:
file type | volume   | has open button 
--------------------------------------
image     | Download | yes
image     | Drive    | yes
folder    | Download | no
folder    | Drive    | no

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

[oka, 2017-01-16 12:47:50]

.

[oka, 2017-01-16 12:49:57]

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

[oka, 2017-01-16 12:49:58]

PTAL.

[oka, 2017-01-16 12:52:38]

Remove dead code

[oka, 2017-01-16 13:17:23]

Rebased.

[fukino, 2017-01-17 06:48:01]

lgtm with a nit.

[oka, 2017-01-17 08:51:28]

Rebase.

[oka, 2017-01-17 08:55:01]

On 2017/01/17 06:48:01, fukino wrote:
> lgtm with a nit.

What is the nit?

[oka, 2017-01-17 09:03:25]

The CQ bit was checked by oka@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-01-17 09:03:33]

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

[fukino, 2017-01-17 09:24:44]

On 2017/01/17 08:55:01, oka wrote:
> On 2017/01/17 06:48:01, fukino wrote:
> > lgtm with a nit.
> 
> What is the nit?

Never mind.
I confused with a comment of another CL.

[oka, 2017-01-17 09:25:32]

The CQ bit was unchecked by oka@chromium.org

[oka, 2017-01-17 09:25:32]

The CQ bit was checked by oka@chromium.org

[oka, 2017-01-17 09:25:32]

The patchset sent to the CQ was uploaded after l-g-t-m from fukino@chromium.org
Link to the patchset: https://codereview.chromium.org/2634053002/#ps100001
(title: "Rebase.")

[commit-bot: I haz the power, 2017-01-17 09:25:45]

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

[oka, 2017-01-17 09:25:47]

The CQ bit was checked by oka@chromium.org

[commit-bot: I haz the power, 2017-01-17 09:25:55]

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

[oka, 2017-01-17 09:25:56]

On 2017/01/17 09:24:44, fukino wrote:
> On 2017/01/17 08:55:01, oka wrote:
> > On 2017/01/17 06:48:01, fukino wrote:
> > > lgtm with a nit.
> > 
> > What is the nit?
> 
> Never mind.
> I confused with a comment of another CL.

Thank you. Set the commit bit.

[commit-bot: I haz the power, 2017-01-17 09:30:58]

CQ is committing da patch.

Bot data: {"patchset_id": 100001, "attempt_start_ts": 1484645147254930,
"parent_rev": "5fd4034984b118b932a891b849320c147f47e702", "commit_rev":
"a9273bfae3e059a8b3d7448db387b3e8b3d7ff2d"}

[commit-bot: I haz the power, 2017-01-17 09:31:27]

Description was changed from

==========
Show open task button only when applicable.

In Quick View, the open task button should be shown only when
1. The dialog is full page mode, and
2. There is a task to open the file.
In this CL, hasTask property is added to files-quick-view element
and quick view controller sets it true only if 1 and 2 are satisfied.

BUG=658448
TEST=manually checked the following cases:
file type | volume   | has open button 
--------------------------------------
image     | Download | yes
image     | Drive    | yes
folder    | Download | no
folder    | Drive    | no

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

to

==========
Show open task button only when applicable.

In Quick View, the open task button should be shown only when
1. The dialog is full page mode, and
2. There is a task to open the file.
In this CL, hasTask property is added to files-quick-view element
and quick view controller sets it true only if 1 and 2 are satisfied.

BUG=658448
TEST=manually checked the following cases:
file type | volume   | has open button 
--------------------------------------
image     | Download | yes
image     | Drive    | yes
folder    | Download | no
folder    | Drive    | no

CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2634053002
Cr-Commit-Position: refs/heads/master@{#444011}
Committed:
https://chromium.googlesource.com/chromium/src/+/a9273bfae3e059a8b3d7448db387...
==========

[commit-bot: I haz the power, 2017-01-17 09:31:28]

Committed patchset #6 (id:100001) as
https://chromium.googlesource.com/chromium/src/+/a9273bfae3e059a8b3d7448db387...