Expose way to set google api key through CWVWebView class method.

google_apis/google_api_keys.h

 // Copyright (c) 2012 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 GOOGLE_APIS_GOOGLE_API_KEYS_H_
 #define GOOGLE_APIS_GOOGLE_API_KEYS_H_
 
 // If you add more includes to this file, you also need to add them to
 // google_api_keys_unittest.cc.
 #include <string>
@@ skipping 55 matching lines @@
 //
 // Note that the key should be escaped for the context you use it in,
 // e.g. URL-escaped if you use it in a URL.
 std::string GetAPIKey();
 
 // Non-stable channels may have a different Google API key.
 std::string GetNonStableAPIKey();
 
 std::string GetRemotingAPIKey();
 
+// Sets the API key.
+void SetAPIKey(std::string api_key);

[Hiroshi Ichikawa, 2017/04/13 06:42:55]

Can you document that this must be called only before running any code using the
API key? I believe sdefresne@ or someone said that many code assumes the API key
is immutable. Same for below.

[Hiroshi Ichikawa, 2017/04/13 06:42:55]

const std::string& would be better. Same for below.

[jzw1, 2017/04/14 02:10:32]

Done.

[jzw1, 2017/04/14 02:10:32]

Done.

+
 // Represents the different sets of client IDs and secrets in use.
 enum OAuth2Client {
   CLIENT_MAIN,         // Several different features use this.
   CLIENT_CLOUD_PRINT,
   CLIENT_REMOTING,
   CLIENT_REMOTING_HOST,
 
   CLIENT_NUM_ITEMS     // Must be last item.
 };
 
 // Retrieves the OAuth2 client ID for the specified client, or the
 // empty string if not set.
 //
 // Note that the ID should be escaped for the context you use it in,
 // e.g. URL-escaped if you use it in a URL.
 std::string GetOAuth2ClientID(OAuth2Client client);
 
 // Retrieves the OAuth2 client secret for the specified client, or the
 // empty string if not set.
 //
 // Note that the secret should be escaped for the context you use it
 // in, e.g. URL-escaped if you use it in a URL.
 std::string GetOAuth2ClientSecret(OAuth2Client client);
 
+// Sets the client id for the specified client.
+void SetClientID(std::string client_id, OAuth2Client client);

[Hiroshi Ichikawa, 2017/04/13 06:42:55]

SetOAuth2ClientID would be better to be consistent with other function names
around. Same for below.

[jzw1, 2017/04/14 02:10:32]

Done.

+
+// Sets the client secret for the specified client.
+void SetClientSecret(std::string client_secret, OAuth2Client client);

[Roger Tawa OOO till Jul 10th, 2017/04/13 17:42:43]

As discussed over email, let's put all these SetXXX methods inside #ifdef for
iOS only.

Nit: could we reverse the order of the two args, since it is a mapping from
client --> client_id/client_secret?

Let's also rename the setters to be consistent with the getters.  So:

void SetOAuth2ClientID(...)
void SetOAuth2ClientSecret(...)

[jzw1, 2017/04/14 02:10:32]

Done.

+
 // Returns the auth token for the data reduction proxy.
 std::string GetSpdyProxyAuthValue();
 
 // Returns if the API key using in the current build is the one for official
 // Google Chrome.
 bool IsGoogleChromeAPIKeyUsed();
 
 }  // namespace google_apis
 
 #endif  // GOOGLE_APIS_GOOGLE_API_KEYS_H_