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

Unified Diff: third_party/WebKit/Source/core/html/track/vtt/VTTRegion.cpp

Issue 2393633005: reflow comments in core/imagebitmap,core/html/track (Closed)
Patch Set: 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/track/vtt/VTTRegion.cpp
diff --git a/third_party/WebKit/Source/core/html/track/vtt/VTTRegion.cpp b/third_party/WebKit/Source/core/html/track/vtt/VTTRegion.cpp
index d1a214d2754452827c36d6d6d8b281f75cb74ee8..530f8e4e0a0bc412d6d3914c005b747692c0bcf5 100644
--- a/third_party/WebKit/Source/core/html/track/vtt/VTTRegion.cpp
+++ b/third_party/WebKit/Source/core/html/track/vtt/VTTRegion.cpp
@@ -46,7 +46,8 @@
namespace blink {
-// The following values default values are defined within the WebVTT Regions Spec.
+// The following values default values are defined within the WebVTT Regions
+// Spec.
// https://dvcs.w3.org/hg/text-tracks/raw-file/default/608toVTT/region.html
// The region occupies by default 100% of the width of the video viewport.
@@ -352,9 +353,9 @@ void VTTRegion::displayLastVTTCueBox() {
DVLOG(VTT_LOG_LEVEL) << "displayLastVTTCueBox";
DCHECK(m_cueContainer);
- // FIXME: This should not be causing recalc styles in a loop to set the "top" css
- // property to move elements. We should just scroll the text track cues on the
- // compositor with an animation.
+ // FIXME: This should not be causing recalc styles in a loop to set the "top"
+ // css property to move elements. We should just scroll the text track cues on
+ // the compositor with an animation.
if (m_scrollTimer.isActive())
return;

Powered by Google App Engine
This is Rietveld 408576698