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

Unified Diff: third_party/WebKit/Source/core/dom/ViewportDescription.cpp

Issue 2390543002: Reflow comments in core/dom/. (Closed)
Patch Set: Reformat comments in core/dom/. 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/dom/ViewportDescription.cpp
diff --git a/third_party/WebKit/Source/core/dom/ViewportDescription.cpp b/third_party/WebKit/Source/core/dom/ViewportDescription.cpp
index 3a665b90d446d126ee57c4035f09b239d3c98180..1d24eef299ec963bc900b5d3a378ad68591a8278 100644
--- a/third_party/WebKit/Source/core/dom/ViewportDescription.cpp
+++ b/third_party/WebKit/Source/core/dom/ViewportDescription.cpp
@@ -3,8 +3,10 @@
* (C) 1999 Antti Koivisto (koivisto@kde.org)
* (C) 2001 Dirk Mueller (mueller@kde.org)
* (C) 2006 Alexey Proskuryakov (ap@webkit.org)
- * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2011 Apple Inc. All rights reserved.
- * Copyright (C) 2008 Torch Mobile Inc. All rights reserved. (http://www.torchmobile.com/)
+ * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2011 Apple Inc. All rights
+ * reserved.
+ * Copyright (C) 2008 Torch Mobile Inc. All rights reserved.
+ * (http://www.torchmobile.com/)
* Copyright (C) 2010 Nokia Corporation and/or its subsidiary(-ies)
* Copyright (C) 2012-2013 Intel Corporation. All rights reserved.
*
@@ -89,9 +91,9 @@ PageScaleConstraints ViewportDescription::resolve(
Length copyMinWidth = minWidth;
// In case the width (used for min- and max-width) is undefined.
if (isLegacyViewportType() && maxWidth.isAuto()) {
- // The width viewport META property is translated into 'width' descriptors, setting
- // the 'min' value to 'extend-to-zoom' and the 'max' value to the intended length.
- // In case the UA-defines a min-width, use that as length.
+ // The width viewport META property is translated into 'width' descriptors,
+ // setting the 'min' value to 'extend-to-zoom' and the 'max' value to the
+ // intended length. In case the UA-defines a min-width, use that as length.
if (zoom == ViewportDescription::ValueAuto) {
copyMinWidth = Length(ExtendToZoom);
copyMaxWidth = legacyFallbackWidth;
@@ -246,7 +248,8 @@ void ViewportDescription::reportMobilePageStats(
!mainFrame->document())
return;
- // Avoid chrome:// pages like the new-tab page (on Android new tab is non-http).
+ // Avoid chrome:// pages like the new-tab page (on Android new tab is
+ // non-http).
if (!mainFrame->document()->url().protocolIsInHTTPFamily())
return;
@@ -264,8 +267,9 @@ void ViewportDescription::reportMobilePageStats(
metaTagTypeHistogram.count(ConstantWidth);
if (mainFrame->view()) {
- // To get an idea of how "far" the viewport is from the device's ideal width, we
- // report the zoom level that we'd need to be at for the entire page to be visible.
+ // To get an idea of how "far" the viewport is from the device's ideal
+ // width, we report the zoom level that we'd need to be at for the
+ // entire page to be visible.
int viewportWidth = maxWidth.intValue();
int windowWidth = mainFrame->host()->visualViewport().size().width();
int overviewZoomPercent =
« no previous file with comments | « third_party/WebKit/Source/core/dom/ViewportDescription.h ('k') | third_party/WebKit/Source/core/dom/VisitedLinkState.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698