Relax HTMLBodyElement's scrollTop/Left property set and get operations.

Source/core/html/HTMLBodyElement.cpp

 /*
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 1999 Antti Koivisto (koivisto@kde.org)
  *           (C) 2000 Simon Hausmann (hausmann@kde.org)
  *           (C) 2001 Dirk Mueller (mueller@kde.org)
  * Copyright (C) 2004, 2006, 2007, 2008, 2009, 2010 Apple Inc. All rights reserved.
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
  * License as published by the Free Software Foundation; either
@@ skipping 235 matching lines @@
     if (zoomFactor == 1)
         return value;
     // Needed because of truncation (rather than rounding) when scaling up.
     if (zoomFactor > 1)
         value++;
     return static_cast<int>(value / zoomFactor);
 }
 
 int HTMLBodyElement::scrollLeft()
 {
+    Document& document = this->document();
+
     // FIXME: The specification is not clear about what is the expected behavior here:
     // http://dev.w3.org/csswg/cssom-view/#dom-element-scrollleft
     // Blink bails out in order to match other engines' behavior (WebKit, IE, Firefox and Opera12).
-    if (!document().inQuirksMode())
-        return 0;
+    if (!document.inQuirksMode())
+        UseCounter::countDeprecation(&document, UseCounter::ScrollLeftBody);
+    else
+        UseCounter::count(&document, UseCounter::ScrollLeftBody);
 
-    // Update the document's layout.
-    Document& document = this->document();
     document.updateLayoutIgnorePendingStylesheets();
     FrameView* view = document.view();
     return view ? adjustForZoom(view->scrollX(), &document) : 0;
 }
 
 void HTMLBodyElement::setScrollLeft(int scrollLeft)
 {
-    if (!document().inQuirksMode())
-        return;
+    Document& document = this->document();
 
-    Document& document = this->document();
+    if (!document.inQuirksMode())
+        UseCounter::countDeprecation(&document, UseCounter::ScrollLeftBody);
+    else
+        UseCounter::count(&document, UseCounter::ScrollLeftBody);
+
     document.updateLayoutIgnorePendingStylesheets();
     Frame* frame = document.frame();
     if (!frame)
         return;
     FrameView* view = frame->view();
     if (!view)
         return;
     view->setScrollPosition(IntPoint(static_cast<int>(scrollLeft * frame->pageZoomFactor()), view->scrollY()));
 }
 
 int HTMLBodyElement::scrollTop()
 {
+    Document& document = this->document();
+
     // FIXME: The specification is not clear about what is the expected behavior here:
     // http://dev.w3.org/csswg/cssom-view/#dom-element-scrolltop .
-    // Blink bails out in order to match other engines' behavior (WebKit, IE, Firefox and Opera12).
-    if (!document().inQuirksMode())
-        return 0;
+    // Blink should bail out in order to match other engines' behavior (WebKit, IE, Firefox and Opera12).
+    if (!document.inQuirksMode())
+        UseCounter::countDeprecation(&document, UseCounter::ScrollTopBody);
+    else
+        UseCounter::count(&document, UseCounter::ScrollTopBody);
 
-    // Update the document's layout.
-    Document& document = this->document();
     document.updateLayoutIgnorePendingStylesheets();
     FrameView* view = document.view();
     return view ? adjustForZoom(view->scrollY(), &document) : 0;
 }
 
 void HTMLBodyElement::setScrollTop(int scrollTop)
 {
-    if (!document().inQuirksMode())
-        return;
+    Document& document = this->document();
 
-    Document& document = this->document();
+    if (!document.inQuirksMode())
+        UseCounter::countDeprecation(&document, UseCounter::ScrollTopBody);
+    else
+        UseCounter::count(&document, UseCounter::ScrollTopBody);
+
     document.updateLayoutIgnorePendingStylesheets();
     Frame* frame = document.frame();
     if (!frame)
         return;
     FrameView* view = frame->view();
     if (!view)
         return;
     view->setScrollPosition(IntPoint(view->scrollX(), static_cast<int>(scrollTop * frame->pageZoomFactor())));
 }
 
@@ skipping 16 matching lines @@
 }
 
 void HTMLBodyElement::addSubresourceAttributeURLs(ListHashSet<KURL>& urls) const
 {
     HTMLElement::addSubresourceAttributeURLs(urls);
 
     addSubresourceURL(urls, document().completeURL(getAttribute(backgroundAttr)));
 }
 
 } // namespace WebCore