Adjust the size of the bubble to the size of its content in Safari

safari/ext/popup.js

 (function()
 {
   // Safari will load the popover once, and then show it everytime the icon is
   // clicked. While Chrome loads it everytime you click the icon. So in order to
   // force the same behavior in Safari, we are going to reload the page of the
   // bubble everytime it is shown.
   safari.application.addEventListener("popover", function()
   {
     document.documentElement.style.display = "none";
     document.location.reload();
   }, true);
 
 
   // Safari doesn't adjust the size of the popover automatically to the size
   // of its content, like when the ad counter is expanded/collapsed. So we add
   // event listeners to do so.
   var updateSize = function()
   {
     safari.self.width = document.body.offsetWidth;
     safari.self.height = document.body.offsetHeight;
   };
 
   window.addEventListener("load", function()
   {
     updateSize();
 
-    if ("WebKitMutationObserver" in window)

[Wladimir Palant, 2014/01/16 07:55:59]

We should go for the standardized name first, future Safari versions should
support it. Maybe like this:

    if ("MutationObserver" in window || "WebKitMutationObserver" in window)
    {
      var observer = new (window.MutationObserver
|| window.WebKitMutationObserver)(updateSize);
      ...

Style nit: please put curly brackets around multiline blocks even if they
consist of only one statement - readability suffers otherwise.

[Sebastian Noack, 2014/01/16 11:00:31]

Done.

-      new WebKitMutationObserver(updateSize).observe(document, {
+    var MutationObserver = window.MutationObserver || window.WebKitMutationObserver;
+    if (MutationObserver)
+    {
+      new MutationObserver(updateSize).observe(document, {
         childList: true, attributes: true,
         characterData: true, subtree: true
       });
+    }
     else
       document.addEventListener("DOMSubtreeModified", updateSize);
   });
 
 
   // import ext into the javascript context of the popover. This code might fail,
   // when the background page isn't ready yet. So it is important to put it below
   // the reloading code above.
   var backgroundPage = safari.extension.globalPage.contentWindow;
-  ext = backgroundPage.ext;
-  TabMap = backgroundPage.TabMap;
+  window.ext = backgroundPage.ext;
+  window.TabMap = backgroundPage.TabMap;
 })();