Review Request: fix issue 5724, 5738 -- wrong position of system menu

chrome/views/window/window_win.cc

Index: chrome/views/window/window_win.cc
===================================================================
--- chrome/views/window/window_win.cc	(revision 12538)
+++ chrome/views/window/window_win.cc	(working copy)
@@ -415,9 +415,11 @@
   // to get a system menu. We also can't take the return value of this
   // function. We need to call it *again* to get a valid HMENU.
   //::GetSystemMenu(GetNativeView(), TRUE);
+  UINT flags = TPM_LEFTBUTTON | TPM_RIGHTBUTTON | TPM_RETURNCMD;
+  if (l10n_util::GetTextDirection() == l10n_util::RIGHT_TO_LEFT)
+    flags |= TPM_RIGHTALIGN;
   HMENU system_menu = ::GetSystemMenu(GetNativeView(), FALSE);
-  int id = ::TrackPopupMenu(system_menu,
-                            TPM_LEFTBUTTON | TPM_RIGHTBUTTON | TPM_RETURNCMD,
+  int id = ::TrackPopupMenu(system_menu, flags,
                             point.x(), point.y(), 0, GetNativeView(), NULL);
   ExecuteSystemMenuCommand(id);
 }
@@ -493,7 +495,7 @@
   force_hidden_ = true;
   ::ShowWindow(GetNativeView(), SW_RESTORE);
   force_hidden_ = false;
-  
+
   // We respond to this in response to WM_DWMCOMPOSITIONCHANGED since that is
   // the only thing we care about - we don't actually respond to WM_THEMECHANGED
   // messages.