Allow multiple base::MessagePumpForUI instances to be created simultanenously on Windows.

base/win/message_window.cc

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
-#include "remoting/host/win/message_window.h"
+#include "base/win/message_window.h"
 
 #include "base/logging.h"
 #include "base/process_util.h"
 #include "base/stringprintf.h"
 #include "base/utf_string_conversions.h"
 #include "base/win/wrapped_window_proc.h"
 
-const char kClassNameFormat[] = "Chromoting_MessageWindow_%p";
+const char kClassNameFormat[] = "Chrome_MessageWindow_%p";
 
-namespace remoting {
+namespace base {
 namespace win {
 
 MessageWindow::MessageWindow()
     : atom_(0),
       instance_(NULL),
       window_(NULL) {
   class_name_ = base::StringPrintf(kClassNameFormat, this);

[darin (slow to review), 2013/05/23 00:07:14]

can't the address of this be reused?  hence my suggestion of using an atomic
sequence number.

[alexeypa (please no reviews), 2013/05/23 17:45:54]

The window class is unregistered in the MessageWindow's destructor. By the time
the address is reused the window class will be long gone.

   instance_ = base::GetModuleFromAddress(static_cast<WNDPROC>(
       &base::win::WrappedWindowProc<WindowProc>));
 }
 
 MessageWindow::MessageWindow(const std::string& class_name, HINSTANCE instance)
     : atom_(0),
       class_name_(class_name),
       instance_(instance),
       window_(NULL) {
 }
@@ skipping 75 matching lines @@
   if (delegate) {
     LRESULT message_result;
     if (delegate->HandleMessage(hwnd, message, wparam, lparam, &message_result))
       return message_result;
   }
 
   return DefWindowProc(hwnd, message, wparam, lparam);
 }
 
 }  // namespace win
-}  // namespace remoting
+}  // namespace base