Fix Up SingletonHwnd Observer Lifetime Issues

content/browser/time_zone_monitor_win.cc

 // Copyright 2014 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 "content/browser/time_zone_monitor.h"
 
 #include <windows.h>
 
 #include "base/basictypes.h"
+#include "base/bind.h"
+#include "base/bind_helpers.h"
 #include "ui/gfx/win/singleton_hwnd.h"
 
 namespace content {
 
-class TimeZoneMonitorWin : public TimeZoneMonitor,
-                           public gfx::SingletonHwnd::Observer {
+class TimeZoneMonitorWin : public TimeZoneMonitor {
  public:
   TimeZoneMonitorWin() : TimeZoneMonitor() {
-    gfx::SingletonHwnd::GetInstance()->AddObserver(this);
+    singletonHwndObserver_.SetWndProc(
+        base::Bind(&TimeZoneMonitorWin::OnTimeChange, base::Unretained(this)));
   }
 
-  virtual ~TimeZoneMonitorWin() {
-    gfx::SingletonHwnd::GetInstance()->RemoveObserver(this);
-  }
+  virtual ~TimeZoneMonitorWin() {}
 
-  // gfx::SingletonHwnd::Observer implementation.
-  virtual void OnWndProc(HWND hwnd,
-                         UINT message,
-                         WPARAM wparam,
-                         LPARAM lparam) override {
+ private:
+  void OnTimeChange(HWND hwnd, UINT message, WPARAM wparam, LPARAM lparam) {
     if (message != WM_TIMECHANGE) {
       return;
     }
 
     NotifyRenderers();
   }
 
- private:
+  gfx::SingletonHwnd::Observer singletonHwndObserver_;

[sky, 2015/04/24 20:58:23]

unix_hacker_style

[robliao, 2015/04/24 21:49:03]

Done.

+
   DISALLOW_COPY_AND_ASSIGN(TimeZoneMonitorWin);
 };
 
 // static
 scoped_ptr<TimeZoneMonitor> TimeZoneMonitor::Create() {
   return scoped_ptr<TimeZoneMonitor>(new TimeZoneMonitorWin());
 }
 
 }  // namespace content