Revert 137554 - convert help page to loadTimeData

BUG=122753
TEST=manual


Review URL: https://chromiumcodereview.appspot.com/10386076

TBR=estade@chromium.org
Review URL: https://chromiumcodereview.appspot.com/10383228

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@137581 0039d316-1c4b-4281-b951-d872f2087c98

chrome/browser/resources/help/help.html

@@ -7,7 +7,7 @@
   <link rel="stylesheet" href="../uber/uber_shared.css">
   <link rel="stylesheet" href="help.css">
   <script src="chrome://resources/js/cr.js"></script>
-  <script src="chrome://resources/js/load_time_data.js"></script>
+  <script src="chrome://resources/js/local_strings.js"></script>
   <script src="chrome://resources/js/util.js"></script>
   <script src="chrome://help-frame/help.js"></script>
 </head>
@@ -97,5 +97,6 @@
   </div>
 </body>
 <script src="chrome://help-frame/strings.js"></script>
-<script src="chrome://resources/js/i18n_template2.js"></script>
+<script src="chrome://resources/js/i18n_template.js"></script>
+<script src="chrome://resources/js/i18n_process.js"></script>
 </html>

chrome/browser/resources/help/help.js

@@ -5,6 +5,8 @@
 <include src="../uber/uber_utils.js">
 
 cr.define('help', function() {
+  var localStrings = new LocalStrings();
+
   /**
    * Encapsulated handling of the help page.
    */
@@ -22,18 +24,17 @@ cr.define('help', function() {
       uber.onContentFrameLoaded();
 
       // Set the title.
-      var title = loadTimeData.getString('helpTitle');
+      var title = localStrings.getString('helpTitle');
       uber.invokeMethodOnParent('setTitle', {title: title});
 
-      $('product-license').innerHTML = loadTimeData.getString('productLicense');
-      if (cr.isChromeOS) {
+      $('product-license').innerHTML = localStrings.getString('productLicense');
+      if (cr.isChromeOS)
         $('product-os-license').innerHTML =
-            loadTimeData.getString('productOsLicense');
-      }
+            localStrings.getString('productOsLicense');
 
       var productTOS = $('product-tos');
       if (productTOS)
-        productTOS.innerHTML = loadTimeData.getString('productTOS');
+        productTOS.innerHTML = localStrings.getString('productTOS');
 
       $('get-help').onclick = chrome.send.bind(chrome, 'openHelpPage');
       $('report-issue').onclick =
@@ -70,8 +71,8 @@ cr.define('help', function() {
       moreInfo.style.height = visible ? '' : moreInfo.scrollHeight + 'px';
       moreInfo.addEventListener('webkitTransitionEnd', function(event) {
         $('more-info-expander').textContent = visible ?
-            loadTimeData.getString('showMoreInfo') :
-            loadTimeData.getString('hideMoreInfo');
+            localStrings.getString('showMoreInfo') :
+                localStrings.getString('hideMoreInfo');
       });
     },
 
@@ -98,17 +99,17 @@ cr.define('help', function() {
       if (status == 'checking') {
         this.setUpdateImage_('working');
         $('update-status').innerHTML =
-            loadTimeData.getString('updateCheckStarted');
+            localStrings.getString('updateCheckStarted');
       } else if (status == 'updating') {
         this.setUpdateImage_('working');
-        $('update-status').innerHTML = loadTimeData.getString('updating');
+        $('update-status').innerHTML = localStrings.getString('updating');
       } else if (status == 'nearly_updated') {
         this.setUpdateImage_('up-to-date');
         $('update-status').innerHTML =
-            loadTimeData.getString('updateAlmostDone');
+            localStrings.getString('updateAlmostDone');
       } else if (status == 'updated') {
         this.setUpdateImage_('up-to-date');
-        $('update-status').innerHTML = loadTimeData.getString('upToDate');
+        $('update-status').innerHTML = localStrings.getString('upToDate');
       } else if (status == 'failed') {
         this.setUpdateImage_('failed');
         $('update-status').innerHTML = message;

chrome/browser/ui/webui/help/help_browsertest.js

-// 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.
-
-/**
- * TestFixture for extension settings WebUI testing.
- * @extends {testing.Test}
- * @constructor
- */
-function HelpPageWebUITest() {}
-
-HelpPageWebUITest.prototype = {
-  __proto__: testing.Test.prototype,
-
-  browsePreload: 'chrome://help-frame/',
-};
-
-// Test opening extension settings has correct location.
-TEST_F('HelpPageWebUITest', 'testOpenHelpPage', function() {
-  assertEquals(this.browsePreload, document.location.href);
-});

chrome/browser/ui/webui/help/help_ui.cc

@@ -21,7 +21,6 @@ ChromeWebUIDataSource* CreateAboutPageHTMLSource() {
       new ChromeWebUIDataSource(chrome::kChromeUIHelpFrameHost);
 
   source->set_json_path("strings.js");
-  source->set_use_json_js_format_v2();
   source->add_resource_path("help.js", IDR_HELP_JS);
   source->set_default_resource(IDR_HELP_HTML);
   return source;

chrome/chrome_tests.gypi

@@ -2855,7 +2855,6 @@
         'browser/ui/views/sync/one_click_signin_bubble_view_browsertest.cc',
         'browser/ui/views/web_dialog_view_browsertest.cc',
         'browser/ui/webui/chrome_url_data_manager_browsertest.cc',
-        'browser/ui/webui/help/help_browsertest.js',
         'browser/ui/webui/ntp/most_visited_browsertest.cc',
         'browser/ui/webui/test_chrome_web_ui_controller_factory_browsertest.cc',
         'browser/ui/webui/bidi_checker_web_ui_test.cc',