Handle titlebar text updates properly

ui/views/widget/widget.cc

Index: ui/views/widget/widget.cc
diff --git a/ui/views/widget/widget.cc b/ui/views/widget/widget.cc
index a4d85a95b4ce2b2a5b356bfd9faf12e7f9604b0d..0083e25c74954c07f9a9529a3c5e85f4718a695f 100644
--- a/ui/views/widget/widget.cc
+++ b/ui/views/widget/widget.cc
@@ -737,8 +737,9 @@ void Widget::UpdateWindowTitle() {
     return;
 
   // If the non-client view is rendering its own title, it'll need to relayout
-  // now.
+  // now and to get a paint update later on.
   non_client_view_->Layout();
+  non_client_view_->UpdateWindowTitle();

[sky, 2012/09/28 22:47:12]

Don't you need to do all this after updating the native_widget_'s title? If you
call this before updating the native title then anyone that asks for the updated
window title won't get the right thing.

Additionally you should Layout() after updating the window title, since that may
cause things to need a bigger size.

[Mr4D (OOO till 08-26), 2012/09/29 00:44:25]

Okay. Changed. Not sure if this might cause problems though since it looked like
that there was a reason for the order.

But tested it in various situations and it worked.

 
   // Update the native frame's text. We do this regardless of whether or not
   // the native frame is being used, since this also updates the taskbar, etc.