Fix the positioning of the TryChromeDialog when the taskbar is hidden.

chrome/browser/first_run/try_chrome_dialog_view.h

Index: chrome/browser/first_run/try_chrome_dialog_view.h
diff --git a/chrome/browser/first_run/try_chrome_dialog_view.h b/chrome/browser/first_run/try_chrome_dialog_view.h
index 1daa77f5b09642561913ac3deb94e20f551cc192..deba38fde14805b12492b2b28fe63a8c6903cc15 100644
--- a/chrome/browser/first_run/try_chrome_dialog_view.h
+++ b/chrome/browser/first_run/try_chrome_dialog_view.h
@@ -15,6 +15,7 @@
 
 namespace gfx {

[huangs, 2017/03/13 15:29:28]

Probably should include; please see below.

[grt (UTC plus 2), 2017/03/14 09:24:17]

Done.

 class Rect;
+class Size;
 }
 
 namespace views {
@@ -93,8 +94,8 @@ class TryChromeDialogView : public views::ButtonListener,
   // it has the following properties: a) is visible and b) is attached to the
   // bottom of the working area. For LTR machines it returns a left side
   // rectangle and for RTL it returns a right side rectangle so that the dialog
-  // does not compete with the standar place of the start menu.
-  gfx::Rect ComputeWindowPosition(int width, int height, bool is_RTL);
+  // does not compete with the standard place of the start menu.
+  gfx::Rect ComputeWindowPosition(gfx::Size size, bool is_RTL);

[huangs, 2017/03/13 15:29:28]

gfx::Rect and gfx::Size are returned / passed by value. This works probably
because the .h file got included somewhere along the line; and predeclaring them
is redundant.  Perhaps directly include?

#include "ui/gfx/geometry/rect.h"
#include "ui/gfx/geometry/size.h"

[grt (UTC plus 2), 2017/03/14 09:24:17]

Done.

 
   // Create a windows region that looks like a toast of width |w| and height
   // |h|. This is best effort, so we don't care much if the operation fails.