Remove NOTREACHED handling in base::CpuInfoBrand().

If the not reachable code handling code gets removed, the rest of the
function works just fine and correctly returns an empty string.

Change-Id: Ia41522564ff242c097463ec804cf868a7b4f7784
Reviewed-on: https://chromium-review.googlesource.com/792298Reviewed-by: David Trainor <dtrainor@chromium.org>
Commit-Queue: Lei Zhang <thestig@chromium.org>
Cr-Commit-Position: refs/heads/master@{#519826}

base/cpu.cc

@@ -107,9 +107,6 @@ std::string* CpuInfoBrand() {
     std::string contents;
     ReadFileToString(FilePath("/proc/cpuinfo"), &contents);
     DCHECK(!contents.empty());
-    if (contents.empty()) {
-      return new std::string();
-    }
 
     std::istringstream iss(contents);
     std::string line;