Upgrade timezone from deprecated base::JSONReader API

Bug: 1070409
Change-Id: I5b2158a0af43c64055c3ea60dd74f490e579548f
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2222070Reviewed-by: James Cook <jamescook@chromium.org>
Commit-Queue: Nigel Tao <nigeltao@chromium.org>
Cr-Commit-Position: refs/heads/master@{#773912}

chromeos/timezone/timezone_request.cc

@@ -180,23 +180,23 @@ bool ParseServerResponse(const GURL& server_url,
           << response_body;
 
   // Parse the response, ignoring comments.
-  std::string error_msg;
-  std::unique_ptr<base::Value> response_value =
-      base::JSONReader::ReadAndReturnErrorDeprecated(
-          response_body, base::JSON_PARSE_RFC, NULL, &error_msg);
-  if (response_value == NULL) {
-    PrintTimeZoneError(server_url, "JSONReader failed: " + error_msg, timezone);
+  base::JSONReader::ValueWithError parsed_json =
+      base::JSONReader::ReadAndReturnValueWithError(response_body);
+  if (!parsed_json.value) {
+    PrintTimeZoneError(server_url,
+                       "JSONReader failed: " + parsed_json.error_message,
+                       timezone);
     RecordUmaEvent(TIMEZONE_REQUEST_EVENT_RESPONSE_MALFORMED);
     return false;
   }
 
   const base::DictionaryValue* response_object = NULL;
-  if (!response_value->GetAsDictionary(&response_object)) {
+  if (!parsed_json.value->GetAsDictionary(&response_object)) {
     PrintTimeZoneError(
         server_url,
         "Unexpected response type : " +
             base::StringPrintf(
-                "%u", static_cast<unsigned int>(response_value->type())),
+                "%u", static_cast<unsigned int>(parsed_json.value->type())),
         timezone);
     RecordUmaEvent(TIMEZONE_REQUEST_EVENT_RESPONSE_MALFORMED);
     return false;