|
Rename usesOverlayScrollbars of settings to mockScrollbarsUseOverlay
usesOverlayScrollbars takes effect only if mockScrollbarsEnabled.
Rename it to mockScrollbarsUseOverlay to avoid misuse.
The existing misuse is that some tests call only
window.internals.setUsesOverlayScrollbars(true) that will do nothing
without window.interals.settings.setMockScrollbarsEnabled(true).
Besides renaming, removed the original wrong placed
window.internals.setUsesOverlayScrollbars() and added
window.internals.settings.setMockScrollbarsUseOverlay().
Removed ScrollbarThemeMock::usesOverlayScrollbars() because
ScrollbarThemeMock is not used when overlayScrollbars is enabled
(ScroolbarThemeOverlayMock is used).
Updated the tests that origially call setUsesOverlayScrollbar(true)
only.
BUG=none
TEST=existing layout tests
|
Unified diffs |
Side-by-side diffs |
Delta from patch set |
Stats (+45 lines, -44 lines) |
Patch |
 |
M |
LayoutTests/fast/dom/partial-layout-block.html
|
View
|
1
|
1 chunk |
+4 lines, -2 lines |
0 comments
|
Download
|
 |
M |
LayoutTests/fast/dom/partial-layout-non-overlay-scrollbars.html
|
View
|
1
|
1 chunk |
+5 lines, -2 lines |
0 comments
|
Download
|
 |
M |
LayoutTests/fast/dom/partial-layout-overlay-scrollbars.html
|
View
|
1
|
1 chunk |
+4 lines, -2 lines |
0 comments
|
Download
|
 |
M |
LayoutTests/fast/scrolling/overlay-scrollbars.html
|
View
|
1
|
1 chunk |
+1 line, -1 line |
0 comments
|
Download
|
 |
M |
LayoutTests/fast/scrolling/scrollbar-tickmarks-hittest.html
|
View
|
1
|
1 chunk |
+2 lines, -1 line |
0 comments
|
Download
|
 |
M |
Source/core/page/Settings.h
|
View
|
1
|
2 chunks |
+3 lines, -3 lines |
0 comments
|
Download
|
 |
M |
Source/core/page/Settings.cpp
|
View
|
1
|
2 chunks |
+5 lines, -5 lines |
0 comments
|
Download
|
 |
M |
Source/core/platform/ScrollbarTheme.cpp
|
View
|
1
|
1 chunk |
+1 line, -1 line |
0 comments
|
Download
|
 |
M |
Source/core/platform/mock/ScrollbarThemeMock.h
|
View
|
|
1 chunk |
+0 lines, -1 line |
0 comments
|
Download
|
 |
M |
Source/core/platform/mock/ScrollbarThemeMock.cpp
|
View
|
|
1 chunk |
+0 lines, -5 lines |
0 comments
|
Download
|
 |
M |
Source/core/testing/InternalSettings.h
|
View
|
1
|
2 chunks |
+2 lines, -2 lines |
0 comments
|
Download
|
 |
M |
Source/core/testing/InternalSettings.cpp
|
View
|
1
|
3 chunks |
+8 lines, -0 lines |
0 comments
|
Download
|
 |
M |
Source/core/testing/InternalSettings.idl
|
View
|
1
|
1 chunk |
+1 line, -0 lines |
0 comments
|
Download
|
 |
M |
Source/core/testing/Internals.h
|
View
|
|
1 chunk |
+0 lines, -2 lines |
0 comments
|
Download
|
 |
M |
Source/core/testing/Internals.cpp
|
View
|
|
2 chunks |
+0 lines, -6 lines |
0 comments
|
Download
|
 |
M |
Source/core/testing/Internals.idl
|
View
|
|
1 chunk |
+0 lines, -2 lines |
0 comments
|
Download
|
 |
M |
Source/web/tests/WebFrameTest.cpp
|
View
|
1
|
9 chunks |
+9 lines, -9 lines |
0 comments
|
Download
|
Total messages: 11 (0 generated)
|