Fix stability and data racing issues, coalesce more updates for JumpList

chrome/browser/ui/views/frame/browser_view.h

Index: chrome/browser/ui/views/frame/browser_view.h
diff --git a/chrome/browser/ui/views/frame/browser_view.h b/chrome/browser/ui/views/frame/browser_view.h
index 0b714ec406baa1d2fb13518c16c3224fc18154b8..a9c32104a2531099c7d667b72499419bfb490958 100644
--- a/chrome/browser/ui/views/frame/browser_view.h
+++ b/chrome/browser/ui/views/frame/browser_view.h
@@ -688,8 +688,8 @@ class BrowserView : public BrowserWindow,
   // Helper class to listen for completion of first page load.
   std::unique_ptr<LoadCompleteListener> load_complete_listener_;
 
-  // The custom JumpList for Windows 7.
-  scoped_refptr<JumpList> jumplist_;
+  // The custom JumpList for Windows.
+  JumpList* jumplist_ = nullptr;

[Peter Kasting, 2017/06/12 22:21:54]

I'm a fan of this kind of in-declaration initialization.  But I'm not a fan of
doing it for just one BT member and not the others.  If you want to do this, I
would write a separate CL that converts all members of this class to use this
form of initialization where possible.

[grt (UTC plus 2), 2017/06/13 07:32:27]

I agree with Peter here. Apologies for suggesting initializing it here rather
than in the .cc file.

[chengx, 2017/06/13 19:16:01]

Done.

[chengx, 2017/06/13 19:16:01]

I've landed crrev/2939583002 to apply the in-declaration initialization to all
member variables where possible in this class.

 #endif
 
   // The timer used to update frames for the Loading Animation.