Use rvalue reference instead of ExtensionBuilder::pass()

extensions/common/extension_builder.h

Index: extensions/common/extension_builder.h
diff --git a/extensions/common/extension_builder.h b/extensions/common/extension_builder.h
index 4b40178b9b71b790ce00bcae139e97d4850f1ec2..143fdc24ee80cd06817c5d075604b653acce22c9 100644
--- a/extensions/common/extension_builder.h
+++ b/extensions/common/extension_builder.h
@@ -24,14 +24,12 @@ class ExtensionBuilder {
   ExtensionBuilder();
   ~ExtensionBuilder();
 
+  ExtensionBuilder(ExtensionBuilder&& other);

[danakj, 2015/12/09 18:46:44]

you should make a move-operator= when you add a move constructor. (similar rules
to copy constructors)

[limasdf, 2015/12/16 09:34:54]

Yes, https://www.chromium.org/rvalue-references #9 says it. Sorry I forgot it.

Done.

+
   // Can only be called once, after which it's invalid to use the builder.
   // CHECKs that the extension was created successfully.
   scoped_refptr<Extension> Build();
 
-  // Workaround to allow you to pass rvalue ExtensionBuilders by reference to
-  // other functions, e.g. UseBuilder(ExtensionBuilder().Pass())
-  ExtensionBuilder& Pass() { return *this; }
-
   // Defaults to FilePath().
   ExtensionBuilder& SetPath(const base::FilePath& path);