Enable permission warnings from ManifestHandlers.

chrome/common/extensions/api/sockets/sockets_handler.h

 // Copyright 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 CHROME_COMMON_EXTENSIONS_API_SOCKETS_SOCKETS_HANDLER_H_
 #define CHROME_COMMON_EXTENSIONS_API_SOCKETS_SOCKETS_HANDLER_H_
 
 #include "base/strings/string16.h"
 #include "chrome/common/extensions/extension.h"
 #include "chrome/common/extensions/permissions/socket_permission_data.h"
 #include "extensions/common/manifest_handler.h"
 
 namespace extensions {
 
 // Parses the "sockets" manifest key.
 class SocketsHandler : public ManifestHandler {
  public:
   SocketsHandler();
   virtual ~SocketsHandler();
 
   virtual bool Parse(Extension* extension, string16* error) OVERRIDE;
+  virtual ManifestPermission* CreatePermission() OVERRIDE;
+  virtual ManifestPermission* CreateInitialRequiredPermission(
+    const Extension* extension) OVERRIDE;
 
  private:
   virtual const std::vector<std::string> Keys() const OVERRIDE;
 
   DISALLOW_COPY_AND_ASSIGN(SocketsHandler);
 };
 
 // The parsed form of the "sockets" manifest entry.
 class SocketsManifestData : public Extension::ManifestData {
  public:
+  typedef std::set<SocketPermissionEntry> SocketPermissionEntrySet;
+  enum PermissionKind {
+    kNone = 0x0000,

[Yoyo Zhou, 2013/11/12 02:39:29]

nit: we usually write (1<<0), (1<<1), etc. for these kind of bitmasks.

[rpaquay, 2013/11/12 21:40:31]

Done.

+    kTcpPermission = 0x0001,
+    kUdpPermission = 0x0002,
+    kTcpServerPermission = 0x0004
+  };
+
   SocketsManifestData();
   virtual ~SocketsManifestData();
 
-  // Gets the ExternallyConnectableInfo for |extension|, or NULL if none was
+  // Gets the SocketsManifestData for |extension|, or NULL if none was
   // specified.
   static SocketsManifestData* Get(const Extension* extension);
 
   static bool CheckRequest(const Extension* extension,
                            const content::SocketPermissionRequest& request);
 
   // Tries to construct the info based on |value|, as it would have appeared in
   // the manifest. Sets |error| and returns an empty scoped_ptr on failure.
   static scoped_ptr<SocketsManifestData> FromValue(
       const base::Value& value,
       std::vector<InstallWarning>* install_warnings,
       string16* error);
 
+  bool HasMessages() const;
+  PermissionMessages GetPermissionMessages() const;
+
+  SocketsManifestData* Diff(const SocketsManifestData* rhs) const;
+  SocketsManifestData* Union(const SocketsManifestData* rhs) const;
+  SocketsManifestData* Intersect(const SocketsManifestData* rhs) const;
+  bool Contains(const SocketsManifestData* rhs) const;
+  bool Equal(const SocketsManifestData* rhs) const;
+  SocketsManifestData* Clone() const;
+
+  // IPC functions
+  void Write(IPC::Message* m) const;
+  bool Read(const IPC::Message* m, PickleIterator* iter);
+  void Log(std::string* log) const;
+
+  const SocketPermissionEntrySet& entries() const { return permissions_; }
+
+  bool is_udp() const { return (kinds_ & kUdpPermission) != 0; }

[Yoyo Zhou, 2013/11/12 02:39:29]

Is "is" or "has" better for this if it's a bitmask?

[rpaquay, 2013/11/12 21:40:31]

Done.

+  bool is_tcp() const { return (kinds_ & kTcpPermission) != 0; }
+  bool is_tcp_server() const { return (kinds_ & kTcpServerPermission) != 0; }
+
  private:
-  typedef std::set<SocketPermissionEntry> PermissionSet;
-
   static bool ParseHostPattern(
       SocketsManifestData* manifest_data,
       content::SocketPermissionRequest::OperationType operation_type,
       const scoped_ptr<std::string>& value,
       string16* error);
 
   void AddPermission(const SocketPermissionEntry& entry);
 
   bool CheckRequestImpl(const Extension* extension,
                         const content::SocketPermissionRequest& request);
 
-  PermissionSet permissions_;
+  bool AddAnyHostMessage(PermissionMessages& messages) const;
+  void AddSubdomainHostMessage(PermissionMessages& messages) const;
+  void AddSpecificHostMessage(PermissionMessages& messages) const;
+  void AddNetworkListMessage(PermissionMessages& messages) const;
+
+  SocketPermissionEntrySet permissions_;
+  int kinds_;  // PermissionKind bits
 };
 
 }  // namespace extensions
 
 #endif  // CHROME_COMMON_EXTENSIONS_API_SOCKETS_SOCKETS_HANDLER_H_