Add key modifier to minimize/restore button click to minimize/restore-all panels for Windows.

Add key modifier to minimize/restore button click to minimize/restore-all panels for Windows.

Changed to use the same technique to detect key modifier for titlebar click.

BUG=123176
TEST=Existing


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=134425

[jianli, 2012-04-27 22:32:10]

http://codereview.chromium.org/10187021/diff/1/chrome/browser/ui/panels/panel...
File chrome/browser/ui/panels/panel_browser_view.cc (right):

http://codereview.chromium.org/10187021/diff/1/chrome/browser/ui/panels/panel...
chrome/browser/ui/panels/panel_browser_view.cc:444:
views::View::ConvertPointToScreen(this, &mouse_location);
|event| is for PanelBrowserFrameView, not PanelBrowserView. Thus
|event.location| is in PanelBrowserFrameView's coordinate system. I think we
should try not to pass event to PanelBrowserView.

[jennb, 2012-04-27 23:27:54]

http://codereview.chromium.org/10187021/diff/1/chrome/browser/ui/panels/panel...
File chrome/browser/ui/panels/panel_browser_view.cc (right):

http://codereview.chromium.org/10187021/diff/1/chrome/browser/ui/panels/panel...
chrome/browser/ui/panels/panel_browser_view.cc:444:
views::View::ConvertPointToScreen(this, &mouse_location);
On 2012/04/27 22:32:10, jianli wrote:
> |event| is for PanelBrowserFrameView, not PanelBrowserView. Thus
> |event.location| is in PanelBrowserFrameView's coordinate system. I think we
> should try not to pass event to PanelBrowserView.

Changed to pass modifier around rather than event. diff against original is
easier to read than the diff against the previous revision as I was mostly
undoing changes I made in the 1st revision.

[jianli, 2012-04-28 00:06:33]

lgtm

[commit-bot: I haz the power, 2012-04-28 00:11:47]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/jennb@chromium.org/10187021/2002

[commit-bot: I haz the power, 2012-04-28 02:16:38]

Change committed as 134425