Add loadCommit and loadStop Event

content/test/data/browser_plugin_embedder.html

Index: content/test/data/browser_plugin_embedder.html
diff --git a/content/test/data/browser_plugin_embedder.html b/content/test/data/browser_plugin_embedder.html
index 45b374e3ac9f9d91494114f5cfb0b6418c9a0168..18751834593523b3367fa25c2b69fa65186fb6dc 100644
--- a/content/test/data/browser_plugin_embedder.html
+++ b/content/test/data/browser_plugin_embedder.html
@@ -5,6 +5,16 @@ function loadAbort(evt) {
 function loadStart(evt) {
   document.title = evt.url;
 }
+function loadStop(evt) {
+  document.title = "loadStop";

[Charlie Reis, 2012/10/08 19:03:39]

Won't this interfere with other tests?  Every test will have a loadStop event.

[irobert, 2012/10/10 23:55:16]

I don't think it will effect other test based on the result I ran on my local
machine.
Done.

+}
+
+var isTopLevel;
+function loadCommit(evt) {
+    document.title = evt.url;

[Charlie Reis, 2012/10/08 19:03:39]

Same as above.  We'll need some way to avoid affecting the other tests.  One way
is to inject a "shouldUpdateTitleOnCommit" variable, similar to
"shouldEchoTitle" in https://codereview.chromium.org/11027065/.  (It defaults to
false, and I set it to true in the test that cares about it.)

[irobert, 2012/10/10 23:55:16]

Done.

+    isTopLevel = evt.isTopLevel;
+}
+
 var redirectOldUrl;
 var redirectNewUrl;
 function loadRedirect(event) {
@@ -53,4 +63,6 @@ document.title = 'embedder';
   plugin.addEventListener('loadStart', loadStart);
   plugin.addEventListener('loadAbort', loadAbort);
   plugin.addEventListener('loadRedirect', loadRedirect);
+  plugin.addEventListener('loadStop', loadStop);
+  plugin.addEventListener('loadCommit', loadCommit);
 </script>