diff --git a/src/infrastructure/markup/remarkup/markuprule/youtube/PhabricatorRemarkupRuleYoutube.php b/src/infrastructure/markup/remarkup/markuprule/youtube/PhabricatorRemarkupRuleYoutube.php
index bf6a7e7dae..52b6575e86 100644
--- a/src/infrastructure/markup/remarkup/markuprule/youtube/PhabricatorRemarkupRuleYoutube.php
+++ b/src/infrastructure/markup/remarkup/markuprule/youtube/PhabricatorRemarkupRuleYoutube.php
@@ -38,12 +38,18 @@ class PhabricatorRemarkupRuleYoutube
     if ($v) {
       $youtube_src = 'https://www.youtube.com/embed/'.$v;
       $iframe =
-        '<div class="embedded-youtube-video">
-              <iframe width="650" height="400" '.
-        'style="margin: 1em auto; border: 0px" '.
-        'src="'.$youtube_src.'" '.
-        'frameborder="0"></iframe>
-            </div>';
+        '<div class="embedded-youtube-video">'.
+          phutil_render_tag(
+            'iframe',
+            array(
+              'width'       => '650',
+              'height'      => '400',
+              'style'       => 'margin: 1em auto; border: 0px;',
+              'src'         => $youtube_src,
+              'frameborder' => 0,
+            ),
+            '').
+        '</div>';
       return $this->getEngine()->storeText($iframe);
     } else {
       return $this->uri;
diff --git a/src/infrastructure/markup/remarkup/markuprule/youtube/__init__.php b/src/infrastructure/markup/remarkup/markuprule/youtube/__init__.php
index 80117182c0..7d055605f1 100644
--- a/src/infrastructure/markup/remarkup/markuprule/youtube/__init__.php
+++ b/src/infrastructure/markup/remarkup/markuprule/youtube/__init__.php
@@ -6,6 +6,7 @@
 
 
 
+phutil_require_module('phutil', 'markup');
 phutil_require_module('phutil', 'markup/engine/remarkup/markuprule/base');
 phutil_require_module('phutil', 'parser/uri');
 phutil_require_module('phutil', 'utils');