Aura Task Manager: Remember the dialog size before closing task manager.

Aura Task Manager: Remember the dialog size before closing task manager.


BUG=105117
TEST=None


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

[NaveenBobbili (Motorola), 2011-12-07 07:54:28]

This patch is specific to Aura framework. The changes for linux are already
committed as 113198. Please review.

[yoshiki, 2011-12-07 08:02:50]

Thank you for the nice patch! LGTM with nit.

http://codereview.chromium.org/8835005/diff/1/chrome/browser/ui/views/html_di...
File chrome/browser/ui/views/html_dialog_view.cc (right):

http://codereview.chromium.org/8835005/diff/1/chrome/browser/ui/views/html_di...
chrome/browser/ui/views/html_dialog_view.cc:182: gfx::Rect dialog_bounds =
GetWidget()->GetClientAreaScreenBounds();
nit: Please add "const" modifier.

[NaveenBobbili (Motorola), 2011-12-07 08:14:59]

Done. Please take a look now.

http://codereview.chromium.org/8835005/diff/1/chrome/browser/ui/views/html_di...
File chrome/browser/ui/views/html_dialog_view.cc (right):

http://codereview.chromium.org/8835005/diff/1/chrome/browser/ui/views/html_di...
chrome/browser/ui/views/html_dialog_view.cc:182: gfx::Rect dialog_bounds =
GetWidget()->GetClientAreaScreenBounds();
On 2011/12/07 08:02:50, yoshiki wrote:
> nit: Please add "const" modifier.

Done.

[James Hawkins, 2011-12-07 17:30:51]

LGTM, though I don't think you need my approval for this change.

[commit-bot: I haz the power, 2011-12-08 02:14:28]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/qghc36@motorola.com/8835005/4001

[NaveenBobbili (Motorola), 2011-12-08 02:14:41]

On 2011/12/07 17:30:51, James Hawkins wrote:
> LGTM, though I don't think you need my approval for this change.

Thanks.

[commit-bot: I haz the power, 2011-12-08 04:23:05]

Change committed as 113560