[webkit-changes] [173034] trunk/Source/WebCore

Wed, 27 Aug 2014 16:30:07 -0700

Title: [173034] trunk/Source/WebCore
Revision
173034
Author
[email protected]
Date
2014-08-27 16:29:28 -0700 (Wed, 27 Aug 2014)

Log Message

Enums in ScrollTypes and PlatformWheelEvent should not use uint64_t storage
https://bugs.webkit.org/show_bug.cgi?id=136318

Reviewed by Simon Fraser.

This regressed in a replay-related patch. Revert back to uint8_t storage size.

* platform/PlatformWheelEvent.h:
* platform/ScrollTypes.h:
* replay/WebInputs.json:

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (173033 => 173034)


--- trunk/Source/WebCore/ChangeLog	2014-08-27 23:24:19 UTC (rev 173033)
+++ trunk/Source/WebCore/ChangeLog	2014-08-27 23:29:28 UTC (rev 173034)
@@ -1,3 +1,16 @@
+2014-08-27  Brian J. Burg  <[email protected]>
+
+        Enums in ScrollTypes and PlatformWheelEvent should not use uint64_t storage
+        https://bugs.webkit.org/show_bug.cgi?id=136318
+
+        Reviewed by Simon Fraser.
+
+        This regressed in a replay-related patch. Revert back to uint8_t storage size.
+
+        * platform/PlatformWheelEvent.h:
+        * platform/ScrollTypes.h:
+        * replay/WebInputs.json:
+
 2014-08-27  Simon Fraser  <[email protected]>
 
         Speling is hard.

Modified: trunk/Source/WebCore/platform/PlatformWheelEvent.h (173033 => 173034)


--- trunk/Source/WebCore/platform/PlatformWheelEvent.h	2014-08-27 23:24:19 UTC (rev 173033)
+++ trunk/Source/WebCore/platform/PlatformWheelEvent.h	2014-08-27 23:29:28 UTC (rev 173034)
@@ -48,13 +48,13 @@
     // and synthesized in other cases where platforms generate line-by-line scrolling events.
     // The ScrollByPageWheelEvent indicates that the wheel event should scroll an entire page.  In this case WebCore's built in paging behavior is used to page
     // up and down (you get the same behavior as if the user was clicking in a scrollbar track to page up or page down).
-    enum PlatformWheelEventGranularity : uint64_t {
+    enum PlatformWheelEventGranularity : uint8_t {
         ScrollByPageWheelEvent,
         ScrollByPixelWheelEvent,
     };
 
 #if PLATFORM(COCOA)
-    enum PlatformWheelEventPhase : uint64_t {
+    enum PlatformWheelEventPhase : uint8_t {
         PlatformWheelEventPhaseNone        = 0,
         PlatformWheelEventPhaseBegan       = 1 << 0,
         PlatformWheelEventPhaseStationary  = 1 << 1,

Modified: trunk/Source/WebCore/platform/ScrollTypes.h (173033 => 173034)


--- trunk/Source/WebCore/platform/ScrollTypes.h	2014-08-27 23:24:19 UTC (rev 173033)
+++ trunk/Source/WebCore/platform/ScrollTypes.h	2014-08-27 23:29:28 UTC (rev 173034)
@@ -30,14 +30,14 @@
 
 namespace WebCore {
 
-    enum ScrollDirection : uint64_t {
+    enum ScrollDirection : uint8_t {
         ScrollUp,
         ScrollDown,
         ScrollLeft,
         ScrollRight
     };
 
-    enum ScrollLogicalDirection : uint64_t {
+    enum ScrollLogicalDirection : uint8_t {
         ScrollBlockDirectionBackward,
         ScrollBlockDirectionForward,
         ScrollInlineDirectionBackward,
@@ -103,7 +103,7 @@
         return ScrollUp;
     }
 
-    enum ScrollGranularity : uint64_t {
+    enum ScrollGranularity : uint8_t {
         ScrollByLine,
         ScrollByPage,
         ScrollByDocument,

Modified: trunk/Source/WebCore/replay/WebInputs.json (173033 => 173034)


--- trunk/Source/WebCore/replay/WebInputs.json	2014-08-27 23:24:19 UTC (rev 173033)
+++ trunk/Source/WebCore/replay/WebInputs.json	2014-08-27 23:29:28 UTC (rev 173034)
@@ -67,13 +67,13 @@
                 "header": "platform/PlatformWheelEvent.h"
             },
             {
-                "name": "PlatformWheelEventGranularity", "mode": "SCALAR", "storage": "uint64_t",
+                "name": "PlatformWheelEventGranularity", "mode": "SCALAR", "storage": "uint8_t",
                 "flags": ["ENUM"],
                 "values": ["ScrollByPageWheelEvent", "ScrollByPixelWheelEvent"],
                 "header": "platform/PlatformWheelEvent.h"
             },
             {
-                "name": "PlatformWheelEventPhase", "mode": "SCALAR", "storage": "uint64_t",
+                "name": "PlatformWheelEventPhase", "mode": "SCALAR", "storage": "uint8_t",
                 "flags": ["ENUM"],
                 "guard": "PLATFORM(COCOA)",
                 "values": [
@@ -92,13 +92,13 @@
                 "header": "plugins/PluginData.h"
             },
             {
-                "name": "ScrollDirection", "mode": "SCALAR", "storage": "uint64_t",
+                "name": "ScrollDirection", "mode": "SCALAR", "storage": "uint8_t",
                 "flags": ["ENUM"],
                 "values": ["ScrollUp", "ScrollDown", "ScrollLeft", "ScrollRight"],
                 "header": "platform/ScrollTypes.h"
             },
             {
-                "name": "ScrollGranularity", "mode": "SCALAR", "storage": "uint64_t",
+                "name": "ScrollGranularity", "mode": "SCALAR", "storage": "uint8_t",
                 "flags": ["ENUM"],
                 "values": [
                     "ScrollByLine",
@@ -110,7 +110,7 @@
                 "header": "platform/ScrollTypes.h"
             },
             {
-                "name": "ScrollLogicalDirection", "mode": "SCALAR", "storage": "uint64_t",
+                "name": "ScrollLogicalDirection", "mode": "SCALAR", "storage": "uint8_t",
                 "flags": ["ENUM"],
                 "values": [
                     "ScrollBlockDirectionBackward",

_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to