Update TaskTraits in sw_reporter_installer_win.cc.

chrome/browser/component_updater/sw_reporter_installer_win.cc

 // Copyright (c) 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/component_updater/sw_reporter_installer_win.h"
 
 #include <stdint.h>
 
 #include <algorithm>
 #include <map>
@@ skipping 118 matching lines @@
 // the current browser startup is complete. (This is the default
 // |reporter_runner| function passed to the |SwReporterInstallerTraits|
 // constructor in |RegisterSwReporterComponent| below.)
 void RunSwReportersAfterStartup(
     const safe_browsing::SwReporterQueue& invocations,
     const base::Version& version) {
   content::BrowserThread::PostAfterStartupTask(
       FROM_HERE, base::ThreadTaskRunnerHandle::Get(),
       base::Bind(&safe_browsing::RunSwReporters, invocations, version,
                  base::ThreadTaskRunnerHandle::Get(),
-                 // Runs LaunchAndWaitForExit() (srt_fetcher_win.cc).
+                 // Runs LaunchAndWaitForExit() which creates and joins a
+                 // process, hence MayBlock() and WithSyncPrimitives().

[gab, 2016/12/21 16:11:01]

// Runs LaunchAndWaitForExit() which creates (MayBlock()) and joins
(WithSyncPrimitives()) a process.

(to be specific about which operation requires which trait?)

[joenotcharles, 2016/12/21 16:20:36]

Yes, please clarify this.

[fdoray, 2016/12/21 16:28:42]

Done.

                  base::CreateTaskRunnerWithTraits(
                      base::TaskTraits()
                          .WithShutdownBehavior(
                              base::TaskShutdownBehavior::CONTINUE_ON_SHUTDOWN)
                          .WithPriority(base::TaskPriority::BACKGROUND)
-                         .WithWait())));
+                         .MayBlock()
+                         .WithSyncPrimitives())));
 }
 
 // Ensures |str| contains only alphanumeric characters and characters from
 // |extras|, and is not longer than |max_length|.
 bool ValidateString(const std::string& str,
                     const std::string& extras,
                     size_t max_length) {
   return str.size() <= max_length &&
          std::all_of(str.cbegin(), str.cend(), [&extras](char c) {
            return base::IsAsciiAlpha(c) || base::IsAsciiDigit(c) ||
@@ skipping 341 matching lines @@
 }
 
 void RegisterProfilePrefsForSwReporter(
     user_prefs::PrefRegistrySyncable* registry) {
   registry->RegisterStringPref(prefs::kSwReporterPromptVersion, "");
 
   registry->RegisterStringPref(prefs::kSwReporterPromptSeed, "");
 }
 
 }  // namespace component_updater