Fix another case where scrollToMakeVisible was off.

Source/modules/accessibility/AXObject.cpp

Index: Source/modules/accessibility/AXObject.cpp
diff --git a/Source/modules/accessibility/AXObject.cpp b/Source/modules/accessibility/AXObject.cpp
index 6c614bc316af40c592fb7ac80277f6d05df46665..4f032150164201b18f0d86ee5f807a19f048f004 100644
--- a/Source/modules/accessibility/AXObject.cpp
+++ b/Source/modules/accessibility/AXObject.cpp
@@ -1134,6 +1134,12 @@ void AXObject::scrollToMakeVisibleWithSubFocus(const IntRect& subfocus) const
     IntPoint scrollPosition = scrollableArea->scrollPosition();
     IntRect scrollVisibleRect = scrollableArea->visibleContentRect();
 
+    // Convert the object rect into local coordinates.
+    if (!scrollParent->isAXScrollView()) {

[je_julie(Not used), 2015/05/01 16:14:38]

Don't we need null checking for scrollParent?
Because other member functions are checking it.

[dmazzoni, 2015/05/01 16:52:23]

It's supposed to not reach this far if scrollParent is null. I added an explicit
check to be sure, thanks.

+        objectRect.moveBy(scrollPosition);
+        objectRect.moveBy(-pixelSnappedIntRect(scrollParent->elementRect()).location());
+    }
+
     int desiredX = computeBestScrollOffset(
         scrollPosition.x(),
         objectRect.x() + subfocus.x(), objectRect.x() + subfocus.maxX(),