Changing Option String Resource to lowerCamelcases in handler_options_handler.cc file.

chrome/browser/ui/webui/options/clear_browser_data_handler.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 "chrome/browser/ui/webui/options/clear_browser_data_handler.h"
 
 #include "base/basictypes.h"
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/prefs/pref_service.h"
@@ skipping 63 matching lines @@
   args.AppendBoolean(visible);
   web_ui()->CallJavascriptFunction(
       "ClearBrowserDataOverlay.setBannerVisibility", args);
 }
 
 void ClearBrowserDataHandler::GetLocalizedValues(
     base::DictionaryValue* localized_strings) {
   DCHECK(localized_strings);
 
   static OptionsStringResource resources[] = {
-    { "clearBrowserDataLabel", IDS_CLEAR_BROWSING_DATA_LABEL },
-    { "contentSettingsAndSearchEnginesRemain",
-      IDS_CLEAR_BROWSING_DATA_SOME_STUFF_REMAINS },
-    { "deleteBrowsingHistoryCheckbox", IDS_DEL_BROWSING_HISTORY_CHKBOX },
-    { "deleteDownloadHistoryCheckbox", IDS_DEL_DOWNLOAD_HISTORY_CHKBOX },
-    { "deleteCacheCheckbox", IDS_DEL_CACHE_CHKBOX },
-    { "deleteCookiesCheckbox", IDS_DEL_COOKIES_CHKBOX },
-    { "deleteCookiesFlashCheckbox", IDS_DEL_COOKIES_FLASH_CHKBOX },
-    { "deletePasswordsCheckbox", IDS_DEL_PASSWORDS_CHKBOX },
-    { "deleteFormDataCheckbox", IDS_DEL_FORM_DATA_CHKBOX },
-    { "deleteHostedAppsDataCheckbox", IDS_DEL_HOSTED_APPS_DATA_CHKBOX },
-    { "deauthorizeContentLicensesCheckbox",
-      IDS_DEAUTHORIZE_CONTENT_LICENSES_CHKBOX },
-    { "clearBrowserDataCommit", IDS_CLEAR_BROWSING_DATA_COMMIT },
-    { "flash_storage_url", IDS_FLASH_STORAGE_URL },
+      {"clearBrowserDataLabel", IDS_CLEAR_BROWSING_DATA_LABEL},

[Bernhard Bauer, 2015/04/07 09:49:35]

I think I would prefer to keep the space after the opening brace. It also means
that broken lines are indented with two spaces, which looks a bit nicer.

[Deepak, 2015/04/07 11:25:13]

On 2015/04/07 09:49:35, Bernhard Bauer wrote:

Done.

[Bernhard Bauer, 2015/04/07 11:36:51]

You changed it in the other file, but not in this one...

+      {"contentSettingsAndSearchEnginesRemain",
+       IDS_CLEAR_BROWSING_DATA_SOME_STUFF_REMAINS},
+      {"deleteBrowsingHistoryCheckbox", IDS_DEL_BROWSING_HISTORY_CHKBOX},
+      {"deleteDownloadHistoryCheckbox", IDS_DEL_DOWNLOAD_HISTORY_CHKBOX},
+      {"deleteCacheCheckbox", IDS_DEL_CACHE_CHKBOX},
+      {"deleteCookiesCheckbox", IDS_DEL_COOKIES_CHKBOX},
+      {"deleteCookiesFlashCheckbox", IDS_DEL_COOKIES_FLASH_CHKBOX},
+      {"deletePasswordsCheckbox", IDS_DEL_PASSWORDS_CHKBOX},
+      {"deleteFormDataCheckbox", IDS_DEL_FORM_DATA_CHKBOX},
+      {"deleteHostedAppsDataCheckbox", IDS_DEL_HOSTED_APPS_DATA_CHKBOX},
+      {"deauthorizeContentLicensesCheckbox",
+       IDS_DEAUTHORIZE_CONTENT_LICENSES_CHKBOX},
+      {"clearBrowserDataCommit", IDS_CLEAR_BROWSING_DATA_COMMIT},
+      {"flashStorageUrl", IDS_FLASH_STORAGE_URL},
   };
 
   RegisterStrings(localized_strings, resources, arraysize(resources));
   RegisterTitle(localized_strings, "clearBrowserDataOverlay",
                 IDS_CLEAR_BROWSING_DATA_TITLE);
   localized_strings->SetString("clearBrowsingDataLearnMoreUrl",
                                kClearBrowsingDataLearnMoreUrl);
 
   ui::Accelerator acc(ui::VKEY_N, ui::EF_CONTROL_DOWN | ui::EF_SHIFT_DOWN);
   localized_strings->SetString(
@@ skipping 103 matching lines @@
   web_ui()->CallJavascriptFunction("ClearBrowserDataOverlay.doneClearing");
 }
 
 void ClearBrowserDataHandler::OnBrowsingHistoryPrefChanged() {
   web_ui()->CallJavascriptFunction(
       "ClearBrowserDataOverlay.setAllowDeletingHistory",
       base::FundamentalValue(*allow_deleting_browser_history_));
 }
 
 }  // namespace options