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

Unified Diff: third_party/WebKit/Source/core/html/MediaDocument.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/MediaDocument.cpp
diff --git a/third_party/WebKit/Source/core/html/MediaDocument.cpp b/third_party/WebKit/Source/core/html/MediaDocument.cpp
index c16f8a088e93795114a830c65d6c73dd5d9a6ef2..cbc672aa46d42f3a6392ca173f22cec233bd6de1 100644
--- a/third_party/WebKit/Source/core/html/MediaDocument.cpp
+++ b/third_party/WebKit/Source/core/html/MediaDocument.cpp
@@ -145,11 +145,12 @@ void MediaDocumentParser::createDocumentStructure() {
body->setAttribute(styleAttr, "margin: 0px;");
HTMLDivElement* div = HTMLDivElement::create(*document());
- // Style sheets for media controls are lazily loaded until a media element is encountered.
- // As a result, elements encountered before the media element will not get the right
- // style at first if we put the styles in mediacontrols.css. To solve this issue, set the
- // styles inline so that they will be applied when the page loads.
- // See w3c example on how to centering an element: https://www.w3.org/Style/Examples/007/center.en.html
+ // Style sheets for media controls are lazily loaded until a media element is
+ // encountered. As a result, elements encountered before the media element
+ // will not get the right style at first if we put the styles in
+ // mediacontrols.css. To solve this issue, set the styles inline so that they
+ // will be applied when the page loads. See w3c example on how to centering
+ // an element: https://www.w3.org/Style/Examples/007/center.en.html
div->setAttribute(styleAttr,
"display: flex;"
"flex-direction: column;"
@@ -197,7 +198,8 @@ void MediaDocumentParser::createDocumentStructure() {
recordDownloadMetric(MediaDocumentDownloadButtonShown);
}
- // According to https://html.spec.whatwg.org/multipage/browsers.html#read-media,
+ // According to
+ // https://html.spec.whatwg.org/multipage/browsers.html#read-media,
// MediaDocument should have a single child which is the video element. Use
// shadow root to hide all the elements we added here.
ShadowRoot& shadowRoot = body->ensureUserAgentShadowRoot();

Powered by Google App Engine
This is Rietveld 408576698