Time out jumplist update for very slow or busy OS

chrome/browser/win/jumplist_updater.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 "chrome/browser/win/jumplist_updater.h"
 
 #include <windows.h>
 #include <propkey.h>
 #include <shobjidl.h>
 
@@ skipping 92 matching lines @@
 JumpListUpdater::JumpListUpdater(const std::wstring& app_user_model_id)
     : app_user_model_id_(app_user_model_id),
       user_max_items_(0) {
 }
 
 JumpListUpdater::~JumpListUpdater() {
 }
 
 // static
 bool JumpListUpdater::IsEnabled() {
-  // JumpList is implemented only on Windows 7 or later.
   // Do not create custom JumpLists in tests. See http://crbug.com/389375.
   return base::win::GetVersion() >= base::win::VERSION_WIN7 &&

[grt (UTC plus 2), 2017/04/24 09:15:07]

please remove this check and "#include "base/win/windows_version.h"" above.

[chengx, 2017/04/24 17:24:40]

Done.

          !base::CommandLine::ForCurrentProcess()->HasSwitch(
              switches::kTestType);
 }
 
 bool JumpListUpdater::BeginUpdate() {
   // TODO(chengx): Remove the UMA histogram after fixing http://crbug.com/40407.
   SCOPED_UMA_HISTOGRAM_TIMER("WinJumplistUpdater.BeginUpdateDuration");
 
   // This instance is expected to be one-time-use only.
   DCHECK(!destination_list_.Get());
@@ skipping 123 matching lines @@
   // It seems the ICustomDestinationList::AppendCategory() function just
   // replaces all items in the given category with the ones in the new list.
   base::win::ScopedComPtr<IObjectArray> object_array;
   result = collection.QueryInterface(object_array.Receive());
   if (FAILED(result))
     return false;
 
   return SUCCEEDED(destination_list_->AppendCategory(category_name.c_str(),
                                                      object_array.Get()));
 }