Remove call to base::TaskTraits::WithPriority() in module_inspector_win.cc.

chrome/browser/conflicts/module_inspector_win.h

 // Copyright 2017 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.
 
 #ifndef CHROME_BROWSER_CONFLICTS_MODULE_INSPECTOR_WIN_H_
 #define CHROME_BROWSER_CONFLICTS_MODULE_INSPECTOR_WIN_H_
 
 #include <memory>
 #include <queue>
 
@@ skipping 41 matching lines @@
   // for inspection.
   void OnInspectionFinished(
       const ModuleInfoKey& module_key,
       std::unique_ptr<ModuleInspectionResult> inspection_result);
 
   OnModuleInspectedCallback on_module_inspected_callback_;
 
   // The modules are put in queue until they are sent for inspection.
   std::queue<ModuleInfoKey> queue_;
 
-  // The traits used on the task that inspects the modules. It originally starts
-  // at a BACKGROUND priority, but is changed to USER_VISIBLE when
-  // IncreaseInspectionPriority() is called.
-  base::TaskTraits inspection_task_traits_;
+  // The TaskPriority of the task that inspects the modules. Is changed to
+  // USER_VISIBLE when IncreaseInspectionPriority() is called.
+  base::TaskPriority inspection_task_priority_ = base::TaskPriority::BACKGROUND;

[Patrick Monette, 2017/05/08 20:35:11]

I prefer to have all the initialization logic at the same place. In this
case, it's easier to move it to the constructor in the .cc file.

[fdoray, 2017/05/08 20:40:02]

Done.

 
   // The vector of paths to %env_var%, used to account for differences in
   // localization and where people keep their files.
   // e.g. c:\windows vs d:\windows
   StringMapping path_mapping_;
 
   base::ThreadChecker thread_checker_;
 
   // Weak pointers are used to safely post the inspection result back to the
   // ModuleInspector from the task scheduler.
   base::WeakPtrFactory<ModuleInspector> weak_ptr_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(ModuleInspector);
 };
 
 #endif  // CHROME_BROWSER_CONFLICTS_MODULE_INSPECTOR_WIN_H_