Handle the case when the updates are disabled for a component.

components/update_client/update_checker.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 "components/update_client/update_checker.h"
 
 #include <stddef.h>
 
 #include <memory>
 #include <string>
@@ skipping 68 matching lines @@
     const CrxUpdateItem* item = items[i];
     const update_client::InstallerAttributes installer_attributes(
         SanitizeInstallerAttributes(item->component.installer_attributes));
     std::string app("<app ");
     base::StringAppendF(&app, "appid=\"%s\" version=\"%s\"", item->id.c_str(),
                         item->component.version.GetString().c_str());
     if (!brand.empty())
       base::StringAppendF(&app, " brand=\"%s\"", brand.c_str());
     if (item->on_demand)
       base::StringAppendF(&app, " installsource=\"ondemand\"");
-
-    for (const auto& attr : installer_attributes)
+    for (const auto& attr : installer_attributes) {
       base::StringAppendF(&app, " %s=\"%s\"", attr.first.c_str(),
                           attr.second.c_str());
+    }
+    base::StringAppendF(&app, ">");
 
-    base::StringAppendF(&app, ">");
-    base::StringAppendF(&app, "<updatecheck />");
+    base::StringAppendF(&app, "<updatecheck");
+    if (item->component.supports_group_policy_enable_component_updates &&
+        !config.EnabledComponentUpdates()) {
+      base::StringAppendF(&app, " \"updatedisabled\"=true");

[waffles, 2016/08/03 23:49:02]

I think you mean " updatedisabled=\"true\"". (Also in the tests.)

+    }
+    base::StringAppendF(&app, "/>");
+
     base::StringAppendF(&app, "<ping rd=\"%d\" ping_freshness=\"%s\" />",
                         metadata->GetDateLastRollCall(item->id),
                         metadata->GetPingFreshness(item->id).c_str());
     if (!item->component.fingerprint.empty()) {
       base::StringAppendF(&app,
                           "<packages>"
                           "<package fp=\"%s\"/>"
                           "</packages>",
                           item->component.fingerprint.c_str());
     }
@@ skipping 106 matching lines @@
 }  // namespace
 
 std::unique_ptr<UpdateChecker> UpdateChecker::Create(
     const scoped_refptr<Configurator>& config,
     PersistedData* persistent) {
   return std::unique_ptr<UpdateChecker>(
       new UpdateCheckerImpl(config, persistent));
 }
 
 }  // namespace update_client