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

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

Index: chrome/browser/ui/views/frame/browser_view.cc
diff --git a/chrome/browser/ui/views/frame/browser_view.cc b/chrome/browser/ui/views/frame/browser_view.cc
index e9fe68b85300acb519b0cdb11b14d49ebdf14321..4c9ce5738431529979cd1a25a80ddbf162d15027 100644
--- a/chrome/browser/ui/views/frame/browser_view.cc
+++ b/chrome/browser/ui/views/frame/browser_view.cc
@@ -2178,7 +2178,7 @@ void BrowserView::LoadingAnimationCallback() {
 
 void BrowserView::OnLoadCompleted() {
 #if defined(OS_WIN)
-  DCHECK(!jumplist_.get());
+  DCHECK(!jumplist_);
   jumplist_ = JumpListFactory::GetForProfile(browser_->profile());

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

on the other hand, is this jumplist_ member even needed? could this be simply:
  // Ensure that this browser's Profile has a JumpList so that the JumpList is
kept up to date.
  JumpListFactory::GetForProfile(browser_->profile());

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

I don't think |jumplist_| is needed, so removed it.

 #endif
 }