Render inline comments somewhat-correctly on 1-up diffs

Summary: You can't interact with them yey, but do a less awful job of rendering them.

Test Plan: {F29204}

Reviewers: btrahan

Reviewed By: btrahan

CC: aran

Maniphest Tasks: T2009

Differential Revision: https://secure.phabricator.com/D4426

src/applications/differential/render/DifferentialChangesetHTMLRenderer.php

@@ -293,7 +293,7 @@ abstract class DifferentialChangesetHTMLRenderer
       '</table>';
   }
 
-  protected function renderShield($message, $force = 'default') {
+  public function renderShield($message, $force = 'default') {
     $end = count($this->getOldLines());
     $reference = $this->getRenderingReference();
 
@@ -333,15 +333,16 @@ abstract class DifferentialChangesetHTMLRenderer
         'Show File Contents');
     }
 
-    return javelin_render_tag(
-      'tr',
-      array(
-        'sigil' => 'context-target',
-      ),
-      '<td class="differential-shield" colspan="6">'.
-        phutil_escape_html($message).
-        $more.
-      '</td>');
+    return $this->wrapChangeInTable(
+      javelin_render_tag(
+        'tr',
+        array(
+          'sigil' => 'context-target',
+        ),
+        '<td class="differential-shield" colspan="6">'.
+          phutil_escape_html($message).
+          $more.
+        '</td>'));
   }
 
   protected function wrapChangeInTable($content) {
@@ -358,5 +359,30 @@ abstract class DifferentialChangesetHTMLRenderer
       $content);
   }
 
+  protected function renderInlineComment(
+    PhabricatorInlineCommentInterface $comment,
+    $on_right = false) {
+
+    return $this->buildInlineComment($comment, $on_right)->render();
+  }
+
+  protected function buildInlineComment(
+    PhabricatorInlineCommentInterface $comment,
+    $on_right = false) {
+
+    $user = $this->getUser();
+    $edit = $user &&
+            ($comment->getAuthorPHID() == $user->getPHID()) &&
+            ($comment->isDraft());
+    $allow_reply = (bool)$user;
+
+    return id(new DifferentialInlineCommentView())
+      ->setInlineComment($comment)
+      ->setOnRight($on_right)
+      ->setHandles($this->getHandles())
+      ->setMarkupEngine($this->getMarkupEngine())
+      ->setEditable($edit)
+      ->setAllowReply($allow_reply);
+  }
 
 }

src/applications/differential/render/DifferentialChangesetOneUpRenderer.php

@@ -45,10 +45,13 @@ final class DifferentialChangesetOneUpRenderer
           break;
         case 'inline':
           $out[] = '<tr><th /><th />';
-          $out[] = '<td style="background: #ddd; color: #888;">';
+          $out[] = '<td>';
 
-          $out[] = 'INLINE COMMENT<br />';
-          $out[] = phutil_escape_html($p['comment']->getContent());
+          $inline = $this->buildInlineComment(
+            $p['comment'],
+            $p['right']);
+          $inline->setBuildScaffolding(false);
+          $out[] = $inline->render();
 
           $out[] = '</td></tr>';
           break;

src/applications/differential/render/DifferentialChangesetRenderer.php

@@ -298,30 +298,10 @@ abstract class DifferentialChangesetRenderer {
    * @param   string|null   Force mode, see above.
    * @return  string        Shield markup.
    */
-  abstract protected function renderShield($message, $force = 'default');
+  abstract public function renderShield($message, $force = 'default');
 
   abstract protected function renderPropertyChangeHeader();
 
-  protected function renderInlineComment(
-    PhabricatorInlineCommentInterface $comment,
-    $on_right = false) {
-
-    $user = $this->getUser();
-    $edit = $user &&
-            ($comment->getAuthorPHID() == $user->getPHID()) &&
-            ($comment->isDraft());
-    $allow_reply = (bool)$user;
-
-    return id(new DifferentialInlineCommentView())
-      ->setInlineComment($comment)
-      ->setOnRight($on_right)
-      ->setHandles($this->getHandles())
-      ->setMarkupEngine($this->getMarkupEngine())
-      ->setEditable($edit)
-      ->setAllowReply($allow_reply)
-      ->render();
-  }
-
   protected function buildPrimitives($range_start, $range_len) {
     $primitives = array();
 

src/applications/differential/render/DifferentialChangesetTestRenderer.php

@@ -17,7 +17,7 @@ abstract class DifferentialChangesetTestRenderer
            "{$away}\n";
   }
 
-  protected function renderShield($message, $force = 'default') {
+  public function renderShield($message, $force = 'default') {
     return "SHIELD ({$force}) {$message}\n";
   }