Color picker window should not have a minimize button.

Color picker window should not have a minimize button.

Adding CanMinimize() (like CanMaximize()) function to WidgetDelegate class.
So that ColorChooserView or any other view inheriting WidgetDelegate, can
decide whether its window can show minimize button.

BUG=365768
R=sadrul@chromium.org

TEST=
1. Go to a page with <input type=color>. Here is one http://jsbin.com/yotasure/1
2. Click on the color input button to bring up the color picker window and observe
3. Color picker window should not have minimize button.

Committed: https://crrev.com/a0aab5b44df674946fbd6b5c170f433918bdb925
Cr-Commit-Position: refs/heads/master@{#295919}

[Gaja, 2014-09-11 07:10:12]

@sadrul
PTAL.

[Gaja, 2014-09-12 08:09:46]

On 2014/09/11 07:10:12, Gajendra N wrote:
> @sadrul
> PTAL.

@sadrul
PTAL.

[Gaja, 2014-09-13 10:27:48]

On 2014/09/12 08:09:46, Gajendra N wrote:
> On 2014/09/11 07:10:12, Gajendra N wrote:
> > @sadrul
> > PTAL.
> 
> @sadrul
> PTAL.

sadrul, PTAL. Thanks!

[Gaja, 2014-09-15 11:19:53]

On 2014/09/13 10:27:48, Gajendra N wrote:
> On 2014/09/12 08:09:46, Gajendra N wrote:
> > On 2014/09/11 07:10:12, Gajendra N wrote:
> > > @sadrul
> > > PTAL.
> > 
> > @sadrul
> > PTAL.
> 
> sadrul, PTAL. Thanks!

@sadrul
PTAL.

[sadrul, 2014-09-15 14:43:53]

The change looks good. I have asked for confirmation whether we want this on all
platforms. In the meantime, you should add a test.

[Gaja, 2014-09-16 03:49:41]

On 2014/09/15 14:43:53, sadrul wrote:
> The change looks good. I have asked for confirmation whether we want this on
all
> platforms. In the meantime, you should add a test.

Thanks for the reply. I have added unit test for minimize button visibility.
Changes in Patch Set 2. Please review.

[Gaja, 2014-09-16 17:49:03]

On 2014/09/16 03:49:41, Gajendra N wrote:
> On 2014/09/15 14:43:53, sadrul wrote:
> > The change looks good. I have asked for confirmation whether we want this on
> all
> > platforms. In the meantime, you should add a test.
> 
> Thanks for the reply. I have added unit test for minimize button visibility.
> Changes in Patch Set 2. Please review.

PTAL.

[Gaja, 2014-09-19 05:49:51]

On 2014/09/16 17:49:03, Gajendra N wrote:
> On 2014/09/16 03:49:41, Gajendra N wrote:
> > On 2014/09/15 14:43:53, sadrul wrote:
> > > The change looks good. I have asked for confirmation whether we want this
on
> > all
> > > platforms. In the meantime, you should add a test.
> > 
> > Thanks for the reply. I have added unit test for minimize button visibility.
> > Changes in Patch Set 2. Please review.
> 
> PTAL.

@sadrul, Please take a look.

[sadrul, 2014-09-22 00:35:22]

LGTM

+jackhou@, since he is making the same change in WidgetDelegate at
https://codereview.chromium.org/583603004/

[jackhou1, 2014-09-22 01:46:57]

On 2014/09/22 00:35:22, sadrul wrote:
> LGTM
> 
> +jackhou@, since he is making the same change in WidgetDelegate at
> https://codereview.chromium.org/583603004/

Oo, thanks for pointing this out. I don't think this clashes with my CL, so
lgtm.

[Gaja, 2014-09-22 02:52:29]

On 2014/09/22 01:46:57, jackhou1 wrote:
> On 2014/09/22 00:35:22, sadrul wrote:
> > LGTM
> > 
> > +jackhou@, since he is making the same change in WidgetDelegate at
> > https://codereview.chromium.org/583603004/
> 
> Oo, thanks for pointing this out. I don't think this clashes with my CL, so
> lgtm.

@sadrul, jackhou
Thanks!

[Gaja, 2014-09-22 02:52:33]

The CQ bit was checked by gajendra.n@samsung.com

[commit-bot: I haz the power, 2014-09-22 02:53:46]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/567463002/20001

[commit-bot: I haz the power, 2014-09-22 03:50:26]

Committed patchset #2 (id:20001) as c392b3af1a94b78c3d569810363b6f970087363e

[commit-bot: I haz the power, 2014-09-22 03:51:16]

Patchset 2 (id:??) landed as
https://crrev.com/a0aab5b44df674946fbd6b5c170f433918bdb925
Cr-Commit-Position: refs/heads/master@{#295919}