Make extensions::DictionaryBuilder and extensions::ListValue unmovable.

There's no reason for these classes to be movable. std::move() is just
being used as a synonym for Build().

In addition:
- Build() is fewer characters than std::move().
- clang-format works better when builder syntax is consistently used,
  which makes it easier for readers to visually match up deeply nested
  builders.
- It's surprising to see std::move() used with what looks like a
  temporary.

BUG=none

Review URL: https://codereview.chromium.org/1739183003

Cr-Commit-Position: refs/heads/master@{#378107}

1 files changed, 6 insertions, 6 deletions

diff --git a/extensions/browser/runtime_data_unittest.cc b/extensions/browser/runtime_data_unittest.cc
index 6e522a4..790dd25 100644
--- a/extensions/browser/runtime_data_unittest.cc
+++ b/extensions/browser/runtime_data_unittest.cc
@@ -21,12 +21,12 @@ namespace {
 // Creates a very simple extension with a background page.
 scoped_refptr<Extension> CreateExtensionWithBackgroundPage() {
   return ExtensionBuilder()
-      .SetManifest(std::move(
-          DictionaryBuilder()
-              .Set("name", "test")
-              .Set("version", "0.1")
-              .Set("background",
-                   std::move(DictionaryBuilder().Set("page", "bg.html")))))
+      .SetManifest(DictionaryBuilder()
+                       .Set("name", "test")
+                       .Set("version", "0.1")
+                       .Set("background",
+                            DictionaryBuilder().Set("page", "bg.html").Build())
+                       .Build())
       .SetID("id2")
       .Build();
 }