Give StatusTrayStateChangerProxy a virtual destructor.

chrome/browser/ui/views/status_icons/status_tray_win.cc

Index: chrome/browser/ui/views/status_icons/status_tray_win.cc
diff --git a/chrome/browser/ui/views/status_icons/status_tray_win.cc b/chrome/browser/ui/views/status_icons/status_tray_win.cc
index 18830d360830dafd33eaf7d948d47b9d30ed42f6..098a46db07be602e179f8567b0431f224d6c75ba 100644
--- a/chrome/browser/ui/views/status_icons/status_tray_win.cc
+++ b/chrome/browser/ui/views/status_icons/status_tray_win.cc
@@ -27,6 +27,9 @@ UINT ReservedIconId(StatusTray::StatusIconType type) {
 }
 }  // namespace
 
+StatusTrayStateChangerProxy::~StatusTrayStateChangerProxy() {

[sky, 2014/07/18 23:06:59]

I would inline this.

[Nico, 2014/07/19 00:24:12]

Done. (On posix, having an out-of-line destructor can be nice since a key
function tells the compiler where to emit its vtable, but for windows-only files
I suppose it doesn't matter.)

+}
+
 // Default implementation for StatusTrayStateChanger that communicates to
 // Exporer.exe via COM.  It spawns a background thread with a fresh COM
 // apartment and requests that the visibility be increased unless the user