Revert of "Redo r113722 - Add Pass(), which implements move semantics, to scoped_ptr, scoped_ar..."

chrome/common/extensions/extension_manifests_unittest.cc

Index: chrome/common/extensions/extension_manifests_unittest.cc
diff --git a/chrome/common/extensions/extension_manifests_unittest.cc b/chrome/common/extensions/extension_manifests_unittest.cc
index 2943b39213805e89330317287c236065ec621343..357ab766ff6fd3d9ee8cb01501d6fa157537be25 100644
--- a/chrome/common/extensions/extension_manifests_unittest.cc
+++ b/chrome/common/extensions/extension_manifests_unittest.cc
@@ -74,19 +74,6 @@ class ExtensionManifestTest : public testing::Test {
     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_; }