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

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

Issue 7277010: Apply CSP to about:conflicts page. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: '' Created 9 years, 5 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
« no previous file with comments | « chrome/browser/resources/about_conflicts.html ('k') | chrome/browser/ui/webui/conflicts_ui.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/resources/about_conflicts.js
===================================================================
--- chrome/browser/resources/about_conflicts.js (revision 0)
+++ chrome/browser/resources/about_conflicts.js (revision 0)
@@ -0,0 +1,59 @@
+/**
Evan Stade 2011/07/14 19:33:44 license header
Tom Sepez 2011/07/14 19:37:54 Done.
+* This variable structure is here to document the structure that the template
+* expects to correctly populate the page.
+*/
+var moduleListDataFormat = {
+ 'moduleList': [
+ {
+ 'type': 'The type of module found',
+ 'type_description':
+ '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',
+ 'product_name': 'The name of the product the module belongs to',
+ 'description': 'The module description',
+ 'version': 'The module version',
+ 'digital_signer': 'The signer of the digital certificate for the module',
+ '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
+*/
+function renderTemplate(moduleListData) {
+ // This is the javascript code that processes the template:
+ var input = new JsEvalContext(moduleListData);
+ var output = document.getElementById('modulesTemplate');
+ jstProcess(input, output);
+}
+
+/**
+* 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.
+*/
+function returnModuleList(moduleListData) {
+ renderTemplate(moduleListData);
+ document.getElementById('loading-message').style.visibility = 'hidden';
+ document.getElementById('body-container').style.visibility = 'visible';
+}
+
+// Get data and have it displayed upon loading.
+document.addEventListener('DOMContentLoaded', requestModuleListData);
+
Evan Stade 2011/07/14 19:33:44 extra line or rietveld error?
Tom Sepez 2011/07/14 19:37:54 Done.
Property changes on: chrome/browser/resources/about_conflicts.js
___________________________________________________________________
Added: svn:eol-style
+ LF
« no previous file with comments | « chrome/browser/resources/about_conflicts.html ('k') | chrome/browser/ui/webui/conflicts_ui.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698