Revert 115441 - Redo r113722 - Add Pass(), which implements move semantics, to scoped_ptr, scoped...

chrome/common/extensions/extension_manifests_unittest.cc

Index: chrome/common/extensions/extension_manifests_unittest.cc
===================================================================
--- chrome/common/extensions/extension_manifests_unittest.cc	(revision 115443)
+++ chrome/common/extensions/extension_manifests_unittest.cc	(working copy)
@@ -74,20 +74,7 @@
     Manifest(DictionaryValue* manifest, const char* name)
         : name_(name), manifest_(manifest) {
     }
-    Manifest(const Manifest& m) {
-      // C++98 requires the copy constructor for a type to be visiable if you
-      // take a const-ref of a temporary for that type.  Since Manifest
-      // contains a scoped_ptr, its implicit copy constructor is declared
-      // Manifest(Manifest&) according to spec 12.8.5.  This breaks the first
-      // requirement and thus you cannot use it with LoadAndExpectError() or
-      // LoadAndExpectSuccess() easily.
-      //
-      // To get around this spec pedantry, we declare the copy constructor
-      // explicitly.  It will never get invoked.
-      NOTREACHED();
-    }
 
-
     const std::string& name() const { return name_; }
 
     DictionaryValue* GetManifest(std::string* error) const {