Update Autofill Private API to provide a list of available countries.

chrome/browser/extensions/api/autofill_private/autofill_private_apitest.cc

 // 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.
 
 #include "base/command_line.h"
 #include "base/macros.h"
 #include "base/values.h"
 #include "chrome/browser/extensions/extension_apitest.h"
 #include "chrome/common/extensions/api/autofill_private.h"
 #include "components/keyed_service/core/keyed_service.h"
@@ skipping 28 matching lines @@
  private:
   DISALLOW_COPY_AND_ASSIGN(AutofillPrivateApiTest);
 };
 
 }  // namespace
 
 IN_PROC_BROWSER_TEST_F(AutofillPrivateApiTest, SaveAddress) {
   EXPECT_TRUE(RunAutofillSubtest("saveAddress")) << message_;
 }
 
+IN_PROC_BROWSER_TEST_F(AutofillPrivateApiTest, GetCountryList) {
+  EXPECT_TRUE(RunAutofillSubtest("getCountryList")) << message_;

[Devlin, 2016/06/10 18:25:11]

semi-drive-by - do all these really need separate browser tests?  Browser tests
are expensive, and most extension APIs are designed to be able to run multiple
tests in a single browsertest to avoid startup/shutdown costs.  Alternatively,
and even better, it looks like a lot of these could just be unittests.

I don't wanna block this CL on that (unless you're up for it ;)), but as food
for thought, and maybe a TODO.

[hcarmona, 2016/06/10 20:42:51]

Sounds good, I've added a TODO at the top of this file to investigate
converting to unittests.

+}
+
 IN_PROC_BROWSER_TEST_F(AutofillPrivateApiTest, GetAddressComponents) {
   EXPECT_TRUE(RunAutofillSubtest("getAddressComponents")) << message_;
 }
 
 IN_PROC_BROWSER_TEST_F(AutofillPrivateApiTest, SaveCreditCard) {
   EXPECT_TRUE(RunAutofillSubtest("saveCreditCard")) << message_;
 }
 
 IN_PROC_BROWSER_TEST_F(AutofillPrivateApiTest, RemoveEntry) {
   EXPECT_TRUE(RunAutofillSubtest("removeEntry")) << message_;
 }
 
 IN_PROC_BROWSER_TEST_F(AutofillPrivateApiTest, ValidatePhoneNumbers) {
   EXPECT_TRUE(RunAutofillSubtest("ValidatePhoneNumbers")) << message_;
 }
 
 }  // namespace extensions