Make the minimize state work when creating a chrome window in CrOS

chrome/browser/extensions/api/tabs/tabs_api.cc

Index: chrome/browser/extensions/api/tabs/tabs_api.cc
diff --git a/chrome/browser/extensions/api/tabs/tabs_api.cc b/chrome/browser/extensions/api/tabs/tabs_api.cc
index 2224bcfd7e67913dd3840938cad7df5ba147e41e..87ae5c7b2528a89e106eea9c20f9fffe47582f8a 100644
--- a/chrome/browser/extensions/api/tabs/tabs_api.cc
+++ b/chrome/browser/extensions/api/tabs/tabs_api.cc
@@ -659,13 +659,11 @@ bool WindowsCreateFunction::RunSync() {
 
   WindowController* controller = new_window->extension_window_controller();
 
-#if defined(OS_LINUX) || defined(OS_CHROMEOS)
+#if defined(OS_LINUX) && !defined(OS_CHROMEOS)
   // On Desktop Linux, window managers may ignore hints until the X11 window is
   // mapped, which happens in the blocking call to Show() above.
   // DesktopWindowTreeHostX11 currently only checks for an attempt to maximize
   // once mapped, but not minimize or fullscreen.

[tapted, 2015/05/26 23:25:30]

I think minimize was added to DWTHX11's ShowWindowWithState in
http://crrev.com/329764 so the Minimize() below should now be redundant on
Desktop Linux as well.

I.e. this entire #if block should be able to be deleted now.

Likely the one below for fullscreen can go too, since handling for that was also
added in r329764.

[joone, 2015/05/27 17:50:21]

Can I create a separated CL for the Desktop Linux case? Because the below code
that handles ui::SHOW_STATE_FULLSCREEN can be removed together.

-  // For ChromeOS, manually Minimize(). Because minimzied window is not
-  // considered to create new window. See http://crbug.com/473228.
   if (create_params.initial_show_state == ui::SHOW_STATE_MINIMIZED)
     new_window->window()->Minimize();
 #endif