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, {
+        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;
 })();