Include scrollbar size in @media width/height

Source/core/platform/ScrollView.h

Index: Source/core/platform/ScrollView.h
diff --git a/Source/core/platform/ScrollView.h b/Source/core/platform/ScrollView.h
index b8f97a38e32c714fb500c26de4bb8173bb321d14..1964dd3ef2235ffae9e9aa5276d8157836f2d799 100644
--- a/Source/core/platform/ScrollView.h
+++ b/Source/core/platform/ScrollView.h
@@ -146,7 +146,12 @@ public:
     void setFixedLayoutSize(const IntSize&);
     bool useFixedLayout() const;
     void setUseFixedLayout(bool enable);
-    
+
+    // Similar to layoutSize functions, but don't exclude the scrollbars. For CSS @media width/height MQ rules.
+    IntSize mediaSize() const;

[jamesr, 2013/05/06 22:52:03]

is this the same as calling visibleContentRect(IncludeScrollbars).size() ? if
not, what's different?

[Xianzhu, 2013/05/06 22:55:44]

They are the same for non-fixed-layout mode. For fixed-layout mode, it's the
fixed layout size which normally reflects the virtual viewport size.

[jamesr, 2013/05/06 23:38:49]

Can we instead give layoutSize() the Includes/ExcludesScrollbars enums, using a
default value so you don't have to change callers. I'm a little concerned about
this being a near-duplicate of layoutSize() since the behavior of layoutSize()
isn't entirely obvious.  The "media" prefix isn't very descriptive either - it's
a concern of how this value is used (for media queries) but not how the value is
computed, and really platform/ScrollView shouldn't care about how media queries
work

+    int mediaWidth() const { return mediaSize().width(); }
+    int mediaHeight() const { return mediaSize().height(); }
+
     // Functions for getting/setting the size of the document contained inside the ScrollView (as an IntSize or as individual width and height
     // values).
     virtual IntSize contentsSize() const OVERRIDE; // Always at least as big as the visibleWidth()/visibleHeight().