((Locatable) element).getCoordinates().inViewPort() doesn't scroll to element in selenium-java

[QALady]

((Locatable) element).getCoordinates().inViewPort() doesn't scroll to element.

Steps to reproduce:

1. Deploy the SmartGwtTester-1.0.war to e.g. Tomcat (pls see comment about attachment below)
2. Execute the following test:

public class ScrollingIssue {

    private static Logger LOG = LoggerFactory.getLogger(ScrollingIssue.class);
    private static Logger BROWSER_LOG = LoggerFactory.getLogger("Browser");

    WebDriver driver = new FirefoxDriver();
    WebDriverWait wait = new WebDriverWait(driver, 5);

    @Test
    public void simpleTest() {
        //open  URL
        driver.get("http://localhost:8888/SmartGwtTester-1.0/");
        driver.manage().window().maximize();
        List<WebElement> results = driver.findElements(By.xpath("//td[@class='treeCell']//tbody/tr"));
        int items = results.size();
        System.out.println("total number of items is " + items);
        int i = 0;
        for (WebElement result : results) {
            ((Locatable) result).getCoordinates().inViewPort();
            WebElement itemLabel = result.findElement(By.xpath("./td[3][@class='treeCell'][contains(@style, 'white-space:nowrap')]"));
            String itemLabelText = itemLabel.getText();
            System.out.println("Click on #" + (i + 1) + " " + itemLabelText);
            WebElement checkbox = result.findElement(By.xpath("./td[2][@class='treeCell']/span[2]"));
            checkbox.click();
            i++;
        }
    }
}

Actual Result: The inViewPort() is not scrolling to 15th element in to view so the test is failing starting this point. All elements before #15 were already in View by default.

The issue is reproducible starting from Selenium 2.35 till Selenium 2.49.1 inclusive. The last working version is 2.34.

I have the war file to reproduce this issue unfortunately the file size is more than 10MB allowed. Please let me know where I can upload it.
Update: The file can be downloaded from here: http://turbobit.net/ih19hhutyehx.html

[barancev]

You can use a file hosting (Dropbox, Google Drive etc) to upload large files.

[QALady]

I have uploaded the war file here: http://turbobit.net/ih19hhutyehx.html

[barancev]

Yes, Selenium does not support custom scrollbars. One have to click to a scrollbar element explicitly to perform scrolling.

[QALady]

As far as I understand Selenium inViewPort() method does not use any custom or not custom scroller to make element visible. instead javascript is involved. also Selenium Webdriver versions before 2.35 did not have this issue. (checked with the same test page). So this defect was introduced starting Selenium 2.35

[barancev]

OK, if it's scrollable by JS then it must be a bug.

[barancev]

Fixed by commit 2f59645