Send traffic-management headers from extension updater.

extensions/browser/updater/manifest_fetch_data.h

 // 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.
 
 #ifndef EXTENSIONS_BROWSER_UPDATER_MANIFEST_FETCH_DATA_H_
 #define EXTENSIONS_BROWSER_UPDATER_MANIFEST_FETCH_DATA_H_
 
 #include <map>
 #include <set>
 #include <string>
@@ skipping 56 matching lines @@
         : rollcall_days(rollcall),
           active_days(active),
           is_enabled(enabled),
           disable_reasons(reasons) {}
   };
 
   ManifestFetchData(const GURL& update_url,
                     int request_id,
                     const std::string& brand_code,
                     const std::string& base_query_params,
-                    PingMode ping_mode);
+                    PingMode ping_mode,
+                    bool foreground_check);

[Devlin, 2017/03/24 15:14:47]

I'd slightly prefer we use an enum for this, e.g.
enum class FetchPriority {
  FOREGROUND,
  BACKGROUND,
}

and then pass that rather than the anonymous true/false everywhere.  WDYT?

[Minh X. Nguyen, 2017/03/27 23:05:29]

Done.

   ~ManifestFetchData();
 
   // Returns true if this extension information was successfully added. If the
-  // return value is false it means the full_url would have become too long, and
+  // return value is false it means the full_url would have become too long or
+  // the request type is not compatible the current request type, and
   // this ManifestFetchData object remains unchanged.
   bool AddExtension(const std::string& id,
                     const std::string& version,
                     const PingData* ping_data,
                     const std::string& update_url_data,
-                    const std::string& install_source);
+                    const std::string& install_source,
+                    bool foreground_check);
 
   const GURL& base_url() const { return base_url_; }
   const GURL& full_url() const { return full_url_; }
   const std::set<std::string>& extension_ids() const { return extension_ids_; }
   const std::set<int>& request_ids() const { return request_ids_; }
+  bool foreground_check() const { return foreground_check_; }
 
   // Returns true if the given id is included in this manifest fetch.
   bool Includes(const std::string& extension_id) const;
 
   // Returns true if a ping parameter for |type| was added to full_url for this
   // extension id.
   bool DidPing(const std::string& extension_id, PingType type) const;
 
   // Assuming that both this ManifestFetchData and |other| have the same
   // full_url, this method merges the other information associated with the
@@ skipping 22 matching lines @@
   std::set<int> request_ids_;
 
   // The brand code to include with manifest fetch queries, if non-empty and
   // |ping_mode_| >= PING.
   const std::string brand_code_;
 
   // The ping mode for this fetch. This determines whether or not ping data
   // (and possibly extra metrics) will be included in the fetch query.
   const PingMode ping_mode_;
 
+  // If true, this is an on-demand update.
+  bool foreground_check_;
+
   DISALLOW_COPY_AND_ASSIGN(ManifestFetchData);
 };
 
 }  // namespace extensions
 
 #endif  // EXTENSIONS_BROWSER_UPDATER_MANIFEST_FETCH_DATA_H_