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, 5 insertions, 4 deletions

diff --git a/extensions/browser/updater/update_service_unittest.cc b/extensions/browser/updater/update_service_unittest.cc
index 94bfad3..67c3087 100644
--- a/extensions/browser/updater/update_service_unittest.cc
+++ b/extensions/browser/updater/update_service_unittest.cc
@@ -183,10 +183,11 @@ TEST_F(UpdateServiceTest, BasicUpdateOperations) {
   ASSERT_TRUE(AddFileToDirectory(temp_dir.path(), bar_html, "world"));
 
   ExtensionBuilder builder;
-  builder.SetManifest(std::move(DictionaryBuilder()
-                                    .Set("name", "Foo")
-                                    .Set("version", "1.0")
-                                    .Set("manifest_version", 2)));
+  builder.SetManifest(DictionaryBuilder()
+                          .Set("name", "Foo")
+                          .Set("version", "1.0")
+                          .Set("manifest_version", 2)
+                          .Build());
   builder.SetID(crx_file::id_util::GenerateId("whatever"));
   builder.SetPath(temp_dir.path());