Changeset 128378 in webkit


Ignore:
Timestamp:
Sep 12, 2012 4:17:00 PM (12 years ago)
Author:
commit-queue@webkit.org
Message:

[chromium] Remove unused WebGestureEvent fields
https://bugs.webkit.org/show_bug.cgi?id=95496

Patch by Rick Byers <rbyers@chromium.org> on 2012-09-12
Reviewed by Adam Barth.

Remove the no-longer used fields from WebGestureEvent, now that
chromium has been updated to use the per-event-type fields instead.
This depends on crrev.com/156346 in chromium.

  • public/WebInputEvent.h:

(WebKit::WebGestureEvent::WebGestureEvent):

  • public/android/WebInputEventFactory.h:
  • src/WebInputEvent.cpp:

(SameSizeAsWebGestureEvent):

  • src/android/WebInputEventFactory.cpp:

(WebKit):
(WebKit::WebInputEventFactory::gestureEvent):

Location:
trunk/Source/WebKit/chromium
Files:
5 edited

Legend:

Unmodified
Added
Removed
  • trunk/Source/WebKit/chromium/ChangeLog

    r128377 r128378  
     12012-09-12  Rick Byers  <rbyers@chromium.org>
     2
     3        [chromium] Remove unused WebGestureEvent fields
     4        https://bugs.webkit.org/show_bug.cgi?id=95496
     5
     6        Reviewed by Adam Barth.
     7
     8        Remove the no-longer used fields from WebGestureEvent, now that
     9        chromium has been updated to use the per-event-type fields instead.
     10        This depends on crrev.com/156346 in chromium.
     11
     12        * public/WebInputEvent.h:
     13        (WebKit::WebGestureEvent::WebGestureEvent):
     14        * public/android/WebInputEventFactory.h:
     15        * src/WebInputEvent.cpp:
     16        (SameSizeAsWebGestureEvent):
     17        * src/android/WebInputEventFactory.cpp:
     18        (WebKit):
     19        (WebKit::WebInputEventFactory::gestureEvent):
     20
    1212012-09-12  Sheriff Bot  <webkit.review.bot@gmail.com>
    222
  • trunk/Source/WebKit/chromium/public/WebInputEvent.h

    r128273 r128378  
    379379    int globalY;
    380380
    381     // FIXME: These are currently overloaded. We're in the process of moving
    382     // to the union below. http://wkb.ug/93123
    383     float deltaX;
    384     float deltaY;
    385     WebRect boundingBox;
    386 
    387381    union {
    388382      struct {
     
    421415        , globalX(0)
    422416        , globalY(0)
    423         , deltaX(0.0f)
    424         , deltaY(0.0f)
    425417    {
    426418      memset(&data, 0, sizeof(data));
  • trunk/Source/WebKit/chromium/public/android/WebInputEventFactory.h

    r116996 r128378  
    8282                                                      int modifiers);
    8383
     84    WEBKIT_EXPORT static WebGestureEvent gestureEvent(WebInputEvent::Type,
     85                                                      double timeStampSeconds,
     86                                                      int x,
     87                                                      int y,
     88                                                      int modifiers);
    8489};
    8590
  • trunk/Source/WebKit/chromium/src/WebInputEvent.cpp

    r127711 r128378  
    6161
    6262struct SameSizeAsWebGestureEvent : public SameSizeAsWebInputEvent {
    63     int gestureData[14];
     63    int gestureData[8];
    6464};
    6565
  • trunk/Source/WebKit/chromium/src/android/WebInputEventFactory.cpp

    r127711 r128378  
    149149// WebGestureEvent ------------------------------------------------------------
    150150
     151// FIXME: remove this obsolete version
    151152WebGestureEvent WebInputEventFactory::gestureEvent(WebInputEvent::Type type,
    152153                                                   double timeStampSeconds,
    153154                                                   int x,
    154155                                                   int y,
    155                                                    float deltaX,
    156                                                    float deltaY,
     156                                                   float,
     157                                                   float,
     158                                                   int modifiers) {
     159    return gestureEvent(type, timeStampSeconds, x, y, modifiers);
     160}
     161
     162WebGestureEvent WebInputEventFactory::gestureEvent(WebInputEvent::Type type,
     163                                                   double timeStampSeconds,
     164                                                   int x,
     165                                                   int y,
    157166                                                   int modifiers)
    158167{
     
    162171    result.x = x;
    163172    result.y = y;
    164     result.deltaX = deltaX;
    165     result.deltaY = deltaY;
    166173    result.timeStampSeconds = timeStampSeconds;
    167174    result.modifiers = modifiers;
Note: See TracChangeset for help on using the changeset viewer.