browser-plugin: Remove event handling code, and use CustomEvents in webkit instead.

content/test/data/browser_plugin_embedder.html

 <script type="text/javascript">
 function sizeChanged(evt) {
-  document.title = "AutoSize(" + evt.newWidth + ", " + evt.newHeight + ")";
+  var data = JSON.parse(evt.detail);
+  document.title = "AutoSize(" + data.newWidth + ", " + data.newHeight + ")";
 }
 
 function loadAbort(evt) {
-  document.title = evt.type;
+  document.title = JSON.parse(evt.detail).reason;

[Charlie Reis, 2012/11/12 18:23:12]

I'm trying to understand what we're exposing to real pages.

It looks like the webview shim will expose things like event.url for event names
like "loadabort", while the browser plugin will expose event.detail={"url":url}
for event names like "-internal-loadabort".  Is that right?

Are we concerned about the internal versions being exposed to real pages, or is
that something we can prevent?  Is this conversion documented anywhere?

[sadrul, 2012/11/12 19:00:16]

That is correct.
I think the main concern is to expose a semi-stable documented API to <webview>
consumers (i.e. 'loadabort' -> event.url, etc). The internal events (i.e.
'-internal-' -> event.detail) are implementation details that can change, and
should not be used externally (i.e. undocumented).

 }
 function loadStart(evt) {
-  document.title = evt.url;
+  document.title = JSON.parse(evt.detail).url;
 }
 function loadStop(evt) {
   document.title = "loadStop";
 }
 
 var commitIsTopLevel;
 function loadCommit(evt) {
-  document.title = "loadCommit:" + evt.url;
-  commitIsTopLevel = evt.isTopLevel;
+  var detail = JSON.parse(evt.detail);
+  document.title = "loadCommit:" + detail.url;
+  commitIsTopLevel = detail.isTopLevel;
 }
 
 var redirectOldUrl;
 var redirectNewUrl;
 function loadRedirect(event) {
   document.title = "redirected";
-  if (event.isTopLevel) {
-    redirectOldUrl = event.oldUrl;
-    redirectNewUrl = event.newUrl;
+  var detail = JSON.parse(event.detail);
+  if (detail.isTopLevel) {
+    redirectOldUrl = detail.oldUrl;
+    redirectNewUrl = detail.newUrl;
   }
 }
 function SetSrc(src) {
   var plugin = document.getElementById('plugin');
   plugin.src = src;
 }
 function SetSize(w, h) {
   var plugin = document.getElementById('plugin');
   plugin.width = w;
   plugin.height = h;
@@ skipping 53 matching lines @@
   } else {
     var name = msg.replace("stop_ack", "").trim();
     if (name !== '') {
       window.document.title = name;
     } else {
       window.document.title = 'main guest';
     }
   }
 }
   var plugin = document.getElementById('plugin');
-  plugin.addEventListener('loadstart', loadStart);
-  plugin.addEventListener('loadabort', loadAbort);
-  plugin.addEventListener('loadredirect', loadRedirect);
+  plugin.addEventListener('-internal-loadstart', loadStart);
+  plugin.addEventListener('-internal-loadabort', loadAbort);
+  plugin.addEventListener('-internal-loadredirect', loadRedirect);
   window.addEventListener('message', receiveMessage, false);
-  plugin.addEventListener('loadstop', loadStop);
-  plugin.addEventListener('loadcommit', loadCommit);
-  plugin.addEventListener('sizechanged', sizeChanged);
+  plugin.addEventListener('-internal-loadstop', loadStop);
+  plugin.addEventListener('-internal-loadcommit', loadCommit);
+  plugin.addEventListener('-internal-sizechanged', sizeChanged);
 </script>