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

Unified Diff: ppapi/tests/manual/delete_plugin.html

Issue 6881012: Keep the module in scope when executing scripts. This prevents a crash when the (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: '' Created 9 years, 8 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: ppapi/tests/manual/delete_plugin.html
===================================================================
--- ppapi/tests/manual/delete_plugin.html (revision 0)
+++ ppapi/tests/manual/delete_plugin.html (revision 0)
@@ -0,0 +1,25 @@
+<html><head><title>
+<body>
+<script type="text/javascript">
+function deletePlugin() {
+ // Remove the plugin.
+ document.getElementById('foo').removeChild(document.getElementById('plugin'));
+
+ // Forces a style recalculation which actually will delete the plugin.
+ // Without this, the plugin would be deleted later from a timer. If the test
+ // fails, this line will cause the crash.
+ var foo = document.getElementById('foo').offsetWidth;
+
+ // If we get here, there's no crash so we succeeded.
+ document.cookie = "COMPLETION_COOKIE=PASS; path=/";
+}
+</script>
+ This test tests deleting a the &lt;object&gt; synchronously from within a
+ script call. Neither the proxy nor the PPAPI implementation should crash in
+ this case.
+ <div id="foo">
+ <object id="plugin" type="application/x-ppapi-tests" width="400" height="400" style="border:5px solid blue;">
+ <param name="customtest" value="delete_plugin">
+ </object>
+ </div>
+</body>
Property changes on: ppapi/tests/manual/delete_plugin.html
___________________________________________________________________
Added: svn:eol-style
+ LF

Powered by Google App Engine
This is Rietveld 408576698