Notification balloons don't want the WS_EX_LAYOUTRTL style flag, since the di...

views/widget/widget.h

Index: views/widget/widget.h
===================================================================
--- views/widget/widget.h	(revision 48671)
+++ views/widget/widget.h	(working copy)
@@ -64,10 +64,20 @@
     NotDeleteOnDestroy
   };
 
+  enum MirroringParam {
+    MirrorOriginInRTL,
+    DontMirrorOriginInRTL
+  };
+
   // Creates a transient popup widget specific to the current platform.
+  // If |mirror_in_rtl| is set to MirrorOriginInRTL, the contents of the
+  // popup will be mirrored if the current locale is RTL.  You should use

[jeremy, 2010/06/10 20:58:58]

"the contents of the popup will be mirrored" -> The view origin will be
mirrored.

+  // DontMirrorOriginInRTL if you are aleady handling the RTL layout within
+  // the widget.
   static Widget* CreatePopupWidget(TransparencyParam transparent,
                                    EventsParam accept_events,
-                                   DeleteParam delete_on_destroy);
+                                   DeleteParam delete_on_destroy,
+                                   MirroringParam mirror_in_rtl);

[idana, 2010/06/16 13:46:16]

Since almost all the call sites use MirrorOriginInRTL (the default behavior), it
makes more sense to overload CreatePopupWidget such that only BalloonView uses
the version which takes the mirroring config.

 
   // Returns the root view for |native_window|. If |native_window| does not have
   // a rootview, this recurses through all of |native_window|'s children until