Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1138)

Unified Diff: third_party/WebKit/Source/core/html/PluginDocument.cpp

Issue 2384273007: reflow comments in core/html/*.{cpp,h},core/html/imports (Closed)
Patch Set: comments Created 4 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Source/core/html/PluginDocument.cpp
diff --git a/third_party/WebKit/Source/core/html/PluginDocument.cpp b/third_party/WebKit/Source/core/html/PluginDocument.cpp
index 84f3c5a2256aeea4d7f0060f4105a3b020646959..06e2f326fef8eb386cf91594c652a046cd96c113 100644
--- a/third_party/WebKit/Source/core/html/PluginDocument.cpp
+++ b/third_party/WebKit/Source/core/html/PluginDocument.cpp
@@ -72,7 +72,8 @@ class PluginDocumentParser : public RawDataDocumentParser {
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());
@@ -96,8 +97,11 @@ void PluginDocumentParser::createDocumentStructure() {
"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.
+ if (isStopped()) {
+ // Possibly detached by a mutation event listener installed in
+ // runScriptsAtDocumentElementAvailable.
+ return;
+ }
m_embedElement = HTMLEmbedElement::create(*document());
m_embedElement->setAttribute(widthAttr, "100%");
@@ -108,8 +112,11 @@ void PluginDocumentParser::createDocumentStructure() {
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.
+ if (isStopped()) {
+ // Possibly detached by a mutation event listener installed in
+ // runScriptsAtDocumentElementAvailable.
+ return;
+ }
toPluginDocument(document())->setPluginNode(m_embedElement.get());
@@ -121,8 +128,11 @@ void PluginDocumentParser::createDocumentStructure() {
// 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.
+ if (isStopped()) {
+ // Possibly detached by a mutation event listener installed in
+ // runScriptsAtDocumentElementAvailable.
+ return;
+ }
}
if (PluginView* view = pluginView())
« no previous file with comments | « third_party/WebKit/Source/core/html/MediaFragmentURIParser.cpp ('k') | third_party/WebKit/Source/core/html/RadioNodeList.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698