CrOS Window cycle ui - set widget bounds with initparams to avoid extra

ui/views/widget/widget.cc

Index: ui/views/widget/widget.cc
diff --git a/ui/views/widget/widget.cc b/ui/views/widget/widget.cc
index ca92c5bd33937456f9f0fb89b183c792f92e93e8..4b3db3f38de3d8f08c9a887d23231c318d6bae06 100644
--- a/ui/views/widget/widget.cc
+++ b/ui/views/widget/widget.cc
@@ -1456,7 +1456,7 @@ void Widget::SetInitialBoundsForFramelessWindow(const gfx::Rect& bounds) {
       native_widget_->CenterWindow(size);
   } else {
     // Use the supplied initial bounds.
-    SetBoundsConstrained(bounds);
+    SetBounds(bounds);

[Evan Stade, 2016/09/13 17:26:25]

Does this make sense? An alternative would be to just remove the inset in
SetBoundsConstrained for frameless windows.

My best guess is that SetBoundsConstrained is designed to make sure normal
windows are within the work area and adds some "helpful" padding, so it might
not need to be applied to frameless windows.

[sky, 2016/09/13 19:44:02]

This does indeed worry me. Did you look at the history to see if it was added
for a specific type? I'm inclined to think we should only do it for TYPE_WINDOW.

[Evan Stade, 2016/09/13 21:59:33]

Do you mean TYPE_WINDOW_FRAMELESS? I don't really think TYPE_MENU or other
frameless types would want to be forced 10px inside the work area either. For
example, when you have a maximized browser window and you click the app menu,
the menu shouldn't be moved 10px to the left (and in fact it isn't, which is
because MenuHost can still call SetBounds() and circumvent this weird behavior
[1]). It appears that SetBoundsConstrained is only used from one place outside
of Widget: network_config_view.cc.

re: history --- for SetInitialBoundsForFramelessWindow, I suspect it was just
copy-pasted from SetInitialBounds[2]. That was added here[3].

[1]
https://cs.chromium.org/chromium/src/ui/views/controls/menu/menu_host.cc?rcl=...
[2] https://codereview.chromium.org/9250029
[3] https://codereview.chromium.org/8477019
[4] chrome/browser/chromeos/options/network_config_view.cc	

   }
 }