Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1276)

Unified Diff: Source/core/page/EventHandler.cpp

Issue 23654015: Avoid using !OS() constructs. (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: git cl try Created 7 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | Source/core/platform/graphics/FontPlatformData.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/core/page/EventHandler.cpp
diff --git a/Source/core/page/EventHandler.cpp b/Source/core/page/EventHandler.cpp
index 183b235223b8d7a155a36a5d1307210eb07a1e92..9e693e9ad7779d1d1e41c29b8a016e62d49a9292 100644
--- a/Source/core/page/EventHandler.cpp
+++ b/Source/core/page/EventHandler.cpp
@@ -2092,15 +2092,15 @@ bool EventHandler::isInsideScrollbar(const IntPoint& windowPoint) const
bool EventHandler::shouldTurnVerticalTicksIntoHorizontal(const HitTestResult& result, const PlatformWheelEvent& event) const
{
-#if OS(UNIX) && !OS(MACOSX)
+#if OS(WINDOWS) || OS(MACOSX)
abarth-chromium 2013/09/09 17:26:24 Should Android do this too?
+ UNUSED_PARAM(result);
+ UNUSED_PARAM(event);
+ return false;
+#else
// GTK+ must scroll horizontally if the mouse pointer is on top of the
// horizontal scrollbar while scrolling with the wheel.
// This code comes from gtk/EventHandlerGtk.cpp.
return !event.hasPreciseScrollingDeltas() && result.scrollbar() && result.scrollbar()->orientation() == HorizontalScrollbar;
-#else
- UNUSED_PARAM(result);
- UNUSED_PARAM(event);
- return false;
#endif
}
« no previous file with comments | « no previous file | Source/core/platform/graphics/FontPlatformData.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698