Support MINIMIZED, MAXIMIZED, FULLSCREEN state when showing a chrome window.

Support MINIMIZED, MAXIMIZED, FULLSCREEN state when showing a chrome window.

Some of ui::WindowShowState were not supported in Linux so this CL implemented
SHOW_STATE_MINIMIZED, SHOW_STATE_MAXIMIZED, and SHOW_STATE_FULLSCREEN state.

BUG=473228
Committed: https://crrev.com/3a3f2f572a3a96957a6180056e7cbd30f551200d
Cr-Commit-Position: refs/heads/master@{#329764}

[joone, 2015-05-13 23:46:26]

joone.hur@intel.com changed reviewers:
+ erg@chromium.org

[joone, 2015-05-13 23:46:27]

Please take a look at this patch.

[Elliot Glaysher, 2015-05-13 23:56:16]

lgtm

[joone, 2015-05-14 00:00:38]

The CQ bit was checked by joone.hur@intel.com

[commit-bot: I haz the power, 2015-05-14 00:02:58]

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

[Peter Kasting, 2015-05-14 00:10:13]

pkasting@chromium.org changed reviewers:
+ pkasting@chromium.org

[Peter Kasting, 2015-05-14 00:10:13]

https://codereview.chromium.org/1125383008/diff/1/ui/views/widget/desktop_aur...
File ui/views/widget/desktop_aura/desktop_window_tree_host_x11.cc (right):

https://codereview.chromium.org/1125383008/diff/1/ui/views/widget/desktop_aur...
ui/views/widget/desktop_aura/desktop_window_tree_host_x11.cc:382:
SetFullscreen(TRUE);
Shouldn't this be "true" instead of "TRUE"?

[commit-bot: I haz the power, 2015-05-14 00:41:45]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-05-14 00:42:34]

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

[joone, 2015-05-14 18:18:57]

https://codereview.chromium.org/1125383008/diff/1/ui/views/widget/desktop_aur...
File ui/views/widget/desktop_aura/desktop_window_tree_host_x11.cc (right):

https://codereview.chromium.org/1125383008/diff/1/ui/views/widget/desktop_aur...
ui/views/widget/desktop_aura/desktop_window_tree_host_x11.cc:382:
SetFullscreen(TRUE);
On 2015/05/14 00:10:13, Peter Kasting wrote:
> Shouldn't this be "true" instead of "TRUE"?

Yes, it should be true. I'll fix it.

[joone, 2015-05-14 21:38:43]

Please land this change.

[joone, 2015-05-14 21:40:02]

Patchset #2 (id:20001) has been deleted

[Peter Kasting, 2015-05-14 21:41:02]

On 2015/05/14 21:38:43, joone wrote:
> Please land this change.

If a CL has already been landed and the Rietveld issue closed, the best thing to
do to land followon modifications is to upload them separately as a new CL on a
different issue.

I'm going to delete the patch here so future spelunkers will see the one you
actually landed on this review.

[Peter Kasting, 2015-05-14 21:41:32]

On 2015/05/14 21:41:02, Peter Kasting wrote:
> I'm going to delete the patch here so future spelunkers will see the one you
> actually landed on this review.

Hmm, someone beat me to it it seems.

[joone, 2015-05-14 21:42:23]

On 2015/05/14 21:38:43, joone wrote:
> Please land this change.

https://codereview.chromium.org/1124193005

[joone, 2015-05-14 21:44:54]

On 2015/05/14 21:41:32, Peter Kasting wrote:
> On 2015/05/14 21:41:02, Peter Kasting wrote:
> > I'm going to delete the patch here so future spelunkers will see the one you
> > actually landed on this review.
> 
> Hmm, someone beat me to it it seems.

Sorry. I deleted it.