Refactor PermissionsData pt1

chrome/renderer/extensions/chrome_extensions_dispatcher_delegate.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 "chrome/renderer/extensions/chrome_extensions_dispatcher_delegate.h"
 
 #include "base/command_line.h"
 #include "base/sha1.h"
 #include "base/strings/string_number_conversions.h"
 #include "chrome/common/chrome_switches.h"
@@ skipping 312 matching lines @@
 void ChromeExtensionsDispatcherDelegate::ClearTabSpecificPermissions(
     const extensions::Dispatcher* dispatcher,
     int tab_id,
     const std::vector<std::string>& extension_ids) {
   for (std::vector<std::string>::const_iterator it = extension_ids.begin();
        it != extension_ids.end();
        ++it) {
     const extensions::Extension* extension =
         dispatcher->extensions()->GetByID(*it);
     if (extension)
-      extensions::PermissionsData::ClearTabSpecificPermissions(extension,
-                                                               tab_id);
+      extensions::PermissionsData::ForExtension(extension)
+          ->ClearTabSpecificPermissions(tab_id);
   }
 }
 
 void ChromeExtensionsDispatcherDelegate::UpdateTabSpecificPermissions(
     const extensions::Dispatcher* dispatcher,
     int page_id,
     int tab_id,
     const std::string& extension_id,
     const extensions::URLPatternSet& origin_set) {
   content::RenderView* view = extensions::TabFinder::Find(tab_id);
 
   // For now, the message should only be sent to the render view that contains
   // the target tab. This may change. Either way, if this is the target tab it
   // gives us the chance to check against the page ID to avoid races.
   DCHECK(view);
   if (view && view->GetPageId() != page_id)
     return;
 
   const extensions::Extension* extension =
       dispatcher->extensions()->GetByID(extension_id);
   if (!extension)
     return;
 
-  extensions::PermissionsData::UpdateTabSpecificPermissions(
-      extension,
-      tab_id,
-      new extensions::PermissionSet(extensions::APIPermissionSet(),
-                                    extensions::ManifestPermissionSet(),
-                                    origin_set,
-                                    extensions::URLPatternSet()));
+  extensions::PermissionsData::ForExtension(extension)
+      ->UpdateTabSpecificPermissions(
+          tab_id,
+          new extensions::PermissionSet(extensions::APIPermissionSet(),
+                                        extensions::ManifestPermissionSet(),
+                                        origin_set,
+                                        extensions::URLPatternSet()));
 }
 
 void ChromeExtensionsDispatcherDelegate::HandleWebRequestAPIUsage(
     bool adblock,
     bool adblock_plus,
     bool other) {
   webrequest_adblock_ = adblock;
   webrequest_adblock_plus_ = adblock_plus;
   webrequest_other_ = other;
 }