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

Unified Diff: Source/WebKit/chromium/src/ViewportAttributesManager.h

Issue 14813025: Refactor viewport initialization logic out of WebViewImpl. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Add tests for new behavior Created 7 years, 7 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: Source/WebKit/chromium/src/ViewportAttributesManager.h
diff --git a/Source/WebKit/chromium/src/ViewportAttributesManager.h b/Source/WebKit/chromium/src/ViewportAttributesManager.h
new file mode 100644
index 0000000000000000000000000000000000000000..e96418cfcd8611fb09a6202c5a358505e98bb3ef
--- /dev/null
+++ b/Source/WebKit/chromium/src/ViewportAttributesManager.h
@@ -0,0 +1,88 @@
+/*
+ * Copyright (C) 2013 Google 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:
+ *
+ * * Redistributions of source code must retain the above copyright
+ * notice, this list of conditions and the following disclaimer.
+ * * Redistributions in binary form must reproduce the above
+ * copyright notice, this list of conditions and the following disclaimer
+ * in the documentation and/or other materials provided with the
+ * distribution.
+ * * Neither the name of Google Inc. nor the names of its
+ * contributors may be used to endorse or promote products derived from
+ * this software without specific prior written permission.
+ *
+ * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
+ * "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
+ * LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
+ * A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
+ * OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
+ * SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
+ * LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
+ * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
+ * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
+ * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
+ * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+ */
+
+#ifndef ViewportAttributesManager_h
+#define ViewportAttributesManager_h
+
+#include "core/dom/ViewportArguments.h"
+#include "core/dom/ViewportAttributes.h"
+#include "core/platform/graphics/IntSize.h"
+
+namespace WebKit {
+
+// This class harmonizes the viewport (particularly page scale) constraints from
+// the meta viewport tag and other sources.
+class ViewportAttributesManager {
+public:
+ ViewportAttributesManager();
+
+ WebCore::ViewportAttributes defaultConstraints() const;
+
+ // Settings defined in the website's viewport tag, if viewport tag support
+ // is enabled.
+ WebCore::ViewportAttributes pageDefined() const { return m_pageDefined; }
kenneth.r.christiansen 2013/05/13 11:00:18 Why not call it pageDefinedConstraints() to be con
aelias_OOO_until_Jul13 2013/05/13 18:13:06 Done.
+ void updatePageDefined(const WebCore::ViewportArguments&, WebCore::IntSize viewSize, int layoutFallbackWidth);
+ void adjustPageDefinedForAndroidWebView(const WebCore::ViewportArguments&, WebCore::IntSize viewSize, int layoutFallbackWidth, float deviceScaleFactor, bool useWideViewport, bool initializeAtMinimumPageScale);
+
+ // Attributes may also be set from Chromium -- this overrides any
+ // page-defined values.
+ WebCore::ViewportAttributes embedderConstraints() const { return m_embedderConstraints; }
+ void setEmbedderConstraints(WebCore::ViewportAttributes);
kenneth.r.christiansen 2013/05/13 11:00:18 setUserAgentConstraints - UA is the terminology us
aelias_OOO_until_Jul13 2013/05/13 18:13:06 Done.
+
+ // Actual computed values, taking into account the above plus the current
+ // viewport size and document width.
+ WebCore::ViewportAttributes finalAttributes() const { return m_finalAttributes; }
+ void computeFinalAttributes();
+ void adjustFinalAttributesToContentsSize(WebCore::IntSize viewSize, WebCore::IntSize contentsSize, int nonOverlayScrollbarWidth);
+
+ void didChangeContentsSize(WebCore::IntSize contentsSize, float pageScaleFactor);
+
+ // This should be set to true on each page load to note that the page scale
+ // factor needs to be reset to its initial value.
+ void setNeedsReset(bool);
+ bool needsReset() const { return m_needsReset; }
+
+ // This is set when one of the inputs to finalAttributes changes.
+ bool constraintsDirty() const { return m_constraintsDirty; }
+
+private:
+ WebCore::ViewportAttributes m_pageDefined;
+ WebCore::ViewportAttributes m_embedderConstraints;
+ WebCore::ViewportAttributes m_finalAttributes;
kenneth.r.christiansen 2013/05/13 11:00:18 The naming is a bit confusing now with attributes
aelias_OOO_until_Jul13 2013/05/13 18:13:06 Renamed to finalConstraints as well.
+
+ int m_lastContentsWidth;
+
+ bool m_needsReset;
+ bool m_constraintsDirty;
+};
+
+} // namespace WebKit
+
+#endif

Powered by Google App Engine
This is Rietveld 408576698