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

This reverts commit r113922.

Win Canary has been failing to link due to OOM since the weekend.  While this was not the original cause, it might be making things worse.  Speculatively remove for now.

Also changes render_widge_host_view to use Owned() instead of Passed().

BUG=none
TEST=existing.

Review URL: http://codereview.chromium.org/8931008

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@114247 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 0 insertions, 13 deletions

diff --git a/chrome/common/extensions/extension_manifests_unittest.cc b/chrome/common/extensions/extension_manifests_unittest.cc
index 2943b39..357ab76 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_; }