| commit | 74d1b155b618c3d956f8aa9bbf957a70e49d541a | |
| author | mgiuca <mgiuca@chromium.org> | |
| Mon, 1 Dec 2014 04:23:23 +0000 (30 20:23 -0800) | ||
| committer | Commit bot <commit-bot@chromium.org> | |
| Mon, 1 Dec 2014 04:23:44 +0000 (1 04:23 +0000) | ||
| tree | 74ad9a64f7e91873ddb28eafe1df98094005eebc | treesnapshot (tar.gz zip) |
| parent | beff0c2ad8771bc5078ac21e5bd114895d490c03 | commitdiff |
Fixed AppsGridViewTest failures with experimental app list enabled.
Now if IsExperimentalAppListEnabled returns true, the tests still pass.
1. Fixed AppsGridViewTest::GetItemTileRectAt to take padding into
account.
2. MouseDragMaxItemsInFolderWithMovement drags to a point inside the
tile rect, not 1px outside.
3. Double size of test grid view (from 320x240 to 640x480).
MouseDragItemReorder was failing due to increased size of the tiles
(test drop targets were outside the window bounds).
BUG=437436
Review URL: https://codereview.chromium.org/768713003
Cr-Commit-Position: refs/heads/master@{#306148}
Now if IsExperimentalAppListEnabled returns true, the tests still pass.
1. Fixed AppsGridViewTest::GetItemTileRectAt to take padding into
account.
2. MouseDragMaxItemsInFolderWithMovement drags to a point inside the
tile rect, not 1px outside.
3. Double size of test grid view (from 320x240 to 640x480).
MouseDragItemReorder was failing due to increased size of the tiles
(test drop targets were outside the window bounds).
BUG=437436
Review URL: https://codereview.chromium.org/768713003
Cr-Commit-Position: refs/heads/master@{#306148}
| ui/app_list/views/apps_grid_view.cc | diffblobblamehistory | |
| ui/app_list/views/apps_grid_view.h | diffblobblamehistory | |
| ui/app_list/views/apps_grid_view_unittest.cc | diffblobblamehistory |