Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(5799)

Unified Diff: chrome/browser/resources/about_conflicts.js

Issue 2939273002: DO NOT SUBMIT: what chrome/browser/resources/ could eventually look like with clang-format (Closed)
Patch Set: Created 3 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/resources/about_conflicts.js
diff --git a/chrome/browser/resources/about_conflicts.js b/chrome/browser/resources/about_conflicts.js
index e8eb92ea557131d30b6dc87642165ee3174706b6..08984983bd1f6c1b51e9cb1ecdeeceef6fee2957 100644
--- a/chrome/browser/resources/about_conflicts.js
+++ b/chrome/browser/resources/about_conflicts.js
@@ -3,15 +3,14 @@
// found in the LICENSE file.
/**
-* This variable structure is here to document the structure that the template
-* expects to correctly populate the page.
-*/
+ * This variable structure is here to document the structure that the template
+ * expects to correctly populate the page.
+ */
var moduleListDataFormat = {
- 'moduleList': [
- {
+ 'moduleList': [{
'type': 'The type of module found',
'type_description':
- 'The type of module (string), defaults to blank for regular modules',
+ 'The type of module (string), defaults to blank for regular modules',
'status': 'The module status',
'location': 'The module path, not including filename',
'name': 'The name of the module',
@@ -22,16 +21,15 @@ var moduleListDataFormat = {
'recommended_action': 'The help tips bitmask',
'possible_resolution': 'The help tips in string form',
'help_url': 'The link to the Help Center article'
- }
-]
+ }]
};
/**
-* Takes the |moduleListData| input argument which represents data about
-* the currently available modules and populates the html jstemplate
-* with that data. It expects an object structure like the above.
-* @param {Object} moduleListData Information about available modules.
-*/
+ * Takes the |moduleListData| input argument which represents data about
+ * the currently available modules and populates the html jstemplate
+ * with that data. It expects an object structure like the above.
+ * @param {Object} moduleListData Information about available modules.
+ */
function renderTemplate(moduleListData) {
// This is the javascript code that processes the template:
var input = new JsEvalContext(moduleListData);
@@ -40,19 +38,19 @@ function renderTemplate(moduleListData) {
}
/**
-* Asks the C++ ConflictsDOMHandler to get details about the available modules
-* and return detailed data about the configuration. The ConflictsDOMHandler
-* should reply to returnModuleList() (below).
-*/
+ * Asks the C++ ConflictsDOMHandler to get details about the available modules
+ * and return detailed data about the configuration. The ConflictsDOMHandler
+ * should reply to returnModuleList() (below).
+ */
function requestModuleListData() {
chrome.send('requestModuleList');
}
/**
-* Called by the WebUI to re-populate the page with data representing the
-* current state of installed modules.
-* @param {Object} moduleListData Information about available modules.
-*/
+ * Called by the WebUI to re-populate the page with data representing the
+ * current state of installed modules.
+ * @param {Object} moduleListData Information about available modules.
+ */
function returnModuleList(moduleListData) {
renderTemplate(moduleListData);
$('loading-message').style.visibility = 'hidden';
« no previous file with comments | « no previous file | chrome/browser/resources/about_flash.js » ('j') | chrome/browser/resources/settings/device_page/stylus.js » ('J')

Powered by Google App Engine
This is Rietveld 408576698