Chromium Code Reviews Chromium Code Reviews
Issue 1114623002:
Plugin Power Saver: Make PPS work well with prerendered pages. (Closed)
Base URL: https://chromium.googlesource.com/chromium/src.git@master| OLD | NEW |
|---|---|
| (Empty) | |
| 1 <html> | |
| 2 <!-- | |
| 3 This test checks the interaction between prerendering and the Plugin Power Saver | |
| 4 feature. It expects: | |
| 5 - The plugin is not loaded during prerender. | |
| 6 - It is loaded (briefly) during the preroll of the plugin. | |
| 7 - It is replaced by the power saver placeholder after the preroll is over. | |
| 8 | |
| 9 This test relies on the Pepper plugin (but not the placeholder) implementing | |
| 10 the postMessage() function on the plugin object. | |
| 11 --> | |
| 12 <head> | |
| 13 <title>Prerender Plugin Delay Loading</title> | |
| 14 | |
| 15 <script> | |
| 16 // Make sure plugin was not loaded while prerendering. | |
| 17 function DidPrerenderPass() { | |
| 18 // |plugin| should be the blocked plugin placeholder, rather than the | |
| 19 // test plugin. | |
| 20 var plugin = window.document.getElementById('plugin'); | |
| 21 return plugin.postMessage === undefined; | |
| 22 } | |
| 23 | |
| 24 // Make sure the plugin briefly exists, then is replaced with a placeholder. | |
| 25 function AwaitPluginPrerollAndPlaceholder() { | |
| 26 var plugin = window.document.getElementById('plugin'); | |
| 27 function handleEvent(event) { | |
| 28 // We should eventually get a message indicating that the plugin has been | |
| 29 // throttled and replaced with a static placeholder. | |
| 30 if (event.data.isThrottled) { | |
|
Lei Zhang
2015/04/29 22:40:43
Do you need to handle the else case and send(false
tommycli
2015/04/29 22:44:19
Actually no. We expect there to be other notificat
| |
| 31 plugin.removeEventListener('message', handleEvent); | |
| 32 window.domAutomationController.send(plugin.postMessage === undefined); | |
| 33 } | |
| 34 } | |
| 35 | |
| 36 // The plugin should (briefly) exist for the preroll phase. | |
| 37 if (plugin.postMessage == undefined) { | |
| 38 window.domAutomationController.send(false); | |
|
Lei Zhang
2015/04/29 22:40:43
Add return here? If you already replied with false
tommycli
2015/04/29 22:44:19
Done. Good point!
| |
| 39 } | |
| 40 | |
| 41 plugin.addEventListener('message', handleEvent); | |
| 42 plugin.postMessage('getPowerSaverStatus'); | |
| 43 } | |
| 44 </script> | |
| 45 | |
| 46 </head> | |
| 47 <body> | |
| 48 | |
| 49 <object id="plugin" type="application/x-ppapi-tests" height='100' width='100'> | |
| 50 </object> | |
| 51 | |
| 52 </body> | |
| 53 </html> | |
| OLD | NEW |
