Support socket endpoint permissions for AppsV2 Socket API.

chrome/common/extensions/permissions/api_permission.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/permissions/api_permission.h"
 
 #include "chrome/common/extensions/permissions/permissions_info.h"
 #include "grit/generated_resources.h"
 #include "ui/base/l10n/l10n_util.h"
 
 namespace {
 
+using extensions::APIPermission;
+using extensions::APIPermissionDetail;
+
 const char kOldUnlimitedStoragePermission[] = "unlimited_storage";
 const char kWindowsPermission[] = "windows";
 const char kTemporaryBackgroundAlias[] = "background_alias_do_not_use";
 
+class SimpleDetail : public APIPermissionDetail {
+ public:
+  SimpleDetail(const APIPermission* permission)
+    : APIPermissionDetail(permission) { }
+
+  virtual bool FromValue(const base::Value* value) OVERRIDE {
+    return true;
+  }
+
+  virtual void ToValue(base::Value** value) const OVERRIDE {
+  }
+
+  virtual bool Check(
+      const APIPermissionDetail::DetailParam* detail) const OVERRIDE {
+    return !detail;
+  }
+
+  virtual bool Equal(const APIPermissionDetail* detail) const OVERRIDE {
+    if (this == detail)
+      return true;
+    CHECK(permission() == detail->permission());
+    return true;
+  }
+
+  virtual APIPermissionDetail* Clone() const OVERRIDE {
+    return new SimpleDetail(permission());
+  }
+
+  virtual APIPermissionDetail* Diff(
+      const APIPermissionDetail* detail) const OVERRIDE {
+    CHECK(permission() == detail->permission());

[miket_OOO, 2012/07/31 18:18:37]

Is this right? Do we really want to crash?

Same with the rest of these set methods -- is this the right behavior?

[Peng, 2012/07/31 19:32:28]

Diff, Union, Intersect can only be used on same API details and caller should
guarantee it.

+    return NULL;
+  }
+
+  virtual APIPermissionDetail* Union(
+      const APIPermissionDetail* detail) const OVERRIDE {
+    CHECK(permission() == detail->permission());
+    return new SimpleDetail(permission());
+  }
+
+  virtual APIPermissionDetail* Intersect(
+      const APIPermissionDetail* detail) const OVERRIDE {
+    CHECK(permission() == detail->permission());
+    return new SimpleDetail(permission());
+  }
+
+  virtual bool Contains(const APIPermissionDetail* detail) const OVERRIDE {
+    CHECK(permission() == detail->permission());
+    return true;
+  }
+
+  virtual void Write(IPC::Message *m) const OVERRIDE {
+  }
+
+  virtual bool Read(const IPC::Message *m, PickleIterator *iter) OVERRIDE {
+    return true;
+  }
+
+  virtual void Log(std::string *l) const OVERRIDE { }
+
+ protected:
+  friend extensions::APIPermissionDetail;
+  virtual ~SimpleDetail() {}
+};
+
 }  // namespace
 
 namespace extensions {
 
 //
 // APIPermission
 //
 
 APIPermission::~APIPermission() {}
 
+scoped_refptr<APIPermissionDetail> APIPermission::CreateDetail() const {
+  scoped_refptr<APIPermissionDetail> p;
+  if (!detail_constructor_.is_null())
+    p = detail_constructor_.Run();
+  if (!p.get())
+    p = new SimpleDetail(this);
+  return p;
+}
+
 PermissionMessage APIPermission::GetMessage_() const {
   return PermissionMessage(
       message_id_, l10n_util::GetStringUTF16(l10n_message_id_));
 }
 
 APIPermission::APIPermission(
     ID id,
     const char* name,
     int l10n_message_id,
     PermissionMessage::ID message_id,
-    int flags)
+    int flags,
+    const DetailConstructor &detail_constructor)

[miket_OOO, 2012/07/31 18:18:37]

whitespace

[Peng, 2012/07/31 19:32:28]

Done.

     : id_(id),
       name_(name),
       flags_(flags),
       l10n_message_id_(l10n_message_id),
-      message_id_(message_id) {}
+      message_id_(message_id),
+      detail_constructor_(detail_constructor) {}
 
 // static
 void APIPermission::RegisterAllPermissions(
     PermissionsInfo* info) {
 
   struct PermissionRegistration {
     APIPermission::ID id;
     const char* name;
     int flags;
     int l10n_message_id;
     PermissionMessage::ID message_id;
+    DetailConstructor detail_constructor;
   } PermissionsToRegister[] = {
     // Register permissions for all extension types.
     { kBackground, "background" },
     { kClipboardRead, "clipboardRead", kFlagNone,
       IDS_EXTENSION_PROMPT_WARNING_CLIPBOARD,
       PermissionMessage::kClipboard },
     { kClipboardWrite, "clipboardWrite" },
     { kDeclarative, "declarative" },
     { kDeclarativeWebRequest, "declarativeWebRequest" },
     { kDownloads, "downloads", kFlagNone,
@@ skipping 110 matching lines @@
       kFlagCannotBeOptional,
       IDS_EXTENSION_PROMPT_WARNING_MEDIA_GALLERIES_ALL_GALLERIES,
       PermissionMessage::kMediaGalleriesAllGalleries },
   };
 
   for (size_t i = 0; i < ARRAYSIZE_UNSAFE(PermissionsToRegister); ++i) {
     const PermissionRegistration& pr = PermissionsToRegister[i];
     info->RegisterPermission(
         pr.id, pr.name, pr.l10n_message_id,
         pr.message_id ? pr.message_id : PermissionMessage::kNone,
-        pr.flags);
+        pr.flags,
+        pr.detail_constructor);
   }
 
   // Register aliases.
   info->RegisterAlias("unlimitedStorage", kOldUnlimitedStoragePermission);
   info->RegisterAlias("tabs", kWindowsPermission);
   // TODO(mihaip): Should be removed for the M20 branch, see
   // http://crbug.com/120447 for more details.
   info->RegisterAlias("background", kTemporaryBackgroundAlias);
 }
 
+APIPermissionDetail::~APIPermissionDetail() {
+}
+
 }  // namespace extensions