Enable forced extension updates on NaCl arch mismatch.

extensions/common/extension.h

 // Copyright (c) 2013 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_COMMON_EXTENSION_H_
 #define EXTENSIONS_COMMON_EXTENSION_H_
 
 #include <algorithm>
 #include <iosfwd>
 #include <map>
@@ skipping 333 matching lines @@
   bool is_legacy_packaged_app() const;
   bool is_extension() const;
   bool is_shared_module() const;
   bool is_theme() const;
 
   bool can_be_incognito_enabled() const;
 
   void AddWebExtentPattern(const URLPattern& pattern);
   const URLPatternSet& web_extent() const { return extent_; }
 
+  // Updates the set of architectures for which this extension has platform-
+  // specific resources installed.
+  void SetPlatformSpecificResourceArchs(const std::set<std::string>& archs);

[Yoyo Zhou, 2014/09/04 02:14:23]

Does this really need to be stored on Extension? I don't think we should be
adding stuff like this in general to it. I think it'd be cleaner to have some
sort of PlatformSpecificArchChecker that the InstalledLoader and the downloader
both make use of. If that doesn't work, I think even InstalledLoader would be a
better place to store this data...

+
+  // Retrieves the set of architectures for which this extension has
+  // platform-specific resources installed.
+  void GetPlatformSpecificResourceArchs(std::set<std::string>* archs) const;
+
  private:
   friend class base::RefCountedThreadSafe<Extension>;
 
   // Chooses the extension ID for an extension based on a variety of criteria.
   // The chosen ID will be set in |manifest|.
   static bool InitExtensionID(extensions::Manifest* manifest,
                               const base::FilePath& path,
                               const ExtensionId& explicit_id,
                               int creation_flags,
                               base::string16* error);
@@ skipping 109 matching lines @@
   bool display_in_new_tab_page_;
 
   // Whether the extension has host permissions or user script patterns that
   // imply access to file:/// scheme URLs (the user may not have actually
   // granted it that access).
   bool wants_file_access_;
 
   // The flags that were passed to InitFromValue.
   int creation_flags_;
 
+  // The set of archs for which platform-specific resources are present in
+  // this extension installation. Null if these have not been identified yet.
+  std::set<std::string> platform_specific_resource_archs_;
+
   DISALLOW_COPY_AND_ASSIGN(Extension);
 };
 
 typedef std::vector<scoped_refptr<const Extension> > ExtensionList;
 typedef std::set<ExtensionId> ExtensionIdSet;
 typedef std::vector<ExtensionId> ExtensionIdList;
 
 // Handy struct to pass core extension info around.
 struct ExtensionInfo {
   ExtensionInfo(const base::DictionaryValue* manifest,
@@ skipping 71 matching lines @@
 
   UpdatedExtensionPermissionsInfo(
       const Extension* extension,
       const PermissionSet* permissions,
       Reason reason);
 };
 
 }  // namespace extensions
 
 #endif  // EXTENSIONS_COMMON_EXTENSION_H_