2011-01-12 Ojan Vafai <ojan@chromium.org>
authorojan@chromium.org <ojan@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 12 Jan 2011 22:58:31 +0000 (22:58 +0000)
committerojan@chromium.org <ojan@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 12 Jan 2011 22:58:31 +0000 (22:58 +0000)
        Reviewed by Adam Barth.

        show shared lines on both sides in code review tool
        https://bugs.webkit.org/show_bug.cgi?id=52308

        We were appending the same DOM node twice. Instead, make
        the from column a clone of the original node.

        * code-review.js:

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

Websites/bugs.webkit.org/ChangeLog
Websites/bugs.webkit.org/code-review.js

index 75942ab..6e2ee04 100644 (file)
@@ -2,6 +2,18 @@
 
         Reviewed by Adam Barth.
 
+        show shared lines on both sides in code review tool
+        https://bugs.webkit.org/show_bug.cgi?id=52308
+
+        We were appending the same DOM node twice. Instead, make
+        the from column a clone of the original node.
+
+        * code-review.js:
+
+2011-01-12  Ojan Vafai  <ojan@chromium.org>
+
+        Reviewed by Adam Barth.
+
         fix wrapping bug in expansion lines in the code review tool
         https://bugs.webkit.org/show_bug.cgi?id=52270
 
index 6836be2..b0786f7 100644 (file)
     var to_class = '';
     var from_attributes = '';
     var to_attributes = '';
-    var from_contents = contents;
+    // Clone the contents so we have two copies we can put back in the DOM.
+    var from_contents = contents.clone(true);
     var to_contents = contents;
 
+    var container_class = 'LineContainer';
+    var container_attributes = '';
+
     if (from && !to) { // This is a remove line.
       from_class = classNames;
       from_attributes = attributes;
       to_class = classNames;
       to_attributes = attributes;
       from_contents = '';
-    }
-
-    var container_class = 'LineContainer';
-    var container_attributes = '';
-    if (!to_attributes && !from_attributes) {
+    } else {
       container_attributes = attributes;
       container_class += ' Line ' + classNames;
     }