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

chrome/browser/extensions/api/sessions/sessions_apitest.cc

 // Copyright 2013 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 "chrome/browser/extensions/api/sessions/sessions_api.h"
 
 #include "base/command_line.h"
 #include "base/path_service.h"
 #include "base/strings/stringprintf.h"
 #include "chrome/browser/extensions/api/tabs/tabs_api.h"
@@ skipping 20 matching lines @@
 #include "chromeos/chromeos_switches.h"
 #endif
 
 namespace utils = extension_function_test_utils;
 
 namespace extensions {
 
 namespace {
 
 // If more sessions are added to session tags, num sessions should be updated.
-const char* kSessionTags[] = {"tag0", "tag1", "tag2", "tag3", "tag4"};
+const char* const kSessionTags[] = {"tag0", "tag1", "tag2", "tag3", "tag4"};
 const size_t kNumSessions = 5;
 
 void BuildSessionSpecifics(const std::string& tag,
                            sync_pb::SessionSpecifics* meta) {
   meta->set_session_tag(tag);
   sync_pb::SessionHeader* header = meta->mutable_header();
   header->set_device_type(sync_pb::SyncEnums_DeviceType_TYPE_LINUX);
   header->set_client_name(tag);
 }
 
@@ skipping 21 matching lines @@
   tab->set_current_navigation_index(0);
   tab->set_pinned(true);
   tab->set_extension_app_id("app_id");
   sync_pb::TabNavigation* navigation = tab->add_navigation();
   navigation->set_virtual_url("http://foo/1");
   navigation->set_referrer("referrer");
   navigation->set_title("title");
   navigation->set_page_transition(sync_pb::SyncEnums_PageTransition_TYPED);
 }
 
-} // namespace
+}  // namespace
 
 class ExtensionSessionsTest : public InProcessBrowserTest {
  public:
   void SetUpCommandLine(CommandLine* command_line) override;
   void SetUpOnMainThread() override;
 
  protected:
   static KeyedService* BuildProfileSyncService(
       content::BrowserContext* profile);
 
   void CreateTestProfileSyncService();
   void CreateTestExtension();
   void CreateSessionModels();
 
   template <class T>
   scoped_refptr<T> CreateFunction(bool has_callback) {
     scoped_refptr<T> fn(new T());
     fn->set_extension(extension_.get());
     fn->set_has_callback(has_callback);
     return fn;
-  };
+  }
 
   Browser* browser_;
   scoped_refptr<extensions::Extension> extension_;
 };
 
 void ExtensionSessionsTest::SetUpCommandLine(CommandLine* command_line) {
 #if defined(OS_CHROMEOS)
   command_line->AppendSwitch(
       chromeos::switches::kIgnoreUserProfileMappingForTests);
 #endif
@@ skipping 246 matching lines @@
   // Disable this test in Metro+Ash for now (http://crbug.com/262796).
   if (CommandLine::ForCurrentProcess()->HasSwitch(switches::kAshBrowserTests))
     return;
 #endif
 
   ASSERT_TRUE(RunExtensionSubtest("sessions",
                                   "sessions.html")) << message_;
 }
 
 }  // namespace extensions