Removing container's Left & Top border from computed scroll offset bounds.

Source/core/dom/Element.cpp

 /*
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 1999 Antti Koivisto (koivisto@kde.org)
  *           (C) 2001 Peter Kelly (pmk@post.com)
  *           (C) 2001 Dirk Mueller (mueller@kde.org)
  *           (C) 2007 David Smith (catfish.man@gmail.com)
  * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2010, 2012, 2013 Apple Inc. All rights reserved.
  *           (C) 2007 Eric Seidel (eric@webkit.org)
  *
  * This library is free software; you can redistribute it and/or
@@ skipping 450 matching lines @@
 }
 
 void Element::scrollIntoViewIfNeeded(bool centerIfNeeded)
 {
     document().updateLayoutIgnorePendingStylesheets();
 
     if (!renderer())
         return;
 
     LayoutRect bounds = boundingBox();
+
+    // Fix start : for crbug.com/295848.

[arpitabahuguna, 2014/09/11 14:14:59]

You don't need this comment line.

+    // The bounding box is computed by considering the y position of element
+    // which includes the parent's top border. But scrolling does not consider
+    // border, so this border top should be removed

[pals, 2014/09/12 13:40:08]

. at the end of the comment.

+    RenderBox* parentBox = renderer()->enclosingScrollableBox();

[arpitabahuguna, 2014/09/11 14:14:59]

Why do you name this parentBox??

+    if (parentBox && parentBox->borderTop() > 0) {

[pals, 2014/09/12 13:40:08]

{} not needed here.

+        bounds.setY(bounds.y().toInt() - parentBox->borderTop());

[pals, 2014/09/12 13:40:08]

I think scrollRectTOVisible expects absolute rect. We should not alter the bound
rect here. May be we should fix somewhere at
RenderLayerScrollableArea::exposeRect.

+    }
+    parentBox = 0;

[arpitabahuguna, 2014/09/11 14:14:58]

Not required.

+    // Fix end : for crbug.com/295848

[arpitabahuguna, 2014/09/11 14:14:59]

Again, this line is not required.

+
     if (centerIfNeeded)
         renderer()->scrollRectToVisible(bounds, ScrollAlignment::alignCenterIfNeeded, ScrollAlignment::alignCenterIfNeeded);
     else
         renderer()->scrollRectToVisible(bounds, ScrollAlignment::alignToEdgeIfNeeded, ScrollAlignment::alignToEdgeIfNeeded);
 }
 
 static float localZoomForRenderer(RenderObject& renderer)
 {
     // FIXME: This does the wrong thing if two opposing zooms are in effect and canceled each
     // other out, but the alternative is that we'd have to crawl up the whole render tree every
@@ skipping 2786 matching lines @@
         return v8::Handle<v8::Object>();
 
     wrapper->SetPrototype(binding->prototype());
 
     wrapperType->refObject(toScriptWrappableBase());
     V8DOMWrapper::associateObjectWithWrapperNonTemplate(this, wrapperType, wrapper, isolate);
     return wrapper;
 }
 
 } // namespace blink