Allows to manually change the stored permanent country in Variations from the translate-internals d…

chrome/browser/translate/chrome_translate_client.h

 // Copyright 2014 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.
 
 #ifndef CHROME_BROWSER_TRANSLATE_CHROME_TRANSLATE_CLIENT_H_
 #define CHROME_BROWSER_TRANSLATE_CHROME_TRANSLATE_CLIENT_H_
 
 #include <string>
 
 #include "base/macros.h"
@@ skipping 38 matching lines @@
   // Returns the ContentTranslateDriver instance associated with this
   // WebContents.
   translate::ContentTranslateDriver& translate_driver() {
     return translate_driver_;
   }
 
   // Helper method to return a new TranslatePrefs instance.
   static scoped_ptr<translate::TranslatePrefs> CreateTranslatePrefs(
       PrefService* prefs);
 
+  // Helper method to force an override of
+  // kVariationsPermanentConsistencyCountry. Returns true if the variable has

[Alexei Svitkine (slow), 2016/04/20 20:30:23]

Nit: No need to say "Helper method". Also, the pref name is an implementation
detail.

Maybe describe the purpose instead:

"Overrides the stored permanent consistency country in prefs. This is intended
to be invoked from a debug UI to allow easy testing of different-country
configurations."

[hamelphi, 2016/04/20 21:39:46]

Done.

+  // been updated. Return false if the override country is the same as the
+  // stored variable, or if the update failed for any other reason.
+  static bool OverrideStoredPermanentCountry(const std::string& country);
+
   // Helper method to return the TranslateAcceptLanguages instance associated
   // with |browser_context|.
   static translate::TranslateAcceptLanguages* GetTranslateAcceptLanguages(
       content::BrowserContext* browser_context);
 
   // Helper method to return the TranslateManager instance associated with
   // |web_contents|, or NULL if there is no such associated instance.
   static translate::TranslateManager* GetManagerFromWebContents(
       content::WebContents* web_contents);
 
@@ skipping 45 matching lines @@
   void ShowBubble(translate::TranslateStep step,
                   translate::TranslateErrors::Type error_type);
 
   translate::ContentTranslateDriver translate_driver_;
   scoped_ptr<translate::TranslateManager> translate_manager_;
 
   DISALLOW_COPY_AND_ASSIGN(ChromeTranslateClient);
 };
 
 #endif  // CHROME_BROWSER_TRANSLATE_CHROME_TRANSLATE_CLIENT_H_