Set the Accept-Language header in Physical Web requests

ios/chrome/common/physical_web/physical_web_request.mm

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #import "ios/chrome/common/physical_web/physical_web_request.h"
 
 #include "base/ios/weak_nsobject.h"
-#include "base/logging.h"
 #include "base/mac/foundation_util.h"
 #include "base/mac/scoped_block.h"
 #include "base/mac/scoped_nsobject.h"
 #include "base/metrics/histogram.h"
 #include "base/strings/sys_string_conversions.h"
 #include "components/version_info/version_info.h"
 #include "google_apis/google_api_keys.h"
 #import "ios/chrome/common/physical_web/physical_web_device.h"
 #import "ios/chrome/common/physical_web/physical_web_types.h"
 #include "ios/web/public/user_agent.h"
@@ skipping 43 matching lines @@
 
 - (instancetype)initWithDevice:(PhysicalWebDevice*)device {
   self = [super init];
   if (self) {
     device_.reset([device retain]);
   }
   return self;
 }
 
 - (instancetype)init {
   NOTREACHED();

[Olivier, 2016/06/09 07:47:31]

This is defined int base/logging.
Please let the include.

[mattreynolds, 2016/06/09 19:43:57]

Oops, re-added

   return nil;
 }
 
 - (void)cancel {
   [urlSessionTask_ cancel];
   block_.reset();
 }
 
 - (void)start:(physical_web::RequestFinishedBlock)block {
   block_.reset([block copy]);
@@ skipping 14 matching lines @@
   // body of the POST request.
   NSDictionary* jsonBody =
       @{ kUrlsKey : @[ @{kUrlKey : [[device_ url] absoluteString]} ] };
   [request_ setHTTPBody:[NSJSONSerialization dataWithJSONObject:jsonBody
                                                         options:0
                                                           error:NULL]];
   [request_ setValue:@"application/json" forHTTPHeaderField:@"Content-Type"];
   [request_ setValue:base::SysUTF8ToNSString(GetUserAgent())
       forHTTPHeaderField:@"User-Agent"];
 
+  // Set the Accept-Language header from the locale language code. This may
+  // cause us to fetch metadata for the wrong region in languages such as
+  // Chinese that vary significantly between regions.
+  // TODO(mattreynolds): Use the same Accept-Language string as WKWebView.
+  NSString* acceptLanguage =
+      [[NSLocale currentLocale] objectForKey:NSLocaleLanguageCode];
+  [request_ setValue:acceptLanguage forHTTPHeaderField:@"Acccept-Language"];
+
   startDate_.reset([[NSDate date] retain]);
   // Starts the request.
   NSURLSession* session =
       [NSURLSession sessionWithConfiguration:[NSURLSessionConfiguration
                                                  ephemeralSessionConfiguration]
                                     delegate:nil
                                delegateQueue:[NSOperationQueue mainQueue]];
   base::WeakNSObject<PhysicalWebRequest> weakSelf(self);
   SessionCompletionProceduralBlock completionHandler =
       ^(NSData* data, NSURLResponse* response, NSError* error) {
@@ skipping 121 matching lines @@
         transmitPower:[device_ transmitPower]
                  rssi:[device_ rssi]
                  rank:physical_web::kMaxRank]);
     if (block_.get() != nil) {
       block_.get()(device, nil);
     }
   }
 }
 
 @end