Add Locale Request Support to Chrome Now

chrome/browser/resources/google_now/background.js

Index: chrome/browser/resources/google_now/background.js
diff --git a/chrome/browser/resources/google_now/background.js b/chrome/browser/resources/google_now/background.js
index fc487cf10cf2ff0f18fc88031dc66cefdd52639f..032df708ea2a3b9798e31f82b1d8728b629a41bc 100644
--- a/chrome/browser/resources/google_now/background.js
+++ b/chrome/browser/resources/google_now/background.js
@@ -582,6 +582,8 @@ function requestNotificationGroups(groupNames) {
     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);