Fix else-after-return in base

$ ninja -C out/Release base
$ tools/clang/scripts/generate_compdb.py -p out/Release \
  > out/Release/compile_commands.json
$ cd out/Release
$ ~/dev/llvm/llvm/tools/clang/tools/extra/clang-tidy/tool/run-clang-tidy.py \
  -p . -clang-tidy-binary ~/dev/llvm/build/bin/clang-tidy \
  -clang-apply-replacements-binary \
  ~/dev/llvm/build/bin/clang-apply-replacements
  -checks=readability-else-after-return -fix ~/dev/chromium_official/src/base/
$ cd ../..
$ git checkout base/third_party
$ git cl format

R=thestig

Change-Id: Ifc44900f372ed4027027ba50f7a7de22d0e2f0ec
Reviewed-on: https://chromium-review.googlesource.com/1214691
Commit-Queue: Thomas Anderson <thomasanderson@chromium.org>
Reviewed-by: Lei Zhang <thestig@chromium.org>
Cr-Commit-Position: refs/heads/master@{#590385}

• base/allocator/allocator_shim_unittest.cc[diff]
• base/allocator/partition_allocator/partition_alloc.cc[diff]
• base/allocator/tcmalloc_unittest.cc[diff]
• base/i18n/rtl.cc[diff]
• base/i18n/string_search.cc[diff]
• base/i18n/time_formatting.cc[diff]
• base/json/json_file_value_serializer.cc[diff]
• base/json/json_parser.cc[diff]
• base/json/json_value_converter_unittest.cc[diff]
• base/message_loop/message_loop_unittest.cc[diff]
• base/metrics/persistent_memory_allocator.cc[diff]
• base/process/kill_posix.cc[diff]
• base/process/launch_posix.cc[diff]
• base/sha1.cc[diff]
• base/strings/safe_sprintf.cc[diff]
• base/test/trace_event_analyzer.cc[diff]
• base/time/time_exploded_posix.cc[diff]
• base/time/time_now_posix.cc[diff]
• base/values.cc[diff]