Move shared file type images into chrome://theme/ scheme

chrome/common/extensions/extension.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/common/extensions/extension.h"
 
 #include <ostream>
 
 #include "base/base64.h"
 #include "base/basictypes.h"
@@ skipping 3473 matching lines @@
 }
 
 const URLPatternSet& Extension::GetEffectiveHostPermissions() const {
   base::AutoLock auto_lock(runtime_data_lock_);
   return runtime_data_.GetActivePermissions()->effective_hosts();
 }
 
 bool Extension::HasHostPermission(const GURL& url) const {
   if (url.SchemeIs(chrome::kChromeUIScheme) &&
       url.host() != chrome::kChromeUIFaviconHost &&
+      url.host() != chrome::kChromeUIThemeHost &&

[asargent_no_longer_on_chrome, 2012/09/14 22:01:01]

Are you sure you want to let non-component extensions load chrome://theme
resource urls?

[zel, 2012/09/14 22:29:25]

No, we don't, I misinterpreted this if statement. Ben, please remove this added
line.

[benrg, 2012/09/14 22:38:16]

Done.

       url.host() != chrome::kChromeUIThumbnailHost &&
       location() != Extension::COMPONENT) {
     return false;
   }
 
   base::AutoLock auto_lock(runtime_data_lock_);
   return runtime_data_.GetActivePermissions()->
       HasExplicitAccessToOrigin(url);
 }
 
@@ skipping 458 matching lines @@
 
 UpdatedExtensionPermissionsInfo::UpdatedExtensionPermissionsInfo(
     const Extension* extension,
     const PermissionSet* permissions,
     Reason reason)
     : reason(reason),
       extension(extension),
       permissions(permissions) {}
 
 }   // namespace extensions