| Index: chrome/common/extensions/docs/extensions/contentSecurityPolicy.html
|
| diff --git a/chrome/common/extensions/docs/extensions/contentSecurityPolicy.html b/chrome/common/extensions/docs/extensions/contentSecurityPolicy.html
|
| index 38be33802df67c2f74189df6c07358bb2e765560..288ceefd8c6d825d18bdd2d7d5cdad2ceee19dd8 100644
|
| --- a/chrome/common/extensions/docs/extensions/contentSecurityPolicy.html
|
| +++ b/chrome/common/extensions/docs/extensions/contentSecurityPolicy.html
|
| @@ -295,9 +295,12 @@
|
| function clickHandler(element) {
|
| setTimeout(<strong>"awesome(); totallyAwesome()"</strong>, 1000);
|
| }
|
| + function main() {
|
| + // Initialization work goes here.
|
| + }
|
| </script>
|
| </head>
|
| - <body>
|
| + <body onload="main();">
|
| <button <strong>onclick="clickHandler(this)"</strong>>
|
| Click for awesomeness!
|
| </button>
|
| @@ -313,8 +316,13 @@
|
| JavaScript file (<code>popup.js</code> would be a good target).
|
| </li>
|
| <li>
|
| - The inline event handler definition must be rewritten in terms of
|
| - <code>addEventListener</code> and extracted into <code>popup.js</code>.
|
| + <p>The inline event handler definitions must be rewritten in terms of
|
| + <code>addEventListener</code> and extracted into <code>popup.js</code>.</p>
|
| + <p>If you're currently kicking off your program's execution via code like
|
| + <code><body onload="main();"></code>, consider replacing it by hooking
|
| + into the document's <code>DOMContentLoaded</code> event, or the window's
|
| + <code>load</code> event, depending on your needs. Below we'll use the
|
| + former, as it generally triggers more quickly.</p>
|
| </li>
|
| <li>
|
| The <code>setTimeout</code> call will need to be rewritten to avoid
|
| @@ -342,13 +350,18 @@ function awesomeTask() {
|
| function clickHandler(e) {
|
| setTimeout(<strong>awesomeTask</strong>, 1000);
|
| }
|
| +function main() {
|
| + // Initialization work goes here.
|
| +}
|
| // Add event listeners once the DOM has fully loaded by listening for the
|
| // `DOMContentLoaded` event on the document, and adding your listeners to
|
| // specific elements when it triggers.
|
| -document.addEventListener('DOMContentLoaded', function () {
|
| +<strong>document.addEventListener('DOMContentLoaded', function () {</strong>
|
| document.querySelector('button').addEventListener('click', clickHandler);
|
| + main();
|
| });
|
| -popup.html:
|
| +</pre>
|
| +<pre>popup.html:
|
| ===========
|
| <!doctype html>
|
| <html>
|
|
|
Chromium Code Reviews
Issue 10832110:
Docs: Clarifying the CSP-friendly replacement for <body onload="XXX">. (Closed)
Base URL: svn://svn.chromium.org/chrome/trunk/src