[Extensions] Fix InstalledLoader histograms to ignore component extenisons

chrome/browser/extensions/installed_loader.cc

 // Copyright (c) 2012 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/extensions/installed_loader.h"
 
 #include "base/files/file_path.h"
 #include "base/metrics/histogram_macros.h"
 #include "base/metrics/sparse_histogram.h"
 #include "base/strings/stringprintf.h"
@@ skipping 371 matching lines @@
       } else {
         UMA_HISTOGRAM_ENUMERATION("Extensions.ExternalItemState",
                                   EXTERNAL_ITEM_NONWEBSTORE_ENABLED,
                                   EXTERNAL_ITEM_MAX_ITEMS);
       }
     }
 
     // From now on, don't count component extensions, since they are only
     // extensions as an implementation detail. Continue to count unpacked
     // extensions for a few metrics.
-    if (location == Manifest::COMPONENT)
+    if (Manifest::IsComponentLocation(location))
       continue;
 
     // Histogram for non-webstore extensions overriding new tab page should
     // include unpacked extensions.
     if (!extension->from_webstore() &&
         URLOverrides::GetChromeURLOverrides(extension).count("newtab")) {
       ++non_webstore_ntp_override_count;
     }
 
     // Don't count unpacked extensions anymore, either.
@@ skipping 207 matching lines @@
 int InstalledLoader::GetCreationFlags(const ExtensionInfo* info) {
   int flags = extension_prefs_->GetCreationFlags(info->extension_id);
   if (!Manifest::IsUnpackedLocation(info->extension_location))
     flags |= Extension::REQUIRE_KEY;
   if (extension_prefs_->AllowFileAccess(info->extension_id))
     flags |= Extension::ALLOW_FILE_ACCESS;
   return flags;
 }
 
 }  // namespace extensions