Changeset 141640 in webkit


Ignore:
Timestamp:
Feb 1, 2013 1:56:29 PM (11 years ago)
Author:
caio.oliveira@openbossa.org
Message:

[Gtk] [WK2] Fix build after r141619
https://bugs.webkit.org/show_bug.cgi?id=108687

Reviewed by Benjamin Poulain.

Take a reference instead of a pointer for decoding functions.

  • Platform/gtk/SharedMemoryGtk.cpp:

(WebKit::SharedMemory::Handle::decode):

  • Shared/gtk/ArgumentCodersGtk.cpp:

(CoreIPC::decodeImage):
(CoreIPC::decodeDataObject):
(CoreIPC::::decode):
(CoreIPC::decodeGKeyFile):
(CoreIPC::decode):

  • Shared/gtk/ArgumentCodersGtk.h:
  • Shared/gtk/LayerTreeContextGtk.cpp:

(WebKit::LayerTreeContext::decode):

Location:
trunk/Source/WebKit2
Files:
5 edited

Legend:

Unmodified
Added
Removed
  • trunk/Source/WebKit2/ChangeLog

    r141639 r141640  
     12013-02-01  Caio Marcelo de Oliveira Filho  <caio.oliveira@openbossa.org>
     2
     3        [Gtk] [WK2] Fix build after r141619
     4        https://bugs.webkit.org/show_bug.cgi?id=108687
     5
     6        Reviewed by Benjamin Poulain.
     7
     8        Take a reference instead of a pointer for decoding functions.
     9
     10        * Platform/gtk/SharedMemoryGtk.cpp:
     11        (WebKit::SharedMemory::Handle::decode):
     12        * Shared/gtk/ArgumentCodersGtk.cpp:
     13        (CoreIPC::decodeImage):
     14        (CoreIPC::decodeDataObject):
     15        (CoreIPC::::decode):
     16        (CoreIPC::decodeGKeyFile):
     17        (CoreIPC::decode):
     18        * Shared/gtk/ArgumentCodersGtk.h:
     19        * Shared/gtk/LayerTreeContextGtk.cpp:
     20        (WebKit::LayerTreeContext::decode):
     21
    1222013-02-01  Caio Marcelo de Oliveira Filho  <caio.oliveira@openbossa.org>
    223
  • trunk/Source/WebKit2/Platform/gtk/SharedMemoryGtk.cpp

    r132361 r141640  
    4747}
    4848
    49 bool SharedMemory::Handle::decode(CoreIPC::ArgumentDecoder* decoder, Handle& handle)
     49bool SharedMemory::Handle::decode(CoreIPC::ArgumentDecoder& decoder, Handle& handle)
    5050{
    5151    notImplemented();
  • trunk/Source/WebKit2/Shared/gtk/ArgumentCodersGtk.cpp

    r133075 r141640  
    5858}
    5959
    60 static bool decodeImage(ArgumentDecoder* decoder, GRefPtr<GdkPixbuf>& pixbuf)
     60static bool decodeImage(ArgumentDecoder& decoder, GRefPtr<GdkPixbuf>& pixbuf)
    6161{
    6262    ShareableBitmap::Handle handle;
    63     if (!decoder->decode(handle))
     63    if (!decoder.decode(handle))
    6464        return false;
    6565
     
    114114}
    115115
    116 static bool decodeDataObject(ArgumentDecoder* decoder, RefPtr<DataObjectGtk>& dataObject)
     116static bool decodeDataObject(ArgumentDecoder& decoder, RefPtr<DataObjectGtk>& dataObject)
    117117{
    118118    RefPtr<DataObjectGtk> data = DataObjectGtk::create();
    119119
    120120    bool hasText;
    121     if (!decoder->decode(hasText))
     121    if (!decoder.decode(hasText))
    122122        return false;
    123123    if (hasText) {
    124124        String text;
    125         if (!decoder->decode(text))
     125        if (!decoder.decode(text))
    126126            return false;
    127127        data->setText(text);
     
    129129
    130130    bool hasMarkup;
    131     if (!decoder->decode(hasMarkup))
     131    if (!decoder.decode(hasMarkup))
    132132        return false;
    133133    if (hasMarkup) {
    134134        String markup;
    135         if (!decoder->decode(markup))
     135        if (!decoder.decode(markup))
    136136            return false;
    137137        data->setMarkup(markup);
     
    139139
    140140    bool hasURL;
    141     if (!decoder->decode(hasURL))
     141    if (!decoder.decode(hasURL))
    142142        return false;
    143143    if (hasURL) {
    144144        String url;
    145         if (!decoder->decode(url))
     145        if (!decoder.decode(url))
    146146            return false;
    147147        data->setURL(KURL(KURL(), url), String());
     
    149149
    150150    bool hasURIList;
    151     if (!decoder->decode(hasURIList))
     151    if (!decoder.decode(hasURIList))
    152152        return false;
    153153    if (hasURIList) {
    154154        String uriList;
    155         if (!decoder->decode(uriList))
     155        if (!decoder.decode(uriList))
    156156            return false;
    157157        data->setURIList(uriList);
     
    159159
    160160    bool hasImage;
    161     if (!decoder->decode(hasImage))
     161    if (!decoder.decode(hasImage))
    162162        return false;
    163163    if (hasImage) {
     
    186186}
    187187
    188 bool ArgumentCoder<DragData>::decode(ArgumentDecoder* decoder, DragData& dragData)
     188bool ArgumentCoder<DragData>::decode(ArgumentDecoder& decoder, DragData& dragData)
    189189{
    190190    IntPoint clientPosition;
    191     if (!decoder->decode(clientPosition))
     191    if (!decoder.decode(clientPosition))
    192192        return false;
    193193
    194194    IntPoint globalPosition;
    195     if (!decoder->decode(globalPosition))
     195    if (!decoder.decode(globalPosition))
    196196        return false;
    197197
    198198    uint64_t sourceOperationMask;
    199     if (!decoder->decode(sourceOperationMask))
     199    if (!decoder.decode(sourceOperationMask))
    200200        return false;
    201201
    202202    uint64_t flags;
    203     if (!decoder->decode(flags))
     203    if (!decoder.decode(flags))
    204204        return false;
    205205
    206206    bool hasPlatformData;
    207     if (!decoder->decode(hasPlatformData))
     207    if (!decoder.decode(hasPlatformData))
    208208        return false;
    209209
     
    227227}
    228228
    229 static bool decodeGKeyFile(ArgumentDecoder* decoder, GOwnPtr<GKeyFile>& keyFile)
     229static bool decodeGKeyFile(ArgumentDecoder& decoder, GOwnPtr<GKeyFile>& keyFile)
    230230{
    231231    DataReference dataReference;
    232     if (!decoder->decode(dataReference))
     232    if (!decoder.decode(dataReference))
    233233        return false;
    234234
     
    252252}
    253253
    254 bool decode(ArgumentDecoder* decoder, GRefPtr<GtkPrintSettings>& printSettings)
     254bool decode(ArgumentDecoder& decoder, GRefPtr<GtkPrintSettings>& printSettings)
    255255{
    256256    GOwnPtr<GKeyFile> keyFile;
     
    275275}
    276276
    277 bool decode(ArgumentDecoder* decoder, GRefPtr<GtkPageSetup>& pageSetup)
     277bool decode(ArgumentDecoder& decoder, GRefPtr<GtkPageSetup>& pageSetup)
    278278{
    279279    GOwnPtr<GKeyFile> keyFile;
  • trunk/Source/WebKit2/Shared/gtk/ArgumentCodersGtk.h

    r133075 r141640  
    4242template<> struct ArgumentCoder<WebCore::DragData> {
    4343    static void encode(ArgumentEncoder&, const WebCore::DragData&);
    44     static bool decode(ArgumentDecoder*, WebCore::DragData&);
     44    static bool decode(ArgumentDecoder&, WebCore::DragData&);
    4545};
    4646
    4747void encode(ArgumentEncoder&, GtkPrintSettings*);
    48 bool decode(ArgumentDecoder*, GRefPtr<GtkPrintSettings>&);
     48bool decode(ArgumentDecoder&, GRefPtr<GtkPrintSettings>&);
    4949
    5050void encode(ArgumentEncoder&, GtkPageSetup*);
    51 bool decode(ArgumentDecoder*, GRefPtr<GtkPageSetup>&);
     51bool decode(ArgumentDecoder&, GRefPtr<GtkPageSetup>&);
    5252
    5353} // namespace CoreIPC
  • trunk/Source/WebKit2/Shared/gtk/LayerTreeContextGtk.cpp

    r132361 r141640  
    4646}
    4747
    48 bool LayerTreeContext::decode(CoreIPC::ArgumentDecoder* decoder, LayerTreeContext& context)
     48bool LayerTreeContext::decode(CoreIPC::ArgumentDecoder& decoder, LayerTreeContext& context)
    4949{
    50     return decoder->decode(context.windowHandle);
     50    return decoder.decode(context.windowHandle);
    5151}
    5252
Note: See TracChangeset for help on using the changeset viewer.