reflow comments in core/html/*.{cpp,h},core/html/imports

third_party/WebKit/Source/core/html/PluginDocument.cpp

 /*
  * Copyright (C) 2006, 2008 Apple Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
  *    notice, this list of conditions and the following disclaimer in the
@@ skipping 54 matching lines @@
 
   void createDocumentStructure();
 
   PluginView* pluginView() const;
 
   Member<HTMLEmbedElement> m_embedElement;
 };
 
 void PluginDocumentParser::createDocumentStructure() {
   // FIXME: Assert we have a loader to figure out why the original null checks
-  // and assert were added for the security bug in http://trac.webkit.org/changeset/87566
+  // and assert were added for the security bug in
+  // http://trac.webkit.org/changeset/87566
   DCHECK(document());
   RELEASE_ASSERT(document()->loader());
 
   LocalFrame* frame = document()->frame();
   if (!frame)
     return;
 
   // FIXME: Why does this check settings?
   if (!frame->settings() ||
       !frame->loader().allowPlugins(NotAboutToInstantiatePlugin))
     return;
 
   HTMLHtmlElement* rootElement = HTMLHtmlElement::create(*document());
   document()->appendChild(rootElement);
   rootElement->insertedByParser();
   if (isStopped())
     return;  // runScriptsAtDocumentElementAvailable can detach the frame.
 
   HTMLBodyElement* body = HTMLBodyElement::create(*document());
   body->setAttribute(styleAttr,
                      "background-color: rgb(38,38,38); height: 100%; width: "
                      "100%; overflow: hidden; margin: 0");
   rootElement->appendChild(body);
   if (isStopped())
-    return;  // Possibly detached by a mutation event listener installed in runScriptsAtDocumentElementAvailable.
+    return;  // Possibly detached by a mutation event listener installed in

[dcheng, 2016/10/05 22:29:17]

Nit: own line, maybe?

(Also surprised that check-webkit-style isn't complaining about braces here, but
I think this needs braces. Same below)

[Nico, 2016/10/05 22:35:17]

Done.

+             // runScriptsAtDocumentElementAvailable.
 
   m_embedElement = HTMLEmbedElement::create(*document());
   m_embedElement->setAttribute(widthAttr, "100%");
   m_embedElement->setAttribute(heightAttr, "100%");
   m_embedElement->setAttribute(nameAttr, "plugin");
   m_embedElement->setAttribute(idAttr, "plugin");
   m_embedElement->setAttribute(srcAttr,
                                AtomicString(document()->url().getString()));
   m_embedElement->setAttribute(typeAttr, document()->loader()->mimeType());
   body->appendChild(m_embedElement);
   if (isStopped())
-    return;  // Possibly detached by a mutation event listener installed in runScriptsAtDocumentElementAvailable.
+    return;  // Possibly detached by a mutation event listener installed in
+             // runScriptsAtDocumentElementAvailable.
 
   toPluginDocument(document())->setPluginNode(m_embedElement.get());
 
   document()->updateStyleAndLayout();
 
   // We need the plugin to load synchronously so we can get the PluginView
   // below so flush the layout tasks now instead of waiting on the timer.
   frame->view()->flushAnyPendingPostLayoutTasks();
   // Focus the plugin here, as the line above is where the plugin is created.
   if (frame->isMainFrame()) {
     m_embedElement->focus();
     if (isStopped())
-      return;  // Possibly detached by a focus event listener installed in runScriptsAtDocumentElementAvailable.
+      return;  // Possibly detached by a focus event listener installed in
+               // runScriptsAtDocumentElementAvailable.
   }
 
   if (PluginView* view = pluginView())
     view->didReceiveResponse(document()->loader()->response());
 }
 
 void PluginDocumentParser::appendBytes(const char* data, size_t length) {
   if (!m_embedElement) {
     createDocumentStructure();
     if (isStopped())
@@ skipping 53 matching lines @@
   m_pluginNode = nullptr;
   HTMLDocument::shutdown();
 }
 
 DEFINE_TRACE(PluginDocument) {
   visitor->trace(m_pluginNode);
   HTMLDocument::trace(visitor);
 }
 
 }  // namespace blink