Cleanup: Better constify some strings in chrome/browser/{chromeos,extensions}.

chrome/browser/extensions/extension_messages_apitest.cc

 // 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.
 
 #include "base/base64.h"
 #include "base/files/file_path.h"
 #include "base/json/json_reader.h"
 #include "base/json/json_writer.h"
 #include "base/path_service.h"
 #include "base/strings/string_number_conversions.h"
@@ skipping 195 matching lines @@
   testing::AssertionResult AreAnyNonWebApisDefinedForIFrame() {
     content::RenderFrameHost* frame = content::FrameMatchingPredicate(
         browser()->tab_strip_model()->GetActiveWebContents(),
         base::Bind(&content::FrameIsChildOfMainFrame));
     return AreAnyNonWebApisDefinedForFrame(frame);
   }
 
   testing::AssertionResult AreAnyNonWebApisDefinedForFrame(
       content::RenderFrameHost* frame) {
     // All runtime API methods are non-web except for sendRequest and connect.
-    const char* non_messaging_apis[] = {
+    const char* const non_messaging_apis[] = {
         "getBackgroundPage",
         "getManifest",
         "getURL",
         "reload",
         "requestUpdateCheck",
         "restart",
         "connectNative",
         "sendNativeMessage",
         "onStartup",
         "onInstalled",
@@ skipping 388 matching lines @@
   // closes upon receipt of the connect.
   std::string tls_channel_id = GetTlsChannelIdFromPortConnect(
       chromium_connectable.get(), true, close_background_message());
   // Because the TLS channel ID has never been generated for this domain,
   // no TLS channel ID is reported.
   EXPECT_EQ(std::string(), tls_channel_id);
   // A subsequent connect will still succeed, even if the background page was
   // previously closed.
   tls_channel_id =
       GetTlsChannelIdFromPortConnect(chromium_connectable.get(), true);
-   // And the empty value is still retrieved.
+  // And the empty value is still retrieved.
   EXPECT_EQ(std::string(), tls_channel_id);
 }
 
 // Tests that enabling and disabling an extension makes the runtime bindings
 // appear and disappear.
 //
 // TODO(kalman): Test with multiple extensions that can be accessed by the same
 // host.
 IN_PROC_BROWSER_TEST_F(ExternallyConnectableMessagingTest,
                        EnablingAndDisabling) {
@@ skipping 344 matching lines @@
   ui_test_utils::NavigateToURL(browser(), chromium_org_url());
   // If the page does ask for it, it isn't empty, even if the background page
   // closes upon receipt of the connect.
   std::string tls_channel_id = GetTlsChannelIdFromPortConnect(
       chromium_connectable.get(), true, close_background_message());
   EXPECT_EQ(expected_tls_channel_id_value, tls_channel_id);
   // A subsequent connect will still succeed, even if the background page was
   // previously closed.
   tls_channel_id =
       GetTlsChannelIdFromPortConnect(chromium_connectable.get(), true);
-   // And the expected value is still retrieved.
+  // And the expected value is still retrieved.
   EXPECT_EQ(expected_tls_channel_id_value, tls_channel_id);
 }
 
 IN_PROC_BROWSER_TEST_F(ExtensionApiTest, MessagingUserGesture) {
   const char kManifest[] = "{"
                           "  \"name\": \"user_gesture\","
                           "  \"version\": \"1.0\","
                           "  \"background\": {"
                           "    \"scripts\": [\"background.js\"]"
                           "  },"
@@ skipping 80 matching lines @@
   ui_test_utils::NavigateToURL(browser(), chromium_org_url());
   EXPECT_EQ(COULD_NOT_ESTABLISH_CONNECTION_ERROR,
             CanConnectAndSendMessagesToMainFrame(invalid.get()));
 }
 
 #endif  // !defined(OS_WIN) - http://crbug.com/350517.
 
 }  // namespace
 
 };  // namespace extensions