Add Locale Request Support to Chrome Now

chrome/browser/resources/google_now/background.js

 // Copyright (c) 2013 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.
 
 'use strict';
 
 /**
  * @fileoverview The event page for Google Now for Chrome implementation.
  * The Google Now event page gets Google Now cards from the server and shows
  * them as Chrome notifications.
@@ skipping 564 matching lines @@
       localStorage['explanatoryCardsShown'] < EXPLANATORY_CARDS_LINK_THRESHOLD;
   if (belowExplanatoryThreshold) {
     requestParameters += '&cardExplanation=true';
     cardShownCallback = countExplanatoryCard;
   }
 
   groupNames.forEach(function(groupName) {
     requestParameters += ('&requestTypes=' + groupName);
   });
 
+  requestParameters += '&uiLocale=' + navigator.language;

[rgustafson, 2014/02/19 19:18:14]

https://developer.mozilla.org/en-US/docs/Web/API/NavigatorLanguage.language

This says it's based on acceptlanguage? Have you tested this? It might just be
out of date.

[robliao, 2014/02/19 19:26:47]

Works for me.

accept-language:en-US,en;q=0.8,de;q=0.6,de-DE;q=0.4,zh-TW;q=0.2,zh;q=0.2,ko;q=0.2

navigator.language = "ko"

Current UI language = "ko"

On 2014/02/19 19:18:14, rgustafson wrote:

[rgustafson, 2014/02/19 19:39:16]

Do you know the actual standard that these strings follow is since apparently
the documentation isn't that great? From what I gather it's RFC 5646, but this
is important so we don't choke on parsing this serverside.

[robliao, 2014/02/19 19:49:19]

A bit more context. The reason why this works is because we get no preferred
languages while getting navigator.language. Haven't dug into why.

This causes us to fall into getting the command line langauge

3:035> k20
ChildEBP RetAddr  
0018c094 11b9c660 content!content::RenderThreadImpl::GetLocale+0xb5
0018c120 1b3c70e6
content!content::RendererWebKitPlatformSupportImpl::defaultLocale+0x60
0018c168 1b3c6f2c blink_platform!WebCore::platformLanguage+0xf6
0018c19c 1b3c6c0d blink_platform!WebCore::userPreferredLanguages+0x9c
0018c1d4 165b36ab blink_platform!WebCore::defaultLanguage+0x4d
0018c1ec 1522a0b2 blink_web!WebCore::Navigator::language+0x2b
0018c228 1522a019
blink_web!WebCore::NavigatorV8Internal::languageAttributeGetter+0x72
0018c234 0f905534
blink_web!WebCore::NavigatorV8Internal::languageAttributeGetterCallback+0x19
0018c270 0f6c0f44 v8!v8::internal::PropertyCallbackArguments::Call+0x74

Sure enough, our command line has
CommandLine:  '"D:\src\out\Debug\chrome.exe" --type=renderer --lang=ko
--force-fieldtrials=[Field Trials] --extension-process
--enable-threaded-compositing --enable-delegated-renderer
--enable-deadline-scheduling --enable-software-compositing --channel=[Channel]
    
So who sets --lang?

That gets set up here:
const CommandLinePrefStore::StringSwitchToPreferenceMapEntry
    CommandLinePrefStore::string_switch_map_[] = {
      { switches::kLang, prefs::kApplicationLocale }

On 2014/02/19 19:39:16, rgustafson wrote:

+
   console.log('requestNotificationGroups: request=' + requestParameters);
 
   var request = buildServerRequest('GET', 'notifications' + requestParameters);
 
   request.onloadend = function(event) {
     console.log('requestNotificationGroups-onloadend ' + request.status);
     if (request.status == HTTP_OK) {
       recordEvent(GoogleNowEvent.REQUEST_FOR_CARDS_SUCCESS);
       processServerResponse(
           JSON.parse(request.responseText), cardShownCallback);
@@ skipping 683 matching lines @@
             lastPollNowPayloads: items.lastPollNowPayloads,
             notificationGroups: items.notificationGroups
           });
 
           updateNotificationsCards();
         }
       });
     });
   }
 });