2010-12-17 Ojan Vafai <ojan@chromium.org>
authorojan@chromium.org <ojan@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 17 Dec 2010 18:33:47 +0000 (18:33 +0000)
committerojan@chromium.org <ojan@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 17 Dec 2010 18:33:47 +0000 (18:33 +0000)
        Speculative fix for the code review tool in Firefox.
        Fixes a JS error and makes the status bubble mostly work
        if postMessage is not supported.

        * PrettyPatch/PrettyPatch.rb:
        * code-review.js:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@74276 268f45cc-cd09-0410-ab3c-d52691b4dbfc

BugsSite/ChangeLog
BugsSite/PrettyPatch/PrettyPatch.rb
BugsSite/code-review.js

index 75e4d49..d4627de 100644 (file)
@@ -1,3 +1,12 @@
+2010-12-17  Ojan Vafai  <ojan@chromium.org>
+
+        Speculative fix for the code review tool in Firefox.
+        Fixes a JS error and makes the status bubble mostly work
+        if postMessage is not supported.
+
+        * PrettyPatch/PrettyPatch.rb:
+        * code-review.js:
+
 2010-12-15  Ojan Vafai  <ojan@chromium.org>
 
         Reviewed by Adam Barth.
index 1dd1681..d7f2d58 100644 (file)
@@ -349,6 +349,9 @@ body {
 }
 
 .statusBubble {
+  /* The width/height get set to the bubble contents via postMessage on browsers that support it. */
+  width: 450px;
+  height: 20px;
   margin: 2px 2px 0 0;
   border: none;
   vertical-align: middle;
index c887220..2a3b2c9 100644 (file)
     if (e.origin != 'https://webkit-commit-queue.appspot.com')
       return;
 
-    $('.statusBubble')[0].style.height = e.data.height;
-    $('.statusBubble')[0].style.width = e.data.width;
-  });
+    if (e.data.height) {
+      $('.statusBubble')[0].style.height = e.data.height;
+      $('.statusBubble')[0].style.width = e.data.width;
+    }
+  }, false);
 
   function handleStatusBubbleLoad(e) {
     e.target.contentWindow.postMessage('containerMetrics', 'https://webkit-commit-queue.appspot.com');