Bug 114299 - [GTK] --minimal build fails with error: class WebCore::EventHandler' has no member named 'dragSourceEndedAt
Summary: [GTK] --minimal build fails with error: class WebCore::EventHandler' has no m...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKitGTK (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks: 113660
  Show dependency treegraph
 
Reported: 2013-04-09 12:08 PDT by Ed Bartosh
Modified: 2013-04-30 08:57 PDT (History)
5 users (show)

See Also:


Attachments
Patch (3.73 KB, patch)
2013-04-11 14:44 PDT, Ed Bartosh
no flags Details | Formatted Diff | Diff
Patch (3.99 KB, patch)
2013-04-13 04:42 PDT, Ed Bartosh
no flags Details | Formatted Diff | Diff
Patch (4.01 KB, patch)
2013-04-29 10:41 PDT, Ed Bartosh
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Ed Bartosh 2013-04-09 12:08:15 PDT
Here is the error:
Failed to run "['Tools/Scripts/build-webkit', '--release', '--gtk', '--update-gtk', '--no-webkit2', '--makeargs="-j2"']" exit_code: 2
orm/gtk/libPlatformGtk_la-ContextMenuItemGtk.lo
In file included from ./autotoolsconfig.h:254:0,
                 from ../../Source/WebCore/config.h:26,
                 from ../../Source/WebKit/gtk/webkit/webkitwebview.cpp:31:
./WebKitFeatures.h:92:0: warning: "ENABLE_SPELLCHECK" redefined [enabled by default]
In file included from ../../Source/WebCore/config.h:26:0,
                 from ../../Source/WebKit/gtk/webkit/webkitwebview.cpp:31:
./autotoolsconfig.h:19:0: note: this is the location of the previous definition
../../Source/WebKit/gtk/webkit/webkitwebview.cpp: In function 'void webkit_web_view_drag_end(GtkWidget*, GdkDragContext*)':
../../Source/WebKit/gtk/webkit/webkitwebview.cpp:1524:28: error: 'class WebCore::EventHandler' has no member named 'dragSourceEndedAt'
../../Source/WebKit/gtk/webkit/webkitwebview.cpp: In function 'void dragExitedCallback(GtkWidget*, WebCore::DragData*, bool)':
../../Source/WebKit/gtk/webkit/webkitwebview.cpp:1537:128: error: 'class WebCore::Page' has no member named 'dragController'
../../Source/WebKit/gtk/webkit/webkitwebview.cpp:1538:124: error: 'class WebCore::Page' has no member named 'dragController'
../../Source/WebKit/gtk/webkit/webkitwebview.cpp: In function 'gboolean webkit_web_view_drag_motion(GtkWidget*, GdkDragContext*, gint, gint, guint)':
../../Source/WebKit/gtk/webkit/webkitwebview.cpp:1555:46: error: 'class WebCore::Page' has no member named 'dragController'
../../Source/WebKit/gtk/webkit/webkitwebview.cpp: In function 'void webkit_web_view_drag_data_received(GtkWidget*, GdkDragContext*, gint, gint, GtkSelectionData*, guint, guint)':
../../Source/WebKit/gtk/webkit/webkitwebview.cpp:1569:46: error: 'class WebCore::Page' has no member named 'dragController'
../../Source/WebKit/gtk/webkit/webkitwebview.cpp: In function 'gboolean webkit_web_view_drag_drop(GtkWidget*, GdkDragContext*, gint, gint, guint)':
../../Source/WebKit/gtk/webkit/webkitwebview.cpp:1582:20: error: 'class WebCore::Page' has no member named 'dragController'
make[1]: *** [Source/WebKit/gtk/webkit/libwebkitgtk_3_0_la-webkitwebview.lo] Error 1
make[1]: *** Waiting for unfinished jobs....
In file included from ./autotoolsconfig.h:254:0,
                 from ../../Source/JavaScriptCore/config.h:26,
                 from ../../Source/WebCore/platform/gtk/ContextMenuItemGtk.cpp:21:
./WebKitFeatures.h:92:0: warning: "ENABLE_SPELLCHECK" redefined [enabled by default]
In file included from ../../Source/JavaScriptCore/config.h:26:0,
                 from ../../Source/WebCore/platform/gtk/ContextMenuItemGtk.cpp:21:
./autotoolsconfig.h:19:0: note: this is the location of the previous definition
make[1]: Leaving directory `/srv/ews/WebKit/WebKitBuild/Release'
make: *** [all] Error 2
Comment 1 Ed Bartosh 2013-04-11 14:44:04 PDT
Created attachment 197674 [details]
Patch
Comment 2 Philippe Normand 2013-04-12 12:58:51 PDT
Comment on attachment 197674 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=197674&action=review

> Source/WebKit/gtk/webkit/webkitwebview.cpp:3111
> +#endif

I think you need an #else here to set the drag_* to NULL.
Comment 3 Ed Bartosh 2013-04-13 04:42:37 PDT
Created attachment 197934 [details]
Patch
Comment 4 Ed Bartosh 2013-04-13 04:43:37 PDT
Updated according to reviewer's suggestions.
Comment 5 Ed Bartosh 2013-04-29 10:41:45 PDT
Created attachment 200026 [details]
Patch
Comment 6 WebKit Commit Bot 2013-04-30 08:57:43 PDT
Comment on attachment 200026 [details]
Patch

Clearing flags on attachment: 200026

Committed r149371: <http://trac.webkit.org/changeset/149371>
Comment 7 WebKit Commit Bot 2013-04-30 08:57:46 PDT
All reviewed patches have been landed.  Closing bug.