Make Clone, Contains, Equal and IPC function non-virtual and implement in manifest_permission.cc

extensions/common/permissions/manifest_permission_set_unittest.cc

 // 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.
 
 #include "base/pickle.h"
 #include "base/values.h"
 #include "extensions/common/extension_messages.h"
 #include "extensions/common/permissions/manifest_permission.h"
 #include "extensions/common/permissions/manifest_permission_set.h"
 #include "ipc/ipc_message.h"
@@ skipping 16 matching lines @@
   }
 
   virtual bool HasMessages() const OVERRIDE {
     return false;
   }
 
   virtual PermissionMessages GetMessages() const OVERRIDE {
     return PermissionMessages();
   }
 
-  virtual bool FromValue(const base::Value* value) OVERRIDE {
-    return false;
-  }
+  virtual bool FromValue(const base::Value* value) OVERRIDE { return true; }
 
   virtual scoped_ptr<base::Value> ToValue() const OVERRIDE {
-    return scoped_ptr<base::Value>(base::Value::CreateNullValue());
-  }
-
-  virtual ManifestPermission* Clone() const OVERRIDE {
-    return new MockManifestPermission(name_);
+    return make_scoped_ptr(base::Value::CreateNullValue());
   }
 
   virtual ManifestPermission* Diff(const ManifestPermission* rhs)
       const OVERRIDE {
     const MockManifestPermission* other =
         static_cast<const MockManifestPermission*>(rhs);
     EXPECT_EQ(name_, other->name_);
     return NULL;
   }
 
   virtual ManifestPermission* Union(const ManifestPermission* rhs)
       const OVERRIDE {
     const MockManifestPermission* other =
         static_cast<const MockManifestPermission*>(rhs);
     EXPECT_EQ(name_, other->name_);
     return new MockManifestPermission(name_);
   }
 
   virtual ManifestPermission* Intersect(const ManifestPermission* rhs)
       const OVERRIDE {
     const MockManifestPermission* other =
         static_cast<const MockManifestPermission*>(rhs);
     EXPECT_EQ(name_, other->name_);
     return new MockManifestPermission(name_);
   }
 
-  virtual bool Contains(const ManifestPermission* rhs) const OVERRIDE {
-    const MockManifestPermission* other =
-        static_cast<const MockManifestPermission*>(rhs);
-    EXPECT_EQ(name_, other->name_);
-    return true;
-  }
-
-  virtual bool Equal(const ManifestPermission* rhs) const OVERRIDE {
-    const MockManifestPermission* other =
-        static_cast<const MockManifestPermission*>(rhs);
-    EXPECT_EQ(name_, other->name_);
-    return true;
-  }
-
-  virtual void Write(IPC::Message* m) const OVERRIDE {
-    IPC::WriteParam(m, name_);
-  }
-
-  virtual bool Read(const IPC::Message* m, PickleIterator* iter) OVERRIDE {
-    std::string read_name;
-    bool result = IPC::ReadParam(m, iter, &read_name);
-    if (!result)
-      return result;
-    EXPECT_EQ(read_name, name_);
-    return true;
-  }
-
-  virtual void Log(std::string* log) const OVERRIDE {
-  }
-
  private:
   std::string name_;
 };
 
 TEST(ManifestPermissionSetTest, General) {
   ManifestPermissionSet set;
   set.insert(new MockManifestPermission("p1"));
   set.insert(new MockManifestPermission("p2"));
   set.insert(new MockManifestPermission("p3"));
   set.insert(new MockManifestPermission("p4"));
@@ skipping 150 matching lines @@
   EXPECT_EQ(expected_permissions, result);
 
   // |result| = |permissions1| - |permissions2| -->
   //   |result| intersect |permissions2| == empty_set
   ManifestPermissionSet result2;
   ManifestPermissionSet::Intersection(result, permissions2, &result2);
   EXPECT_TRUE(result2.empty());
 }
 
 }  // namespace extensions