2010-12-10 Joone Hur <joone@kldp.org>
[WebKit-https.git] / WebKit / gtk / ChangeLog
1 2010-12-10  Joone Hur  <joone@kldp.org>
2
3         Reviewed by Eric Seidel.
4
5         [GTK] Add DRT support for pageProperty, isPageBoxVisible, pageSizeAndMarginsInPixels, and addUserStyleSheet
6         https://bugs.webkit.org/show_bug.cgi?id=50783
7         
8         This patch allows to unskip the following test cases:
9         printing/page-rule-selection.html
10         printing/page-format-data.html
11
12         * WebCoreSupport/DumpRenderTreeSupportGtk.cpp:
13         (DumpRenderTreeSupportGtk::pageProperty): Added.
14         (DumpRenderTreeSupportGtk::isPageBoxVisible): Ditto.
15         (DumpRenderTreeSupportGtk::pageSizeAndMarginsInPixels): Ditto.
16         (DumpRenderTreeSupportGtk::addUserStyleSheet): Ditto.
17         * WebCoreSupport/DumpRenderTreeSupportGtk.h:
18
19 2010-12-09  Gustavo Noronha Silva  <gustavo.noronha@collabora.co.uk>
20
21         Reviewed by Martin Robinson.
22
23         [GTK] Split webkitprivate.{cpp,h} in more manageable chunks
24         https://bugs.webkit.org/show_bug.cgi?id=50698
25
26         Split private bits of WebKitWebInspector to its own private
27         header, and to its implementation file.
28
29         * WebCoreSupport/InspectorClientGtk.cpp:
30         * webkit/webkitprivate.cpp:
31         * webkit/webkitprivate.h:
32         * webkit/webkitwebinspector.cpp:
33         (isSchemaAvailable):
34         (inspectorGSettings):
35         * webkit/webkitwebinspectorprivate.h: Added.
36         * webkit/webkitwebview.cpp:
37
38 2010-12-09  Gustavo Noronha Silva  <gustavo.noronha@collabora.co.uk>
39
40         Reviewed by Martin Robinson.
41
42         [GTK] Split webkitprivate.{cpp,h} in more manageable chunks
43         https://bugs.webkit.org/show_bug.cgi?id=50698
44
45         Split private bits of WebKitNetworkRequest to its own private
46         header, and to its implementation file. Also take the opportunity
47         to use PlatformRefPtr for NetworkRequest where it makes sense, and
48         make the creation less verbose by adding kitNew.
49
50         * WebCoreSupport/FrameLoaderClientGtk.cpp:
51         (WebKit::FrameLoaderClient::dispatchWillSendRequest):
52         (WebKit::FrameLoaderClient::dispatchDecidePolicyForMIMEType):
53         (WebKit::FrameLoaderClient::dispatchDecidePolicyForNewWindowAction):
54         (WebKit::FrameLoaderClient::dispatchDecidePolicyForNavigationAction):
55         (WebKit::FrameLoaderClient::download):
56         (WebKit::FrameLoaderClient::startDownload):
57         * webkit/webkitdownload.cpp:
58         * webkit/webkitnetworkrequest.cpp:
59         (WebKit::kitNew):
60         (WebKit::core):
61         * webkit/webkitnetworkrequestprivate.h: Added.
62         * webkit/webkitprivate.cpp:
63         * webkit/webkitprivate.h:
64         * webkit/webkitwebdatasource.cpp:
65         (webkit_web_data_source_get_initial_request):
66         (webkit_web_data_source_get_request):
67         * webkit/webkitwebframe.cpp:
68
69 2010-12-09  Gustavo Noronha Silva  <gustavo.noronha@collabora.co.uk>
70
71         Reviewed by Martin Robinson.
72
73         [GTK] Split webkitprivate.{cpp,h} in more manageable chunks
74         https://bugs.webkit.org/show_bug.cgi?id=50698
75
76         Split private bits of WebKitWebFrame to its own private header, and
77         to its implementation file.
78
79         * GNUmakefile.am:
80         * WebCoreSupport/ChromeClientGtk.cpp:
81         * WebCoreSupport/DragClientGtk.cpp:
82         * WebCoreSupport/DumpRenderTreeSupportGtk.cpp:
83         * WebCoreSupport/FrameLoaderClientGtk.cpp:
84         * webkit/webkitprivate.cpp:
85         * webkit/webkitprivate.h:
86         * webkit/webkitwebdatasource.cpp:
87         * webkit/webkitwebframe.cpp:
88         (WebKit::getViewFromFrame):
89         (WebKit::core):
90         (WebKit::kit):
91         * webkit/webkitwebframeprivate.h: Added.
92         * webkit/webkitwebpolicydecision.cpp:
93         * webkit/webkitwebview.cpp:
94
95 2010-12-09  Antonio Gomes  <agomes@rim.com>
96
97         Reviewed by Martin Robinson.
98
99         [Gtk] Populate DumpRenderTreeSupportGtk (part IV)
100         https://bugs.webkit.org/show_bug.cgi?id=48429
101
102         Moved also webkit_worker_thread_count() to DumpRenderTreeSupportGtk,
103         since it was only used by DRT for now.
104
105         Since this method was the only one in webkitworker.cpp, the file was
106         also removed.
107
108         * GNUmakefile.am:
109         * WebCoreSupport/DumpRenderTreeSupportGtk.cpp:
110         (DumpRenderTreeSupportGtk::workerThreadCount):
111         * WebCoreSupport/DumpRenderTreeSupportGtk.h:
112         * webkit/webkitprivate.h:
113         * webkit/webkitworkers.cpp: Removed.
114
115 2010-12-06  Philippe Normand  <pnormand@igalia.com>
116
117         Reviewed by Martin Robinson.
118
119         [GTK] Support the Mozilla-style Fullscreen Javascript API
120         https://bugs.webkit.org/show_bug.cgi?id=50572
121
122         Fullscreen API support. It is controlled at runtime using a new
123         setting called enable-fullscreen, set to FALSE by default.
124
125         * WebCoreSupport/ChromeClientGtk.cpp:
126         (WebKit::ChromeClient::supportsFullScreenForElement):
127         (WebKit::ChromeClient::enterFullScreenForElement):
128         (WebKit::ChromeClient::exitFullScreenForElement):
129         * WebCoreSupport/ChromeClientGtk.h:
130         * webkit/webkitwebsettings.cpp:
131         (webkit_web_settings_class_init):
132         (webkit_web_settings_set_property):
133         (webkit_web_settings_get_property):
134         (webkit_web_settings_copy):
135         * webkit/webkitwebview.cpp:
136         (webkit_web_view_update_settings):
137
138 2010-12-09  Joone Hur  <joone@kldp.org>
139
140         Reviewed by Martin Robinson.
141
142         [GTK] Support for viewport meta tag
143         https://bugs.webkit.org/show_bug.cgi?id=45443
144
145         This patch adds a new gobject to offer the viewport properties to user agents.
146         It is WebKitViewportAttributes that contains the viewport size, initial scale with limits, 
147         and information about whether a user is able to scale the contents in the viewport.
148         
149         This feature was introduced by Safari on the iPhone. For more information on 
150         the viewport properties, refer to the Safari reference library at 
151         http://developer.apple.com/safari/library/documentation/appleapplications/reference/safarihtmlref/articles/metatags.html
152
153         * GNUmakefile.am: Added webkitviewportattributes.h webkitviewportattributes.cpp. 
154         * WebCoreSupport/ChromeClientGtk.cpp:
155         (WebKit::ChromeClient::dispatchViewportDataDidChange): Added this callback to know whether the viewport arguments is available.
156         * WebCoreSupport/ChromeClientGtk.h:
157         * WebCoreSupport/DumpRenderTreeSupportGtk.cpp:
158         (DumpRenderTreeSupportGtk::dumpConfigurationForViewport): Added this function to compute the viewport attributes with the given availableWidth and availableHeight.
159         * WebCoreSupport/DumpRenderTreeSupportGtk.h:
160         * WebCoreSupport/FrameLoaderClientGtk.cpp: 
161         (WebKit::postCommitFrameViewSetup): Invalidate the viewport attributes.
162         * webkit/webkit.h:
163         * webkit/webkitdefines.h:
164         * webkit/webkitprivate.h: Added webkitViewportAttributesRecompute as a private function.
165         * webkit/webkitviewportattributes.cpp: Added.
166         (webkit_viewport_attributes_class_init):
167         (webkit_viewport_attributes_init):
168         (webkit_viewport_attributes_get_property):
169         (webkit_viewport_attributes_set_property):
170         (webkitViewportAttributesRecompute): Recompute the optimal viewport attributes and emit the viewport-attributes-changed signal.
171         (webkit_viewport_attributes_recompute): a public API for calling webkitViewportAttributesRecompute.
172         * webkit/webkitviewportattributes.h: Added.
173         * webkit/webkitwebview.cpp:
174         (webkit_web_view_get_property): Added PROP_VIEWPORT_ATTRIBUTES.
175         (webkit_web_view_dispose): Deallocate the WebKitViewportAttributes instance.
176         (webkit_web_view_class_init): Added the viewport-attributes-recompute-requested and viewport-attributes-changed signals.
177         (webkit_web_view_init): Set the WebKitViewportAttributes instance.
178         (webkit_web_view_get_viewport_attributes): Added a getter to obtain the WebKitViewportAttributes instance held by the given WebKitWebView.
179         * webkit/webkitwebview.h:
180         * webkit/webkitwebviewprivate.h: Added WebKitViewportAttributes as a private variable.
181
182 2010-12-09  Xan Lopez  <xlopez@igalia.com>
183
184         Reviewed by Martin Robinson.
185
186         Fix 'make install' woes with some versions of automake.
187
188         * GNUmakefile.am: use $(top_builddir) instead of $(builddir),
189         since we actually want a valid path for the file regardless of the
190         place from where the script is called.
191
192 2010-12-08  Gustavo Noronha Silva  <gustavo.noronha@collabora.co.uk>
193
194         Build fix. Take webView{Enter,Exit}Fullscreen back out of the
195         WebKit namespace, and remove a function signature that got in by
196         mistake because of a bad merge.
197
198         * webkit/webkitprivate.h:
199         * webkit/webkitwebview.cpp:
200         (WebKit::core):
201         (WebKit::kit):
202
203 2010-12-08  Brian Weinstein  <bweinstein@apple.com>
204
205         Reviewed by John Sullivan.
206
207         Layering Violation in ContextMenu - member variable of type HitTestResult
208         https://bugs.webkit.org/show_bug.cgi?id=50586
209         
210         Update users of ContextMenu and ContextMenuController to match where the new functions 
211         are located.
212
213         * WebCoreSupport/ContextMenuClientGtk.cpp:
214         (WebKit::ContextMenuClient::getCustomMenuFromDefaultItems):
215
216 2010-12-08  Martin Robinson  <mrobinson@igalia.com>
217
218         Reviewed by John Sullivan.
219
220         [GTK] Layering violation in ContextMenuGtk.cpp
221         https://bugs.webkit.org/show_bug.cgi?id=50676
222
223         ContextMenuGtk accesses the page's ContextMenuController, which is a layering
224         violation. This was fixed for Mac with change r73469. We make a similar fix here
225         by moving the "activated" signal connection to the WebKit layer.
226
227         * webkit/webkitwebview.cpp:
228         (popupMenuItemActivated): Have ContextMenuController handle this signal.
229         (webkit_web_view_forward_context_menu_event): For all items in the untouched
230         context menu, connect their "activated" signal to the new popupMenuItemActivated
231         handler.
232
233 2010-12-08  Antonio Gomes  <agomes@rim.com>
234
235         Reviewed by Martin Robinson.
236
237         [Gtk] Populate DumpRenderTreeSupportGtk (part III)
238         https://bugs.webkit.org/show_bug.cgi?id=48429
239
240         Adjusted some call sites of previously internal API declared
241         webkitprivate.h that have moved to DumpRenderTreeSupportGtk.
242
243         * WebCoreSupport/DumpRenderTreeSupportGtk.cpp:
244         (DumpRenderTreeSupportGtk::gcCollectJavascriptObjects):
245         (DumpRenderTreeSupportGtk::gcCollectJavascriptObjectsOnAlternateThread):
246         (DumpRenderTreeSupportGtk::gcCountJavascriptObjects):
247         (DumpRenderTreeSupportGtk::layoutFrame):
248         * WebCoreSupport/DumpRenderTreeSupportGtk.h:
249         * webkit/webkitprivate.h:
250         * webkit/webkitwebframe.cpp:
251
252 2010-12-08  Gustavo Noronha Silva  <gustavo.noronha@collabora.co.uk>
253
254         Reviewed by Martin Robinson.
255
256         [GTK] Split webkitprivate.{cpp,h} in more manageable chunks
257         https://bugs.webkit.org/show_bug.cgi?id=50698
258
259         Split private bits of WebKitWebView to its own private header, and
260         to its implementation file. Take the opportunity to fix includes
261         sorting to comply with the style guide.
262
263         * WebCoreSupport/ChromeClientGtk.cpp:
264         (WebKit::ChromeClient::enterFullscreenForNode):
265         (WebKit::ChromeClient::exitFullscreenForNode):
266         * WebCoreSupport/ContextMenuClientGtk.cpp:
267         * WebCoreSupport/DragClientGtk.cpp:
268         * WebCoreSupport/DumpRenderTreeSupportGtk.cpp:
269         * WebCoreSupport/EditorClientGtk.cpp:
270         * WebCoreSupport/FrameLoaderClientGtk.cpp:
271         * WebCoreSupport/InspectorClientGtk.cpp:
272         * WebCoreSupport/PasteboardHelperGtk.cpp:
273         * webkit/webkitprivate.cpp:
274         (webkit_init):
275         * webkit/webkitprivate.h:
276         * webkit/webkitwebbackforwardlist.cpp:
277         * webkit/webkitwebdatasource.cpp:
278         * webkit/webkitwebframe.cpp:
279         * webkit/webkitwebview.cpp:
280         (WebKit::core):
281         (WebKit::kit):
282         (WebKit::webViewEnterFullscreen):
283         (WebKit::webViewExitFullscreen):
284         * webkit/webkitwebviewprivate.h: Added.
285
286 2010-12-08  Gustavo Noronha Silva  <gustavo.noronha@collabora.co.uk>
287
288         Reviewed by Xan Lopez.
289
290         The storage for the subresources is a GHashTable which does not
291         guarantee order, so we need to make our checking more resilient.
292
293         * tests/testwebresource.c:
294         (test_web_resource_sub_resource_loading):
295
296 2010-12-08  Alejandro G. Castro  <alex@igalia.com>
297
298         Reviewed by Gustavo Noronha Silva.
299
300         Fix assertion in the API test, we are requesting the favicon and
301         the subresource so we have to make sure we are assigning just the
302         first subresource.
303
304         * tests/testwebresource.c:
305         (resource_request_starting_sub_cb):
306
307 2010-12-07  Gustavo Noronha Silva  <gustavo.noronha@collabora.co.uk>
308
309         Reviewed by Martin Robinson.
310
311         [GTK] clears the subresources prematurely
312         https://bugs.webkit.org/show_bug.cgi?id=50634
313
314         Make adding the main resource explicit, and only clear
315         subresources when the new load is committed, avoid clearing them
316         prematurely.
317
318         * WebCoreSupport/FrameLoaderClientGtk.cpp:
319         (WebKit::FrameLoaderClient::assignIdentifierToInitialRequest):
320         (WebKit::FrameLoaderClient::provisionalLoadStarted):
321         (WebKit::FrameLoaderClient::prepareForDataSourceReplacement):
322         (WebKit::postCommitFrameViewSetup):
323         * tests/testwebdatasource.c:
324         (notify_load_status_lifetime_cb):
325         (test_webkit_web_data_source_lifetime):
326         (main):
327         * webkit/webkitprivate.h:
328         * webkit/webkitwebview.cpp:
329         (webkit_web_view_add_main_resource):
330         (webkit_web_view_add_resource):
331         (webkit_web_view_clear_resources):
332
333 2010-12-07  Gustavo Noronha Silva  <gustavo.noronha@collabora.co.uk>
334
335         Reviewed by Xan Lopez.
336
337         [GTK] testwebdatasource - unreachable test is broken
338         https://bugs.webkit.org/show_bug.cgi?id=50629
339
340         Fix, and disable testwebdatasource's unreachable test (it is not
341         passing now that it actually tests what it should). This also
342         fixes the wait timer to actually work as expected.
343
344         * tests/testwebdatasource.c:
345         (notify_load_status_unreachable_cb):
346         (notify_load_status_cb):
347         (test_webkit_web_data_source):
348
349 2010-12-07  Martin Robinson  <mrobinson@igalia.com>
350
351         Unreviewed, rolling out r73392.
352         http://trac.webkit.org/changeset/73392
353         https://bugs.webkit.org/show_bug.cgi?id=50489
354
355         This commit caused crashes on the GTK+ bots
356
357         * WebCoreSupport/FrameLoaderClientGtk.cpp:
358         (WebKit::FrameLoaderClient::finishedLoading):
359         * webkit/webkitwebview.cpp:
360         (webkit_web_view_get_encoding):
361
362 2010-12-06  Nate Chapin  <japhet@chromium.org>
363
364         Reviewed by Adam Barth.
365
366         Update calls to DocumentWriter.
367         https://bugs.webkit.org/show_bug.cgi?id=50489
368
369         * WebCoreSupport/FrameLoaderClientGtk.cpp:
370         (WebKit::FrameLoaderClient::finishedLoading):
371         * webkit/webkitwebview.cpp:
372         (webkit_web_view_get_encoding):
373
374 2010-12-06  Martin Robinson  <mrobinson@igalia.com>
375
376         Reviewed by Xan Lopez.
377
378         [GTK] Build is broken after r73353
379         https://bugs.webkit.org/show_bug.cgi?id=50568
380
381         Fix the gtk-doc build by making all members of the cache model
382         enum public. gtk-doc does not support making only some enum
383         members private. We should also expose it so that users querying
384         the cache model do not get mysterious, undocumented results.
385
386         * webkit/webkitwebview.h: Fix gtk-doc for the cache model enum.
387
388 2010-12-06  Derek Weitzel  <djw8605@gmail.com>
389
390         Reviewed by Martin Robinson.
391
392         [GTK] API Documentation referencing depreciated GTK_WIDGET_TOPLEVEL
393         https://bugs.webkit.org/show_bug.cgi?id=50512
394
395         Fixed reference to depreciated GTK_WIDGET_TOPLEVEL.
396
397         * webkit/webkitwebview.cpp:
398
399 2010-12-06  Joone Hur  <joone@kldp.org>
400
401         Reviewed by Xan Lopez.
402
403         [GTK] The webkit cache model needs to be set when WebFrameLoaderClient::didPerformFirstNavigation() is called
404         https://bugs.webkit.org/show_bug.cgi?id=50430
405
406         * WebCoreSupport/FrameLoaderClientGtk.cpp:
407         (WebKit::FrameLoaderClient::didPerformFirstNavigation): Set the webview cache model.
408         * webkit/webkitprivate.cpp:
409         (webkit_init): Removed the code of setting the webview cache model.
410
411 2010-12-04  Antonio Gomes  <agomes@rim.com>
412
413         Reviewed by Martin Robinson.
414
415         [Gtk] Populate DumpRenderTreeSupportGtk (Part II)
416         https://bugs.webkit.org/show_bug.cgi?id=48429
417
418         Moved more private method declared with WEBKIT_API to
419         DumpRenderTreeSupportGtk, since they were only being used by DRT.
420
421         * WebCoreSupport/DumpRenderTreeSupportGtk.cpp:
422         (DumpRenderTreeSupportGtk::executeCoreCommandByName):
423         (DumpRenderTreeSupportGtk::isCommandEnabled):
424         (DumpRenderTreeSupportGtk::whiteListAccessFromOrigin):
425         (DumpRenderTreeSupportGtk::resetOriginAccessWhiteLists):
426         * WebCoreSupport/DumpRenderTreeSupportGtk.h:
427         * webkit/webkitprivate.cpp:
428         * webkit/webkitprivate.h:
429         * webkit/webkitwebview.cpp:
430
431 2010-11-08  Antonio Gomes  <agomes@rim.com>
432
433         Reviewed by Martin Robinson.
434
435         [Gtk] Populate DumpRenderTreeSupportGtk
436         https://bugs.webkit.org/show_bug.cgi?id=48429
437
438         Moved most of the webkit_web_frame* methods defined as private APIs in webkitprivate.h
439         to DumpRenderTreeSupportGtk, as static class methods. These methods had
440         not gone throught the formal API review process and were only being used by DRT for now.
441
442         The essence of each original method name were kept. For example
443         webkit_web_frame_get_children(...) was renamed to DumpRenderTreeSupportGtk::getFrameChildren(...),
444         and so on.
445
446         Also some of the method bodies were changed to return non-glib types (gchar* -> CString).
447
448         * WebCoreSupport/DumpRenderTreeSupportGtk.cpp:
449         (DumpRenderTreeSupportGtk::getFrameChildren):
450         (DumpRenderTreeSupportGtk::getInnerText):
451         (DumpRenderTreeSupportGtk::dumpRenderTree):
452         (DumpRenderTreeSupportGtk::counterValueForElementById):
453         (DumpRenderTreeSupportGtk::pageNumberForElementById):
454         (DumpRenderTreeSupportGtk::numberOfPagesForFrame):
455         (DumpRenderTreeSupportGtk::getPendingUnloadEventCount):
456         (DumpRenderTreeSupportGtk::pauseAnimation):
457         (DumpRenderTreeSupportGtk::pauseTransition):
458         (DumpRenderTreeSupportGtk::pauseSvgAnimation):
459         (DumpRenderTreeSupportGtk::markerTextForListItem):
460         (DumpRenderTreeSupportGtk::numberOfActiveAnimations):
461         (DumpRenderTreeSupportGtk::suspendAnimations):
462         (DumpRenderTreeSupportGtk::resumeAnimations):
463         (DumpRenderTreeSupportGtk::clearMainFrameName):
464         (DumpRenderTreeSupportGtk::getFocusedAccessibleElement):
465         * WebCoreSupport/DumpRenderTreeSupportGtk.h:
466         * webkit/webkitprivate.h:
467         * webkit/webkitwebframe.cpp:
468
469 2010-12-02  Martin Robinson  <mrobinson@igalia.com>
470
471         Reviewed by Xan Lopez.
472
473         [GTK] DumpRenderTree has two copies of JavaScriptCore
474         https://bugs.webkit.org/show_bug.cgi?id=49877
475
476         Keep WebKit/WebCore symbols hidden.
477
478         * GNUmakefile.am: Move the symbol visibility rules here, because
479         now JSC symbols are visible, but WebCore/WebKit symbols are hidden.
480
481 2010-12-02  Daniel Bates  <dbates@rim.com>
482
483         Ignore WebKit/gtk/docs/GNUmakefile.in and WebKit/gtk/po/*.pot so that
484         these files do not appear in an svn stat. This change makes SVN
485         consistent with the current Git ignore behavior for these files.
486
487         * docs: Added property svn:ignore.
488         * po: Added property svn:ignore.
489
490 2010-12-01  Jia Pu  <jpu@apple.com>
491
492         Reviewed by Darin Adler.
493
494         Support multiple correction candidates panel for misspelled word on Mac OS X.
495         https://bugs.webkit.org/show_bug.cgi?id=50137
496         <rdar://problem/8568059>
497
498         Adopted new function signature defined in base class.
499
500         * WebCoreSupport/EditorClientGtk.cpp:
501         (WebKit::EditorClient::getGuessesForWord):
502         * WebCoreSupport/EditorClientGtk.h:
503
504 2010-11-30  Nicolas Dufresne  <nicolas.dufresne@collabora.co.uk>
505
506         Reviewed by Martin Robinson.
507
508         [GTK] Fix wrong type in assert return value
509         https://bugs.webkit.org/show_bug.cgi?id=50208
510
511         Use appropriate return type in different g_return_val_if_fail(). This
512         removes some compiler warning.
513
514         * webkit/webkitwebdatasource.cpp:
515         (webkit_web_data_source_is_loading):
516         * webkit/webkitwebframe.cpp:
517         (webkit_web_frame_page_number_for_element_by_id):
518         (webkit_web_frame_number_of_pages):
519
520 2010-11-30  Mario Sanchez Prada  <msanchez@igalia.com>
521
522         Reviewed by Chris Fleizach.
523
524         [GTK] Implement ROLE_COMBO_BOX
525         https://bugs.webkit.org/show_bug.cgi?id=25678
526
527         New test to check the implementation of the combo boxes.
528
529         * tests/testatk.c:
530         (testWebkitAtkComboBox): New test, checking that the roles and the
531         implemented interfaces for a combo box and its descendants work.
532         (main): Added the new unit test.
533
534 2010-11-30  Carlos Garcia Campos  <cgarcia@igalia.com>
535
536         Reviewed by Martin Robinson.
537
538         [GTK] Test fast/events/mouseover-mouseout2.html crashes
539         https://bugs.webkit.org/show_bug.cgi?id=50214
540
541         * WebCoreSupport/ChromeClientGtk.cpp:
542         (WebKit::ChromeClient::mouseDidMoveOverElement):
543         * webkit/webkitwebview.cpp:
544         (webkit_web_view_query_tooltip):
545
546 2010-11-30  Sheriff Bot  <webkit.review.bot@gmail.com>
547
548         Unreviewed, rolling out r72764.
549         http://trac.webkit.org/changeset/72764
550         https://bugs.webkit.org/show_bug.cgi?id=50215
551
552         This change is causing assertion failures on the debug bots.
553         (Requested by mrobinson on #webkit).
554
555         * WebCoreSupport/FrameLoaderClientGtk.cpp:
556         (WebKit::notifyStatus):
557         * tests/testatk.c:
558         (main):
559
560 2010-11-29  Xan Lopez  <xlopez@igalia.com>
561
562         Reviewed by Martin Robinson.
563
564         * NEWS: update for 1.3.7 release.
565
566 2010-11-29  Carlos Garcia Campos  <cgarcia@igalia.com>
567
568         Reviewed by Xan Lopez.
569
570         [Gtk] Tooltips should be fully keyboard accessible
571         https://bugs.webkit.org/show_bug.cgi?id=25525
572
573         Position the tooltip under the current focused element when the
574         tooltip has been triggered using the keyboard.
575
576         * webkit/webkitwebview.cpp:
577         (webkit_web_view_query_tooltip):
578         (webkit_web_view_show_help):
579         (webkit_web_view_class_init):
580
581 2010-11-29  Carlos Garcia Campos  <cgarcia@igalia.com>
582
583         Reviewed by Xan Lopez.
584
585         [GTK] Tooltips not shown when scrolling
586         https://bugs.webkit.org/show_bug.cgi?id=50143
587
588         Convert tooltip area to window coordinates so that tooltips are
589         shown even after scrolling.
590
591         * WebCoreSupport/ChromeClientGtk.cpp:
592         (WebKit::ChromeClient::mouseDidMoveOverElement):
593
594 2010-11-26  Carlos Garcia Campos  <cgarcia@igalia.com>
595
596         Reviewed by Xan Lopez.
597
598         [GTK] Inspector: add a way to inspect a WebKitDOMNode
599         https://bugs.webkit.org/show_bug.cgi?id=49903
600
601         * webkit/webkitwebinspector.cpp:
602         (webkit_web_inspector_inspect_node):
603         * webkit/webkitwebinspector.h:
604
605 2010-11-26  Mario Sanchez Prada  <msanchez@igalia.com>
606
607         Reviewed by Xan Lopez.
608
609         [GTK] events missing when a document is (re)loaded
610         https://bugs.webkit.org/show_bug.cgi?id=25831
611
612         Emit the right signals when reloading a document.
613
614         * WebCoreSupport/FrameLoaderClientGtk.cpp:
615         (WebKit::notifyAccessibilityStatus): New function, to make sure
616         the signals involved in reloading a document are properly emitted.
617         (WebKit::notifyStatus): Also notify accessibility if enabled.
618
619         New unit test to check the signals are being actually emitted.
620
621         * tests/testatk.c:
622         (stateChangedCb): Callback for 'state-change' signal emitted by
623         the accessible object associated to the webView.
624         (documentReloadCb): Callback for 'reload' signal.
625         (documentLoadCompleteCb): Callback for 'load-complete' signal.
626         (webviewLoadStatusChangedCb): Callback for 'notify::load-status'
627         signal, emitted by the WebKitWebView object (not the AtkObject).
628         (testWebkitAtkDocumentReloadEvents): New unit test.
629         (main): Added the new unit test.
630
631 2010-11-24  Xan Lopez  <xlopez@igalia.com>
632
633         Reviewed by Martin Robinson.
634
635         [GTK] Make kit() methods return the proper type instead of gpointer
636         https://bugs.webkit.org/show_bug.cgi?id=50034
637
638         * WebCoreSupport/EditorClientGtk.cpp:
639         (WebKit::EditorClient::shouldDeleteRange): get rid of unneeded cast.
640         (WebKit::EditorClient::shouldShowDeleteInterface): ditto.
641         (WebKit::EditorClient::shouldBeginEditing): ditto.
642         (WebKit::EditorClient::shouldEndEditing): ditto.
643         (WebKit::EditorClient::shouldInsertText): ditto.
644         (WebKit::EditorClient::shouldChangeSelectedRange): ditto.
645         (WebKit::EditorClient::shouldApplyStyle): ditto.
646         (WebKit::EditorClient::shouldInsertNode): ditto.
647         * webkit/webkitprivate.cpp:
648         (WebKit::kit): ditto.
649         * webkit/webkitwebview.cpp:
650         (webkit_web_view_get_dom_document): ditto.
651
652 2010-11-24  Xan Lopez  <xlopez@igalia.com>
653
654         Reviewed by Martin Robinson.
655
656         Fix crashes in GTK+ bots. Use kit() instead of wrapFoo, otherwise
657         the DOM objects won't be in the cache and we'll ASSERT when
658         unreffing them.
659
660         * WebCoreSupport/EditorClientGtk.cpp:
661         (WebKit::EditorClient::shouldDeleteRange): use kit to get the
662         GObject wrapper for the DOM object.
663         (WebKit::EditorClient::shouldShowDeleteInterface): ditto.
664         (WebKit::EditorClient::shouldBeginEditing): ditto.
665         (WebKit::EditorClient::shouldEndEditing): ditto.
666         (WebKit::EditorClient::shouldInsertText): ditto.
667         (WebKit::EditorClient::shouldChangeSelectedRange): ditto.
668         (WebKit::EditorClient::shouldApplyStyle): ditto.
669         (WebKit::EditorClient::shouldInsertNode): ditto.
670
671 2010-11-24  Martin Robinson  <mrobinson@igalia.com>
672
673         Reviewed by Xan Lopez.
674
675         [GTK] The WebKitWebView should expose a set of signals encapsulating the behavior of the EditorClient
676         https://bugs.webkit.org/show_bug.cgi?id=49143
677
678         Add signals that encapsulate the functionality of the Mac EditingDelegate. These signals
679         are private for now, but DRT consumes them to produce proper output for editing tests.
680         After some time, these may be part of the public API.
681
682         * WebCoreSupport/EditorClientGtk.cpp:
683         (WebKit::EditorClient::shouldDeleteRange): Fire the appropriate editing signal.
684         (WebKit::EditorClient::shouldShowDeleteInterface): Ditto.
685         (WebKit::EditorClient::shouldBeginEditing): Ditto.
686         (WebKit::EditorClient::shouldEndEditing): Ditto.
687         (WebKit::kit): Added helper function.
688         (WebKit::EditorClient::shouldInsertText): Fire the appropriate editing signal.
689         (WebKit::EditorClient::shouldChangeSelectedRange): Ditto.
690         (WebKit::EditorClient::shouldApplyStyle): Ditto.
691         (WebKit::EditorClient::didBeginEditing): Ditto.
692         (WebKit::EditorClient::respondToChangedContents): Ditto.
693         (WebKit::EditorClient::respondToChangedSelection): Ditto.
694         (WebKit::EditorClient::didEndEditing): Ditto.
695         (WebKit::EditorClient::shouldInsertNode): Ditto.
696         * webkit/webkitwebview.cpp:
697         (webkit_web_view_real_should_allow_editing_action): Added this default signal
698         callback which explicitly allows cancellable editing actions.
699         (webkit_web_view_class_init): Add new editing signal declarations to the view.
700         * webkit/webkitwebview.h:
701         * webkitmarshal.list: Added some new marshaling definitions.
702
703 2010-11-24  Sheriff Bot  <webkit.review.bot@gmail.com>
704
705         Unreviewed, rolling out r72499.
706         http://trac.webkit.org/changeset/72499
707         https://bugs.webkit.org/show_bug.cgi?id=50022
708
709         This change is causing crashes on the bots. (Requested by
710         mrobinson on #webkit).
711
712         * tests/testatk.c:
713         (main):
714
715 2010-11-23  Xan Lopez  <xlopez@igalia.com>
716
717         Reviewed by Martin Robinson.
718
719         [GTK] Fix IFrame DOM bindings API
720         https://bugs.webkit.org/show_bug.cgi?id=49980
721
722         * tests/testdomdocument.c:
723         (test_dom_document_garbage_collection): change to use the fixed
724         IFrame APIs.
725
726 2010-11-23  Carlos Garcia Campos  <cgarcia@igalia.com>
727
728         Reviewed by Martin Robinson.
729
730         focus issue with links that have tooltips
731         https://bugs.webkit.org/show_bug.cgi?id=49498
732
733         Set the area of the widget where the tooltip should be shown when
734         a new tooltip is set. Since the widget is the same (the view), if
735         the tooltip area is not set and a new tooltip is triggered while
736         the previous one is still visible, the text of the tooltip is
737         updated but its position doesn't change.
738
739         * WebCoreSupport/ChromeClientGtk.cpp:
740         (WebKit::ChromeClient::mouseDidMoveOverElement):
741         * webkit/webkitprivate.h:
742         * webkit/webkitwebview.cpp:
743         (webkit_web_view_query_tooltip):
744
745 2010-11-21  Mario Sanchez Prada  <msanchez@igalia.com>
746
747         Reviewed by Chris Fleizach.
748
749         [GTK] Implement ROLE_COMBO_BOX
750         https://bugs.webkit.org/show_bug.cgi?id=25678
751
752         New test to check the implementation of the combo boxes.
753
754         * tests/testatk.c:
755         (testWebkitAtkComboBox): New test, checking that the roles and the
756         implemented interfaces for a combo box and its descendants work.
757         (main): Added the new unit test.
758
759 2010-11-20  Xan Lopez  <xlopez@igalia.com>
760
761         Reviewed by Martin Robinson.
762
763         Remove leftover code in unit test.
764
765         * GNUmakefile.am: remove unneeded utils file in dependency list.
766         * tests/testdomdocument.c:
767         (main): do not change to the directory of a non existing resource
768         list.
769
770 2010-11-20  Xan Lopez  <xlopez@igalia.com>
771
772         Reviewed by Martin Robinson.
773
774         [GTK] Memory managament for DOM GObject wrappers
775         https://bugs.webkit.org/show_bug.cgi?id=40302
776
777         Initial attempt to have a kind of poor man's garbage collection
778         for GObject DOM objects. We tie the object's lifecycle to their
779         parent Frame and/or its document; that is, all objects will be
780         collected when their parent frame is destroyed or when that frame
781         transitions to a new document. Manual management (calling
782         g_object_unref on them) is also allowed, and is required for
783         objects that are not in the DOM tree (eg NodeLists).
784
785         * GNUmakefile.am: add testutils file to testdomdocument.c
786         * WebCoreSupport/FrameLoaderClientGtk.cpp:
787         (WebKit::FrameLoaderClient::setMainFrameDocumentReady): collect
788         DOM objects on document transition.
789         * tests/testdomdocument.c:
790         (dom_document_fixture_teardown): only destroy the view if it's not
791         already destroyed, since the new test detroys it on its own.
792         (test_dom_document_get_elements_by_tag_name): manually unref node list.
793         (test_dom_document_get_elements_by_class_name): ditto.
794         (test_dom_document_get_links): ditto.
795         (test_dom_document_garbage_collection): test all possible
796         scenarios for DOM object "garbage collection".
797         (main): add new test.
798         * tests/testdomnode.c:
799         (test_dom_node_hierarchy_navigation): manually unref node list.
800         (test_dom_node_insertion): ditto.
801         * webkit/webkithittestresult.cpp:
802         (webkit_hit_test_result_dispose): unref the inner node on dispose
803         now that the memory management model is clarified.
804         (webkit_hit_test_result_class_init): hook dispose.
805         * webkit/webkitwebframe.cpp:
806         (webkit_web_frame_core_frame_gone): clear DOM objects attached to this frame.
807         (webkit_web_frame_finalize): ditto.
808
809 2010-11-18  Xan Lopez  <xlopez@igalia.com>
810
811         Reviewed by Martin Robinson.
812
813         Fix introspection build, we were missing a -I flag.
814
815         * GNUmakefile.am:
816
817 2010-11-18  Mario Sanchez Prada  <msanchez@igalia.com>
818
819         Reviewed by Martin Robinson.
820
821         [Gtk]  atk_text_get_selection returns the wrong offsets after a link
822         https://bugs.webkit.org/show_bug.cgi?id=49514
823
824         Updated test case to also chech this specific subcase.
825
826         We need to explicitly check the case of having a selection in a
827         paragraph after an embedded object (i.e. a link) to make sure the
828         right calculations are being done when the paragraph is composed
829         of more than just one text object, and the current selection is
830         made only in one of them.
831
832         * tests/testatk.c:
833         (testWebkitAtkTextSelections): Make sure that a selection after a
834         link in a paragraph is working ok when asking for the text
835         selection from the point of view of the paragraph.
836
837 2010-11-17  Martin Robinson  <mrobinson@igalia.com>
838
839         Reviewed by Gustavo Noronha Silva.
840
841         [GTK] Changing any GNUmakefile.am in the source tree forces a rebuld of the DOM bindings and WebKit APi
842         https://bugs.webkit.org/show_bug.cgi?id=49400
843
844         Add GNUmakefile.am and include all WebKit API build rules. Make webkitenum.*
845         depend on this new file, instead of the generated GNUmakefile.am.
846
847         * GNUmakefile.am: Copied from GNUmakefile.am.
848
849 2010-11-17  Gustavo Noronha Silva  <gustavo.noronha@collabora.co.uk>
850
851         Reviewed by Martin Robinson.
852
853         Fix a typo that leads to emitting the wrong signal if it is done
854         by using the signals array.
855
856         * webkit/webkitwebview.cpp:
857         (webkit_web_view_class_init):
858
859 2010-11-16  Carlos Garcia Campos  <cgarcia@igalia.com>
860
861         Reviewed by Martin Robinson.
862
863         [GTK] Some key-press events can't be handled by WebView
864         https://bugs.webkit.org/show_bug.cgi?id=48986
865
866         The problem is that "popup-menu" and "show-help" signals are
867         handled by GtkTextView, as they are added to the key bindings set
868         of every widget in gtk_widget_class_init(). For all other bindings
869         handled by GtkTextView we are stopping the signal emission in
870         their callbacks, so we only need to do the same for "popup-menu"
871         and "show-help" signals.
872
873         * WebCoreSupport/EditorClientGtk.cpp:
874         (WebKit::toggleOverwriteCallback):
875         (WebKit::popupMenuCallback):
876         (WebKit::showHelpCallback):
877         (WebKit::EditorClient::EditorClient):
878
879 2010-11-16  Carlos Garcia Campos  <cgarcia@igalia.com>
880
881         Reviewed by Martin Robinson.
882
883         [GTK] Show default context menu for the currently focused element when activated with keyboard
884         https://bugs.webkit.org/show_bug.cgi?id=49376
885
886         * webkit/webkitwebview.cpp:
887         (webkit_web_view_popup_menu_handler):
888
889 2010-11-11  Joone Hur  <joone@kldp.org>
890
891         Reviewed by Antonio Gomes.
892
893         [GTK][DRT] Implement LayoutTestController::nodesFromRect
894         https://bugs.webkit.org/show_bug.cgi?id=46598
895
896         Support nodesFromRect in DRT
897
898         * WebCoreSupport/DumpRenderTreeSupportGtk.cpp:
899         (DumpRenderTreeSupportGtk::nodesFromRect): Added.
900         * WebCoreSupport/DumpRenderTreeSupportGtk.h:
901
902 2010-11-11  Chang Shu  <chang.shu@nokia.com>
903
904         Reviewed by Antonio Gomes.
905
906         [GTK] Replace "MoveForward"/"MoveBackward" with "MoveRight"/"MoveLeft"
907         to make spatial navigation work on input/textarea.
908         https://bugs.webkit.org/show_bug.cgi?id=49056
909
910         * WebCoreSupport/EditorClientGtk.cpp:
911         (WebKit::moveCursorCallback):
912
913 2010-11-08  Nicolas Dufresne  <nicolas.dufresne@collabora.co.uk>
914
915         Reviewed by Martin Robinson.
916
917         [GTK] Link with target name set does not work
918         https://bugs.webkit.org/show_bug.cgi?id=48865
919
920         When a new page is created with a name (target=myFrame), the new
921         mainFrame could not be found because they where not stored in the
922         same PageGroup. As PageGroup are not exposed externally so the
923         simpliest solution is to use a global page group name. This also fixes
924         issue with visited link coloration across pages. After this change the
925         private function webkit_web_view_set_group_name() was no longer used
926         so it was removed completly.
927
928         * WebCoreSupport/ChromeClientGtk.cpp:
929         (WebKit::ChromeClient::closeWindowSoon):
930         * WebCoreSupport/InspectorClientGtk.cpp:
931         (WebKit::InspectorClient::openInspectorFrontend):
932         * webkit/webkitprivate.h:
933         * webkit/webkitwebview.cpp:
934         (webkit_web_view_init):
935
936 2010-11-08  Alexey Proskuryakov  <ap@apple.com>
937
938         Reviewed by Darin Adler.
939
940         https://bugs.webkit.org/show_bug.cgi?id=48685
941         Notify UI process about focused frame
942
943         Added an empty implementation of the new ChromeClient method.
944
945         * WebCoreSupport/ChromeClientGtk.cpp:
946         (WebKit::ChromeClient::focusedFrameChanged):
947         * WebCoreSupport/ChromeClientGtk.h:
948
949 2010-11-08  Martin Robinson  <mrobinson@igalia.com>
950
951         Reviewed by Andreas Kling.
952
953         [GTK] Error page templates are not filled properly
954         https://bugs.webkit.org/show_bug.cgi?id=49148
955
956         Replace a call to makeString with String::format. This was mistakenly
957         changed to makeString in an earlier commit.
958
959         * WebCoreSupport/FrameLoaderClientGtk.cpp:
960         (WebKit::FrameLoaderClient::dispatchDidFailLoad):
961
962 2010-11-08  Xan Lopez  <xlopez@igalia.com>
963
964         Reviewed by Martin Robinson.
965
966         * NEWS: update for release.
967
968 2010-11-08  Csaba Osztrogonac  <ossy@webkit.org>
969
970         Unreviewed, rolling out r71466.
971         http://trac.webkit.org/changeset/71466
972         https://bugs.webkit.org/show_bug.cgi?id=48865
973
974         It broke layout tests on GTK bots.
975
976         * WebCoreSupport/ChromeClientGtk.cpp:
977         (WebKit::ChromeClient::closeWindowSoon):
978         * webkit/webkitprivate.h:
979         * webkit/webkitwebview.cpp:
980         (webkit_web_view_init):
981         (webkit_web_view_set_group_name):
982
983 2010-11-07  Adam Barth  <abarth@webkit.org>
984
985         Reviewed by Eric Seidel.
986
987         Rename Cache to MemoryCache
988         https://bugs.webkit.org/show_bug.cgi?id=49159
989
990         * webkit/webkitwebview.cpp:
991
992 2010-11-05  Nicolas Dufresne  <nicolas.dufresne@collabora.co.uk>
993
994         Reviewed by Martin Robinson.
995
996         [GTK] Link with target name set does not work correctly
997         https://bugs.webkit.org/show_bug.cgi?id=48865
998
999         When a new page is created with a name (target=myFrame), the new
1000         mainFrame could not be found because they where not stored in the
1001         same PageGroup. As PageGroup are not exposed externally so the
1002         simpliest solution is to use a global page group name. This also fixes
1003         issue with visited link coloration across pages. After this change the
1004         private function webkit_web_view_set_group_name() was no longer used
1005         so it was removed completly.
1006
1007         * WebCoreSupport/ChromeClientGtk.cpp:
1008         (WebKit::ChromeClient::closeWindowSoon):
1009         * webkit/webkitprivate.h:
1010         * webkit/webkitwebview.cpp:
1011         (webkit_web_view_init):
1012
1013 2010-11-05  Chris Marrin  <cmarrin@apple.com>
1014
1015         Reviewed by Simon Fraser.
1016
1017         Move resumeAnimations/suspendAnimations from Frame to AnimationController.
1018         https://bugs.webkit.org/show_bug.cgi?id=49073
1019
1020         * webkit/webkitwebframe.cpp:
1021         (webkit_web_frame_suspend_animations):
1022         (webkit_web_frame_resume_animations):
1023
1024 2010-11-05  Chris Marrin  <cmarrin@apple.com>
1025
1026         Unreviewed.
1027
1028         Small fix to GTK build
1029
1030         * webkit/webkitwebframe.cpp:
1031         (webkit_web_frame_suspend_animations):
1032         (webkit_web_frame_resume_animations):
1033
1034 2010-11-05  Chris Marrin  <cmarrin@apple.com>
1035
1036         Reviewed by Simon Fraser.
1037
1038         Make suspendAnimations/resumeAnimations and setCSSAnimations traverse through subframes and remember state
1039         https://bugs.webkit.org/show_bug.cgi?id=46945
1040
1041         * webkit/webkitwebframe.cpp:
1042         (webkit_web_frame_suspend_animations):
1043         (webkit_web_frame_resume_animations):
1044
1045 2010-11-03  Daniel Bates  <dbates@rim.com>
1046
1047         For unnamed frames, window.name returns a generated name
1048         https://bugs.webkit.org/show_bug.cgi?id=6751
1049
1050         Part 1 of 2.
1051
1052         Substitute FrameTree::uniqueName() for FrameTree::name() in the GTK port.
1053
1054         * webkit/webkitwebframe.cpp:
1055         (webkit_web_frame_get_name):
1056
1057 2010-11-02  Martin Robinson  <mrobinson@igalia.com>
1058
1059         Reviewed by Adam Barth.
1060
1061         [Cairo] Remove PlatformRefPtrCairo
1062         https://bugs.webkit.org/show_bug.cgi?id=48192
1063
1064         Replace instances of PlatformRefPtr smart pointers that wrap Cairo
1065         types with RefPtrs.
1066
1067         * WebCoreSupport/DragClientGtk.cpp: Use RefPtr instead of PlatformRefPtr.
1068         (WebKit::dragIconWindowDrawEventCallback):
1069         * WebCoreSupport/DragClientGtk.h: Ditto.
1070
1071 2010-11-02  Daniel Bates  <dbates@rim.com>
1072
1073         Reviewed by Martin Robinson.
1074
1075         Set frame name before appending it to the frame tree in the Apple Windows,
1076         GTK, and EFL ports
1077         https://bugs.webkit.org/show_bug.cgi?id=48806
1078
1079         Make the frame creation process in the GTK-port consistent with the
1080         Mac, Qt, and Haiku ports. In particular, set the name of the new
1081         frame before it's appended to the frame tree.
1082
1083         At this time we cannot test this change since it is being masked by
1084         HTMLFrameElementBase::setName() <http://trac.webkit.org/browser/trunk/WebCore/html/HTMLFrameElementBase.cpp?rev=70976#L160>.
1085         We'll be able to test this once we fix bug #6751.
1086
1087         * WebCoreSupport/FrameLoaderClientGtk.cpp:
1088         (WebKit::FrameLoaderClient::createFrame):
1089
1090 2010-11-01  Jennifer Braithwaite  <jennb@google.com>
1091
1092         Reviewed by Martin Robinson.
1093
1094         GTK: Update resource tracking when moving a frame between documents
1095         https://bugs.webkit.org/show_bug.cgi?id=48362
1096
1097         * WebCoreSupport/FrameLoaderClientGtk.cpp:
1098         (WebKit::FrameLoaderClient::transferLoadingResourceFromPage):
1099         Assign resource id to current web view and remove from former web view.
1100         * webkit/webkitprivate.h:
1101         * webkit/webkitwebview.cpp:
1102         (webkit_web_view_remove_resource):
1103         Added.
1104
1105 2010-11-01  Mario Sanchez Prada  <msanchez@igalia.com>
1106
1107         Reviewed by Martin Robinson.
1108
1109         [GTK] Cleanup needed in testatk.c file
1110         https://bugs.webkit.org/show_bug.cgi?id=48759
1111
1112         Cleanup done in testatk.c, mainly addressing the following issues:
1113
1114         - Removed any trace of extra main loops in tests, so now we just
1115         spin the original main loop in all of them to get the a11y
1116         objects.
1117
1118         - As we no longer use the extra main loops, removed the bail_out
1119         function, used along with g_idle_add() to stop those extra loops.
1120
1121         - As we now spin the loop in every test, created a new function
1122         waitForAccessibilityObjects() to write the loop just once.
1123
1124         - Fixed name in incorrectly named test testWekitAtkTextSelections.
1125
1126         - Alphabetically sorted the list of includes.
1127
1128         - Renamed all identifiers to camel Case notation.
1129
1130         - Declare variables when needed only (instead of declaring all of
1131         them at the beginning of the function).
1132
1133         - Don't declare two variables of the same type in the same line.
1134
1135         - No more short names like 'obj', 'textObj' or 'alloc'. Use names
1136         like 'object', 'textObject' and 'allocation' instead.
1137
1138         - Fixed indentation.
1139
1140         - Make sure all comments finish with '.'
1141
1142         - Replace NULL's with 0's.
1143
1144         * tests/testatk.c:
1145         (waitForAccessibleObjects): New, manually spins the main context
1146         to make sure accessible objects are created before continuing.
1147         (testGetTextFunction): Fixed this function to be compliant with
1148         the WebKit's coding style.
1149         (runGetTextTests): Ditto.
1150         (testWebkitAtkGetTextAtOffsetForms): Ditto.
1151         (testWebkitAtkGetTextAtOffset): Ditto.
1152         (testWebkitAtkGetTextAtOffsetNewlines): Ditto.
1153         (testWebkitAtkGetTextAtOffsetTextarea): Ditto.
1154         (testWebkitAtkGetTextAtOffsetTextInput): Ditto.
1155         (testWebkitAtkGetTextInParagraphAndBodySimple): Ditto.
1156         (testWebkitAtkGetTextInParagraphAndBodyModerate): Ditto.
1157         (testWebkitAtkGetTextInTable): Ditto.
1158         (testWebkitAtkGetHeadersInTable): Ditto.
1159         (compAtkAttribute): Ditto.
1160         (atkAttributeSetAttributeNameHasValue): Ditto.
1161         (testWebkitAtkTextAttributes): Ditto.
1162         (testWebkitAtkTextSelections): Ditto.
1163         (testWebkitAtkGetExtents): Ditto.
1164         (testWebkitAtkLayoutAndDataTables): Ditto.
1165         (testWebkitAtkLinksWithInlineImages): Ditto.
1166         (testWebkitAtkHypertextAndHyperlinks): Ditto.
1167         (testWebkitAtkListsOfItems): Ditto.
1168         (testWebkitAtkTextChangedNotifications): Ditto.
1169         (main): Ditto.
1170
1171 2010-11-01  Brady Eidson  <beidson@apple.com>
1172
1173         Reviewed by Anders Carlsson.
1174
1175         <rdar://problem/7660547> and https://bugs.webkit.org/show_bug.cgi?id=48699
1176         Context menu support for WebKit 2.
1177
1178         * WebCoreSupport/ChromeClientGtk.h:
1179         (WebKit::ChromeClient::showContextMenu):
1180
1181 2010-11-01  Martin Robinson  <mrobinson@igalia.com>
1182
1183         Reviewed by Xan Lopez.
1184
1185         [GTK] Pasting markup into Thunderbird compose window produces no text
1186         https://bugs.webkit.org/show_bug.cgi?id=43737
1187
1188         Added a test verifying that the meta tag prefix on markup data exists.
1189
1190         * tests/testcopyandpaste.c:
1191         (load_status_cb): Add a check for the meta tag prefix.
1192
1193 2010-11-01  Mario Sanchez Prada  <msanchez@igalia.com>
1194
1195         Reviewed by Martin Robinson.
1196
1197         [Gtk] AtkHyperlink needs to be implemented
1198         https://bugs.webkit.org/show_bug.cgi?id=33785
1199
1200         New test to ensure the AtkHypertext/AtkHyperlink stuff works.
1201
1202         * tests/testatk.c:
1203         (testWebkitAtkHypertextAndHyperlinks): New test, checking both the
1204         implementation of the AtkHypertext interface, the subclass of the
1205         AtkHyperlink abstract class, and the implementation of the
1206         AtkAction interface in that subclass.
1207         (main): Added the new unit test.
1208
1209 2010-10-30  Xan Lopez  <xlopez@igalia.com>
1210
1211         Reviewed by Martin Robinson.
1212
1213         [GTK] Use new width for height APIs in GTK+ 3.x
1214         https://bugs.webkit.org/show_bug.cgi?id=48709
1215
1216         Use the new width for height APIs in GTK+ 3.x, since size-request
1217         is deprecated.
1218
1219         For now we just return the same value for preferred and minimum
1220         width/height, which should match the 2.x behavior. Probably we
1221         could do something smarter for the minimum values.
1222
1223         * webkit/webkitwebview.cpp:
1224         (webkit_web_view_get_preferred_width): return our preferred width.
1225         (webkit_web_view_get_preferred_height): return our preferred height.
1226         (webkit_web_view_class_init): hook the new default handlers.
1227
1228 2010-10-29  Daniel Bates  <dbates@rim.com>
1229
1230         No review, rolling out 70971.
1231         http://trac.webkit.org/changeset/70971
1232         https://bugs.webkit.org/show_bug.cgi?id=6751
1233
1234         Rolling out changeset 70971 <http://trac.webkit.org/changeset/70971> since
1235         it caused layout test failures on all bots. In particular, the
1236         child count in a generated frame name differs after this patch. We need
1237         to look into this further.
1238
1239         * webkit/webkitwebframe.cpp:
1240         (webkit_web_frame_get_name):
1241
1242 2010-10-28  Antonio Gomes  <agomes@rim.com>
1243
1244         Reviewed by Ojan Vafai.
1245
1246         Needs a "LinuxEditingBehavior", perhaps with a better name
1247         https://bugs.webkit.org/show_bug.cgi?id=36627
1248
1249         Added the corresponding GTK+ setting to WebCore's EditingUnixBehavior: WEBKIT_EDITING_BEHAVIOR_UNIX.
1250
1251         * webkit/webkitwebsettings.cpp:
1252         (webkit_web_settings_class_init):
1253         * webkit/webkitwebsettings.h:
1254
1255 2010-10-29  Daniel Bates  <dbates@rim.com>
1256
1257         Reviewed by Adam Barth.
1258
1259         For unnamed frames, window.name returns a generated name
1260         https://bugs.webkit.org/show_bug.cgi?id=6751
1261
1262         Modified GTK-port to use FrameTree::uniqueName().
1263
1264         * webkit/webkitwebframe.cpp:
1265         (webkit_web_frame_get_name):
1266
1267 2010-10-29  Martin Robinson  <mrobinson@igalia.com>
1268
1269         Build fix for GTK+ after BackForwardListImpl introduction.
1270
1271         * webkit/webkitwebbackforwardlist.cpp:
1272         (webkit_web_back_forward_list_new_with_web_view):
1273
1274 2010-10-29  Darin Adler  <darin@apple.com>
1275
1276         Reviewed by Sam Weinig.
1277
1278         Change BackForwardList clients to use BackForwardListImpl to prepare for further refactoring
1279         https://bugs.webkit.org/show_bug.cgi?id=48574
1280
1281         * webkit/webkitprivate.h:
1282         * webkit/webkitwebbackforwardlist.cpp:
1283         (webkit_web_back_forward_list_dispose):
1284         (webkit_web_back_forward_list_new_with_web_view):
1285         (webkit_web_back_forward_list_go_forward):
1286         (webkit_web_back_forward_list_go_back):
1287         (webkit_web_back_forward_list_contains_item):
1288         (webkit_web_back_forward_list_go_to_item):
1289         (webkit_web_back_forward_list_get_forward_list_with_limit):
1290         (webkit_web_back_forward_list_get_back_list_with_limit):
1291         (webkit_web_back_forward_list_get_back_item):
1292         (webkit_web_back_forward_list_get_current_item):
1293         (webkit_web_back_forward_list_get_forward_item):
1294         (webkit_web_back_forward_list_get_nth_item):
1295         (webkit_web_back_forward_list_get_back_length):
1296         (webkit_web_back_forward_list_get_forward_length):
1297         (webkit_web_back_forward_list_get_limit):
1298         (webkit_web_back_forward_list_set_limit):
1299         (webkit_web_back_forward_list_add_item):
1300         (webkit_web_back_forward_list_clear):
1301         (WebKit::core):
1302         * webkit/webkitwebview.cpp:
1303         (webkit_web_view_set_maintains_back_forward_list):
1304         (webkit_web_view_get_back_forward_list):
1305         Use BackForwardListImpl.
1306
1307 2010-10-29  Alexey Proskuryakov  <ap@apple.com>
1308
1309         Reviewed by Darin Adler.
1310
1311         https://bugs.webkit.org/show_bug.cgi?id=48576
1312         Let WebKit2 client know when a frame is a frameset
1313
1314         Added a blank implementation of the new FrameLoaderClient method.
1315
1316         * WebCoreSupport/FrameLoaderClientGtk.h:
1317         * WebCoreSupport/FrameLoaderClientGtk.cpp:
1318         (WebKit::FrameLoaderClient::dispatchDidBecomeFrameset):
1319
1320 2010-10-29  Alejandro G. Castro  <alex@igalia.com>
1321
1322         Reviewed by Martin Robinson.
1323
1324         [GTK] Add the GtkScrollablePolicy property to the webview
1325         https://bugs.webkit.org/show_bug.cgi?id=48357
1326
1327         Added the GtkScrollablePolicy properties to the webview widget, it
1328         is now required for GtkScrollable widgets in GTK+ 3.x.
1329
1330         * webkit/webkitprivate.h:
1331         * webkit/webkitwebview.cpp:
1332         (setHorizontalScrollPolicy): Added.
1333         (setVerticalScrollPolicy): Added.
1334         (getHorizontalScrollPolicy): Added.
1335         (getVerticalScrollPolicy): Added.
1336         (webkit_web_view_get_property):
1337         (webkit_web_view_set_property):
1338         (webkit_web_view_class_init):
1339
1340 2010-10-28  Sergio Villar Senin  <svillar@igalia.com>
1341
1342         Reviewed by Martin Robinson.
1343
1344         [GTK] WebKitWebFrame's load-status is not properly notified to the tests
1345         https://bugs.webkit.org/show_bug.cgi?id=48048
1346
1347         "frame-created" signal must be issued before loading any data,
1348         that way clients can correctly capture all the loading related
1349         signals for that frame.
1350
1351         * WebCoreSupport/FrameLoaderClientGtk.cpp:
1352         (WebKit::FrameLoaderClient::createFrame):
1353
1354 2010-10-27  Mario Sanchez Prada  <msanchez@igalia.com>
1355
1356         Reviewed by Martin Robinson.
1357
1358         getTextAtOffset returns incorrect results if a link includes text and an image
1359         https://bugs.webkit.org/show_bug.cgi?id=47365
1360
1361         New test to make sure this bug is actually fixed.
1362
1363         * tests/testatk.c:
1364         (testWebkitAtkLinksWithInlineImages): New test.
1365         (main): Added new test.
1366
1367 2010-10-26  Antonio Gomes  <agomes@rim.com>
1368
1369         Reviewed by Martin Robinson.
1370
1371         [GTK] Implement DumpRenderTreeSupportGtk (similarly to DumpRenderTreeSupportQt idea)
1372         https://bugs.webkit.org/show_bug.cgi?id=48199
1373
1374         Implemented an initial version of the DumpRenderTreeSupportGtk class.
1375         The main idea is that it groups together the bodies of many LayoutTestController
1376         methods, so they can directly access WebCore features. This way we eliminate the
1377         need of adding APIs to WebKitGTK just to expand its test coverage through DRT.
1378
1379         The DumpRenderTreeSupportGtk class so far includes a getter and setter for the WebKitTabToLinksPreferenceKey
1380         property as a sample implementation. Also it makes it possible to unskip fast/events/tab-focus-anchor.html and
1381         to fix stderr messages from all 22 spatial-navigation tests in fast/events/spatial-navigation.
1382
1383         * WebCoreSupport/ChromeClientGtk.cpp:
1384         (WebKit::ChromeClient::tabsToLinks):
1385         * WebCoreSupport/DumpRenderTreeSupportGtk.cpp: Added.
1386         (DumpRenderTreeSupportGtk::DumpRenderTreeSupportGtk):
1387         (DumpRenderTreeSupportGtk::~DumpRenderTreeSupportGtk):
1388         (DumpRenderTreeSupportGtk::setDumpRenderTreeModeEnabled):
1389         (DumpRenderTreeSupportGtk::dumpRenderTreeModeEnabled): Getter to the WebKitTabToLinksPreferenceKey property.
1390         (DumpRenderTreeSupportGtk::setLinksIncludedInFocusChain): Setter to the WebKitTabToLinksPreferenceKey property.
1391         (DumpRenderTreeSupportGtk::linksIncludedInFocusChain):
1392         * WebCoreSupport/DumpRenderTreeSupportGtk.h: Added.
1393
1394 2010-10-26  Jenn Braithwaite  <jennb@chromium.org>
1395
1396         Reviewed by Dmitry Titov.
1397
1398         Resource tracking failure when trying to move a frame between documents
1399         https://bugs.webkit.org/show_bug.cgi?id=44713
1400
1401         * WebCoreSupport/FrameLoaderClientGtk.cpp:
1402         (WebKit::FrameLoaderClient::transferLoadingResourceFromPage):
1403         Empty method.
1404         * WebCoreSupport/FrameLoaderClientGtk.h:
1405
1406 2010-10-26  Mario Sanchez Prada  <msanchez@igalia.com>
1407
1408         Reviewed by Chris Fleizach.
1409
1410         [Gtk] Layout tables should indicate that they are not data tables via an object attribute
1411         https://bugs.webkit.org/show_bug.cgi?id=35422
1412
1413         New unit test to check the 'layout-guess' attribute in tables.
1414
1415         * tests/testatk.c:
1416         (atkAttributeSetAttributeNameHasValue): New, looks for a attribute
1417         with a specific value, by the name of attribute.
1418         (atkAttributeSetContainsAttributeName): New, just checks whether
1419         an attribute with a specified name is present in the set.
1420         (atkAttributeSetAttributeHasValue): Modified to relay on the new
1421         function atkAttributeSetAttributeNameHasValue().
1422         (testWebkitAtkLayoutAndDataTables): New test.
1423         (main): Added the new test.
1424
1425 2010-10-26  Xan Lopez  <xlopez@igalia.com>
1426
1427         Reviewed by Martin Robinson.
1428
1429         [GTK] Port to new GtkScrollable interface in GTK+ 3.x
1430         https://bugs.webkit.org/show_bug.cgi?id=48202
1431
1432         Use the new GtkScrollable interface when compiling against GTK+
1433         3.x.
1434
1435         * webkit/webkitwebview.cpp:
1436         (setHorizontalAdjustment):
1437         (setVerticalAdjustment):
1438         (getHorizontalAdjustment):
1439         (getVerticalAdjustment):
1440         (webkit_web_view_get_property):
1441         (webkit_web_view_set_property):
1442         (webkit_web_view_class_init):
1443
1444 2010-10-22  Sam Weinig  <sam@webkit.org>
1445
1446         Reviewed by Anders Carlsson.
1447
1448         WebKit2 needs to pass the current event modifier flags when requesting a new window
1449         https://bugs.webkit.org/show_bug.cgi?id=48140
1450
1451         * WebCoreSupport/ChromeClientGtk.cpp:
1452         (WebKit::ChromeClient::createWindow):
1453         * WebCoreSupport/ChromeClientGtk.h:
1454         * WebCoreSupport/FrameLoaderClientGtk.cpp:
1455         (WebKit::FrameLoaderClient::dispatchCreatePage):
1456         * WebCoreSupport/FrameLoaderClientGtk.h:
1457         Add NavigationAction parameter.
1458
1459 2010-10-20  Philippe Normand  <pnormand@igalia.com>
1460
1461         Reviewed by Xan Lopez.
1462
1463         [GTK] use gtk_widget_hide instead of gtk_widget_hide_all in the fullscreen video conroller
1464         https://bugs.webkit.org/show_bug.cgi?id=47972
1465
1466         * WebCoreSupport/FullscreenVideoController.cpp:
1467         (FullscreenVideoController::hideHud):
1468         (FullscreenVideoController::exitFullscreen):
1469
1470 2010-10-19  Joone Hur  <joone@kldp.org>
1471
1472         Reviewed by Martin Robinson.
1473
1474         [GTK] Needs fixing mistyped comments in the API documentation
1475         https://bugs.webkit.org/show_bug.cgi?id=47656
1476
1477         Fixed mistyped comments in the API documenentation.
1478
1479         * webkit/webkitwebsettings.cpp: Fixed a mistyped comment in the the section block.
1480         * webkit/webkitwebwindowfeatures.cpp: Fixed a mistyped comment in the section block.
1481
1482 2010-10-18  Xan Lopez  <xlopez@igalia.com>
1483
1484         Reviewed by Martin Robinson.
1485
1486         * NEWS: update for 1.3.5 release.
1487         * docs/webkitgtk-sections.txt: add new APIs.
1488
1489 2010-10-18  Pavel Feldman  <pfeldman@chromium.org>
1490
1491         Reviewed by Simon Fraser.
1492
1493         Web Inspector: [crash] when Inspector Open in CSSStyleSelector::loadPendingImages().
1494         https://bugs.webkit.org/show_bug.cgi?id=46224
1495
1496         * WebCoreSupport/InspectorClientGtk.cpp:
1497         (WebKit::InspectorClient::sendMessageToFrontend):
1498
1499 2010-10-14  Antonio Gomes  <agomes@rim.com>
1500
1501         Reviewed by Martin Robinson and Xan Lopez.
1502
1503         [Gtk]: DRT does not support frame flattening testing
1504         https://bugs.webkit.org/show_bug.cgi?id=38650
1505
1506         Implement enable-frame-flattening API for Gtk+.
1507
1508         * webkit/webkitwebsettings.cpp:
1509         (webkit_web_settings_class_init):
1510         (webkit_web_settings_set_property):
1511         (webkit_web_settings_get_property):
1512         (webkit_web_settings_copy):
1513         * webkit/webkitwebview.cpp:
1514         (webkit_web_view_update_settings):
1515         (webkit_web_view_settings_notify):
1516
1517 2010-10-15  Nikolas Zimmermann  <nzimmermann@rim.com>
1518
1519         Reviewed by Dirk Schulze.
1520
1521         Replace some String::format() usages by StringConcatenate in WebKit
1522         https://bugs.webkit.org/show_bug.cgi?id=47714
1523
1524         * WebCoreSupport/FrameLoaderClientGtk.cpp:
1525         (WebKit::FrameLoaderClient::dispatchDidFailLoad):
1526         * webkit/webkitwebsettings.cpp:
1527         (webkitOSVersion):
1528         (webkitUserAgent):
1529
1530 2010-10-14  Joone Hur  <joone@kldp.org>
1531
1532         Reviewed by Andreas Kling.
1533
1534         [GTK] google.co.kr was missed when checking Google domains
1535         https://bugs.webkit.org/show_bug.cgi?id=47652
1536
1537         WebKitGtk+ maintains a list of Google domains for working well with Google sites.
1538         Add google.co.kr also to the list to be checked with other Google domains.
1539
1540         * WebCoreSupport/FrameLoaderClientGtk.cpp:
1541         (WebKit::initializeDomainsList): Added google.co.kr.
1542
1543 2010-10-13  Sergio Villar Senin  <svillar@igalia.com>
1544
1545         Reviewed by Martin Robinson.
1546
1547         WebKitGtk+ to use the new API from the imported SoupURILoader code
1548
1549         [GTK] Add HTTP caching support
1550         https://bugs.webkit.org/show_bug.cgi?id=44261
1551
1552         ResourceHandleInternal m_msg is now called m_soupMessage.
1553
1554         * webkit/webkitdownload.cpp:
1555         (webkit_download_new_with_handle):
1556         (webkit_download_start):
1557
1558 2010-10-13  Xan Lopez  <xlopez@igalia.com>
1559
1560         Reviewed by Gustavo Noronha.
1561
1562         [GTK] Remove more warnings from the g-i scanner run
1563         https://bugs.webkit.org/show_bug.cgi?id=47255
1564
1565         Fixes almost all warnings coming from the g-i scanner.
1566
1567         * webkit/webkitsecurityorigin.cpp: Rename parameter names in
1568         gtk-doc blurb to the actual name of the parameter.
1569         * webkit/webkitsoupauthdialog.c: ditto.
1570         (webkit_soup_auth_dialog_class_init): Write missing doc for the
1571         'current-toplevel' signal.
1572         * webkit/webkitsoupauthdialog.h: Rename signal parameter to its
1573         actual name.
1574         * webkit/webkitwebdatabase.cpp: Rename parameter names in
1575         gtk-doc blurb to the actual name of the parameter.
1576         * webkit/webkitwebinspector.cpp: ditto.
1577         (webkit_web_inspector_class_init): ditto.
1578         * webkit/webkitwebview.cpp: ditto.
1579         (webkit_web_view_class_init): ditto.
1580         * webkit/webkitwebview.h: Rename the parameter names to be in sync
1581         with the ones we use in the function definition.
1582
1583 2010-10-12  Sheriff Bot  <webkit.review.bot@gmail.com>
1584
1585         Unreviewed, rolling out r69589.
1586         http://trac.webkit.org/changeset/69589
1587         https://bugs.webkit.org/show_bug.cgi?id=47547
1588
1589         Wrong patch landed from a bug with two patches (Requested by
1590         mrobinson on #webkit).
1591
1592         * webkit/webkitdownload.cpp:
1593         (webkit_download_new_with_handle):
1594         (webkit_download_start):
1595
1596 2010-10-12  Sergio Villar Senin  <svillar@igalia.com>
1597
1598         Reviewed by Martin Robinson.
1599
1600         [GTK] Add HTTP caching support
1601         https://bugs.webkit.org/show_bug.cgi?id=44261
1602
1603         ResourceHandle's m_msg renamed to m_soupMsg and it's also now a PlatformRefPtr
1604
1605         * webkit/webkitdownload.cpp:
1606         (webkit_download_new_with_handle):
1607         (webkit_download_start):
1608
1609 2010-10-09  Carlos Garcia Campos  <cgarcia@igalia.com>
1610
1611         Reviewed by Xan Lopez.
1612
1613         [GTK] Removed unused callback in webkitwebview
1614         https://bugs.webkit.org/show_bug.cgi?id=47092
1615
1616         In webkit_web_view_forward_context_menu_event() a callback is
1617         connected for the destroy signal of the popup menu, however we
1618         reuse the same menu always, holding a ref to it, so it's not
1619         destroyed when withdrawn by the user.
1620
1621         * webkit/webkitwebview.cpp:
1622         (webkit_web_view_forward_context_menu_event):
1623
1624 2010-10-08  Martin Robinson  <mrobinson@igalia.com>
1625
1626         Reviewed by Xan Lopez.
1627
1628         [GTK] Remove duplicate code in WebKit/gtk/WebCoreSupport/DragClientGtk.cpp
1629         https://bugs.webkit.org/show_bug.cgi?id=47367
1630
1631         Remove as much duplicate code as possible from DragClientGtk. Most of this code
1632         is GTK2/GTK3 independent, so only the bits that aren't should be protected by
1633         #ifdefs.
1634
1635         * WebCoreSupport/DragClientGtk.cpp:
1636         (WebKit::dragIconWindowDrawEventCallback): Modified the name of this callback
1637         so that it is shared between build types.
1638         (WebKit::DragClient::DragClient): Updated the signal connectors to reflect the change above.
1639         (WebKit::DragClient::~DragClient): The build types share the callback name, so remove
1640         the #ifdef here.
1641         (WebKit::DragClient::drawDragIconWindow): Changed the name of this method and make it
1642         independent of the build type.
1643         * WebCoreSupport/DragClientGtk.h: Update method declaration and remove #ifdefs.
1644
1645 2010-10-07  Carlos Garcia Campos  <cgarcia@igalia.com>
1646
1647         Reviewed by Xan Lopez.
1648
1649         [GTK] Use draw instead of expose_event in WebView when building with gtk3
1650         https://bugs.webkit.org/show_bug.cgi?id=47338
1651
1652         * webkit/webkitwebview.cpp:
1653         (webkit_web_view_expose_event):
1654         (webkit_web_view_draw):
1655         (webkit_web_view_class_init):
1656
1657 2010-10-07  Carlos Garcia Campos  <cgarcia@igalia.com>
1658
1659         Reviewed by Xan Lopez.
1660
1661         [GTK] Don't use GtkObject
1662         https://bugs.webkit.org/show_bug.cgi?id=47090
1663
1664         GtkObject has been removed in gtk3.
1665
1666         * WebCoreSupport/EditorClientGtk.cpp:
1667         (WebKit::EditorClient::generateEditorCommands):
1668         * WebCoreSupport/FullscreenVideoController.cpp:
1669         (FullscreenVideoController::createHud):
1670
1671 2010-10-07  Carlos Garcia Campos  <cgarcia@igalia.com>
1672
1673         Reviewed by Xan Lopez.
1674
1675         [GTK] Fix the build for GTK+ 3
1676         https://bugs.webkit.org/show_bug.cgi?id=47249
1677
1678         Use GdkVisual instead of GdkColormap. GdkColormap has been removed
1679         in gtk3
1680
1681         * webkit/webkitwebview.cpp:
1682         (webkit_web_view_realize):
1683
1684 2010-10-07  Carlos Garcia Campos  <cgarcia@igalia.com>
1685
1686         Reviewed by Xan Lopez.
1687
1688         [GTK] Fix the build for GTK+ 3
1689         https://bugs.webkit.org/show_bug.cgi?id=47249
1690
1691         Do not use GdkDrawable deprecated API. Some methods of GdkDrawable
1692         are deprecated in gtk2 and have been removed in gtk3. Equivalent
1693         API has been added to GdkWindow.
1694
1695         * tests/testcopyandpaste.c:
1696         (runPasteTestCallback):
1697         * webkit/webkitwebview.cpp:
1698         (webkit_web_view_button_press_event):
1699
1700 2010-10-07  Carlos Garcia Campos  <cgarcia@igalia.com>
1701
1702         Reviewed by Xan Lopez.
1703
1704         [GTK] Use draw signal instead of expose_event in DragClientGtk when building with gtk3
1705         https://bugs.webkit.org/show_bug.cgi?id=47326
1706
1707         * WebCoreSupport/DragClientGtk.cpp:
1708         (WebKit::dragIconWindowDrawCallback):
1709         (WebKit::DragClient::DragClient):
1710         (WebKit::DragClient::~DragClient):
1711         (WebKit::DragClient::startDrag):
1712         (WebKit::DragClient::dragIconWindowDraw):
1713         * WebCoreSupport/DragClientGtk.h:
1714
1715 2010-10-07  Carlos Garcia Campos  <cgarcia@igalia.com>
1716
1717         Reviewed by Xan Lopez.
1718
1719         [GTK] Fix the build for GTK+ 3
1720         https://bugs.webkit.org/show_bug.cgi?id=47249
1721
1722         Don't use gtk_size_request_get_size(). It has been removed,
1723         gtk_widget_get_preferred_size() should be used instead
1724
1725         * webkit/webkitwebview.cpp:
1726         (PopupMenuPositionFunc):
1727
1728 2010-10-06  Sheriff Bot  <webkit.review.bot@gmail.com>
1729
1730         Unreviewed, rolling out r69201.
1731         http://trac.webkit.org/changeset/69201
1732         https://bugs.webkit.org/show_bug.cgi?id=47279
1733
1734         This change broke the WebKitGTK+ build for GTK+ 2.0.
1735         (Requested by mrobinson on #webkit).
1736
1737         * tests/testcopyandpaste.c:
1738         (runPasteTestCallback):
1739         * webkit/webkitwebview.cpp:
1740         (webkit_web_view_button_press_event):
1741
1742 2010-10-06  Carlos Garcia Campos  <cgarcia@igalia.com>
1743
1744         Reviewed by Xan Lopez.
1745
1746         Port to gtk+3 (2.91.0)
1747         https://bugs.webkit.org/show_bug.cgi?id=47249
1748
1749         Do not use GdkDrawable deprecated API
1750
1751         Some methods of GdkDrawable are deprecated in gtk2 and have been
1752         remmoved in gtk3. Equivalent API has been added to GdkWindow.
1753
1754         * tests/testcopyandpaste.c:
1755         (runPasteTestCallback):
1756         * webkit/webkitwebview.cpp:
1757         (webkit_web_view_button_press_event):
1758
1759 2010-09-28  Jenn Braithwaite  <jennb@chromium.org>
1760
1761         Reviewed by Dmitry Titov.
1762
1763         Added oldPage param to FrameLoaderClient::didTransferChildFrameToNewDocument.
1764         https://bugs.webkit.org/show_bug.cgi?id=46663
1765
1766         * WebCoreSupport/FrameLoaderClientGtk.cpp:
1767         (WebKit::FrameLoaderClient::didTransferChildFrameToNewDocument):
1768         * WebCoreSupport/FrameLoaderClientGtk.h:
1769
1770 2010-09-27  Mario Sanchez Prada  <msanchez@igalia.com>
1771
1772         Reviewed by Chris Fleizach.
1773
1774         [Gtk] Adjust atk_text_get_text_at_offset to account for bullets/numbers in list items
1775         https://bugs.webkit.org/show_bug.cgi?id=45381
1776
1777         Updated test to match the new exposure of list item markers.
1778
1779         Also, added some extra assertions in that test to make sure the
1780         accessible objects associated to the items implement AtkText.
1781
1782         * tests/testatk.c:
1783         (testWebkitAtkListsOfItems):
1784         (main):
1785
1786 2010-09-27  Philippe Normand  <pnormand@igalia.com>
1787
1788         Reviewed by Martin Robinson.
1789
1790         [GTK] Doesn't build with gtk+ 2.14
1791         https://bugs.webkit.org/show_bug.cgi?id=46565
1792
1793         Fixed the build for GTK+ 2.14. Don't use gdk_window_get_cursor()
1794         for that GTK+ version as I found no acceptable way to work-around
1795         its absence.
1796
1797         * WebCoreSupport/FullscreenVideoController.cpp:
1798         (FullscreenVideoController::showHud):
1799         (FullscreenVideoController::hideHud):
1800         (FullscreenVideoController::enterFullscreen):
1801
1802 2010-09-26  Jenn Braithwaite  <jennb@chromium.org>
1803
1804         Reviewed by Adam Barth.
1805
1806         GTK client needs updating when live iframe element is moved
1807         between pages.
1808         https://bugs.webkit.org/show_bug.cgi?id=46300
1809
1810         Test: fast/frames/iframe-reparenting-adopt-node.html
1811
1812         * WebCoreSupport/FrameLoaderClientGtk.cpp:
1813         (WebKit::FrameLoaderClient::didTransferChildFrameToNewDocument):
1814         Update the webView for the frame to match the parent frame's
1815         webView.
1816
1817 2010-09-24  Martin Robinson  <mrobinson@igalia.com>
1818
1819         Reviewed by Gustavo Noronha Silva.
1820
1821         [GTK] opening local files on win32
1822         https://bugs.webkit.org/show_bug.cgi?id=31066
1823
1824         Add tests that check the mime type returned by local files loaded via file URLs.
1825
1826         * tests/testmimehandling.c:
1827         (testRemoteMimeType): Added.
1828         (testLocalMimeType): Added.
1829         (main): Run tests by using g_test_add_data_func with either 
1830         testRemoteMimeType or testLocalMimeType.
1831
1832 2010-09-23  Martin Robinson  <mrobinson@igalia.com>
1833
1834         Reviewed by Nate Chapin.
1835
1836         [GTK] Some tests from r68174 fail on the GTK+ bots
1837         https://bugs.webkit.org/show_bug.cgi?id=46396
1838
1839         Fix since version number in this new WebKitWebSettings property.
1840
1841         * webkit/webkitwebsettings.cpp:
1842         (webkit_web_settings_class_init): Fix the since version number.
1843
1844 2010-09-23  Nate Chapin  <japhet@chromium.org>
1845
1846         Unreviewed, build fix.
1847
1848         Add a missing comma in webkitwebsettings.cpp.
1849
1850         * webkit/webkitwebsettings.cpp:
1851
1852 2010-09-23  Nate Chapin  <japhet@chromium.org>
1853
1854         Reviewed by Darin Fisher.
1855
1856         Add hyperlink auditing settings (i.e., <a ping>).
1857         https://bugs.webkit.org/show_bug.cgi?id=30458
1858
1859         * webkit/webkitwebsettings.cpp:
1860         (webkit_web_settings_class_init):
1861         (webkit_web_settings_set_property):
1862         (webkit_web_settings_get_property):
1863         (webkit_web_settings_copy):
1864         * webkit/webkitwebview.cpp:
1865         (webkit_web_view_update_settings):
1866         (webkit_web_view_settings_notify):
1867
1868 2010-09-22  Mario Sanchez Prada  <msanchez@igalia.com>
1869
1870         Reviewed by Martin Robinson.
1871
1872         [GTK] Replace instances of g_timeout_add() with g_idle_add() in testatk.c
1873         https://bugs.webkit.org/show_bug.cgi?id=46284
1874
1875         Use g_idle_add() whenever g_timeout_add() was being used.
1876
1877         This is desired because it's better to rely on the main loop to
1878         decide when it's a good moment to execute the task (when idle)
1879         than manually setting a timeout, which also sets a minimum amount
1880         of time needed to get the function executed.
1881
1882         * tests/testatk.c:
1883         (testWekitAtkTextSelections):
1884         (testWebkitAtkGetExtents):
1885         (testWebkitAtkListsOfItems):
1886         (main):
1887
1888 2010-09-22  Mario Sanchez Prada  <msanchez@igalia.com>
1889
1890         Reviewed by Martin Robinson.
1891
1892         [Gtk] object:text-changed events should be emitted for entries and password text
1893         https://bugs.webkit.org/show_bug.cgi?id=25898
1894
1895         New unit test to make sure text-changed signals are emitted
1896
1897         * tests/testatk.c:
1898         (textChangedCb): New. Signal handler for the
1899         text-changed::insert and text-changed::delete signals.
1900         (checkTextChangesAndBailOut): New. Source function to check
1901         the global result of the test and quit from the main loop.
1902         (testWebkitAtkTextChangedNotifications): New test.
1903         (main):
1904
1905 2010-09-22  Martin Robinson  <mrobinson@igalia.com>
1906
1907         Reviewed by Xan Lopez.
1908
1909         Added information about drop shadow improvements to the NEWS file.
1910
1911         * NEWS: Added information about drop shadow improvements.
1912
1913 2010-09-21  Xan Lopez  <xlopez@igalia.com>
1914
1915         Reviewed by Martin Robinson.
1916
1917         [GTK] Fix transfer annotations for WebKitGTK+ static API
1918         https://bugs.webkit.org/show_bug.cgi?id=46244
1919
1920         Deal with all the warnings related to transfer issues. Mostly the
1921         problem was the transfer being missing, but there were also typos
1922         in the function name or missing ':' at the end of the function
1923         name in the gtk-doc.
1924
1925         * webkit/webkitdownload.cpp:
1926         * webkit/webkitnetworkrequest.cpp:
1927         * webkit/webkitnetworkresponse.cpp:
1928         * webkit/webkitsecurityorigin.cpp:
1929         * webkit/webkitwebdatabase.cpp:
1930         * webkit/webkitwebdatasource.cpp:
1931         * webkit/webkitwebframe.cpp:
1932         * webkit/webkitwebhistoryitem.cpp:
1933         * webkit/webkitwebinspector.cpp:
1934         * webkit/webkitwebresource.cpp:
1935         * webkit/webkitwebsettings.cpp:
1936         * webkit/webkitwebview.cpp:
1937         (webkit_web_view_class_init):
1938         (webkit_web_view_get_main_frame):
1939
1940 2010-09-21  Xan Lopez  <xlopez@igalia.com>
1941
1942         Reviewed by Gustavo Noronha.
1943
1944         Update for 1.3.4 release.
1945
1946         * NEWS:
1947
1948 2010-09-17  Darin Adler  <darin@apple.com>
1949
1950         Reviewed by Sam Weinig.
1951
1952         REGRESSION (r60104): Zoom level is unexpectedly reset on page reload
1953         https://bugs.webkit.org/show_bug.cgi?id=42863
1954
1955         * webkit/webkitwebview.cpp:
1956         (webkit_web_view_get_zoom_level):
1957         (webkit_web_view_apply_zoom_level):
1958         (webkit_web_view_set_full_content_zoom):
1959         Call functions on Frame instead of FrameView.
1960
1961 2010-09-16  Martin Robinson  <mrobinson@igalia.com>
1962
1963         Reviewed by Xan Lopez.
1964
1965         [GTK] Implement dissolveDragImageToFraction
1966         https://bugs.webkit.org/show_bug.cgi?id=45826
1967
1968         To support full alpha in the drag icon, changed the drag icon to be a 
1969         GtkWindow with an RGBA colormap. Added logic to support painting the drag
1970         image to this window during the expose event.
1971
1972         * WebCoreSupport/DragClientGtk.cpp:
1973         (WebKit::dragIconWindowExposeEventCallback): Added.
1974         (WebKit::DragClient::DragClient): Initialize the new drag icon window.
1975         (WebKit::DragClient::~DragClient): Disconnect the expose event signal.
1976         (WebKit::DragClient::startDrag): Resize the drag icon window the appropriate size and
1977         if necessary, set its colormap.
1978         (WebKit::DragClient::dragIconWindowExposeEvent): Added.
1979         * WebCoreSupport/DragClientGtk.h: Added new member and method declarations.
1980
1981 2010-09-14  Philippe Normand  <pnormand@igalia.com>
1982
1983         Reviewed by Eric Carlson and Martin Robinson.
1984
1985         [GTK] eventSender.contextClick() should return the contents of the context menu
1986         https://bugs.webkit.org/show_bug.cgi?id=39102
1987
1988         New private WebView API to retrieve the context-menu widget. This
1989         is used by DRT only.
1990
1991         * webkit/webkitprivate.h:
1992         * webkit/webkitwebview.cpp:
1993         (webkit_web_view_get_context_menu):
1994
1995 2010-09-16  Alejandro G. Castro  <alex@igalia.com>
1996
1997         Reviewed by Xan Lopez.
1998
1999         GTK+ 3.x. updates, gtk_widget_size_request is deprecated and
2000         "activate-slider" style property for scrollbars is gone.
2001
2002         * webkit/webkitwebview.cpp:
2003         (PopupMenuPositionFunc):
2004
2005 2010-09-15  Martin Robinson  <mrobinson@igalia.com>
2006
2007         Reviewed by David Levin.
2008
2009         [GTK] [REGRESSION] r67591 broke the testwebbackforwardlist API test and introduced a memory leak
2010         https://bugs.webkit.org/show_bug.cgi?id=45865
2011
2012         When initializing the backForwardList private member of the WebView, do
2013         so with adoptPlatformRef to prevent a memory leak.
2014
2015         * webkit/webkitwebview.cpp:
2016         (webkit_web_view_init): Initialize member with adoptPlatformRef.
2017
2018 2010-09-15  Martin Robinson  <mrobinson@igalia.com>
2019
2020         Reviewed by Eric Seidel.
2021
2022         [Gtk] Use GOwnPtr for code that needs it
2023         https://bugs.webkit.org/show_bug.cgi?id=21594
2024
2025         Convert as reference counted private members of WebKitWebView to
2026         smart pointers as possible. This removes a lot of unecessary manual
2027         memory management. Also convert some pointer members away from pointer
2028         types, now that we are sure their destructors are called.
2029
2030         * WebCoreSupport/ContextMenuClientGtk.cpp:
2031         (WebKit::inputMethodsMenuItem): Updated to reflect PlatformRefPtr changes.
2032         * WebCoreSupport/DragClientGtk.cpp:
2033         (WebKit::DragClient::startDrag): Ditto.
2034         * WebCoreSupport/EditorClientGtk.cpp:
2035         (WebKit::EditorClient::setInputMethodState): Ditto.
2036         (WebKit::EditorClient::respondToChangedSelection): Ditto.
2037         (WebKit::EditorClient::handleInputMethodKeydown): Ditto.
2038         (WebKit::EditorClient::handleInputMethodMousePress): Ditto.
2039         (WebKit::EditorClient::EditorClient): Ditto.
2040         (WebKit::EditorClient::~EditorClient): Ditto.
2041         * WebCoreSupport/FrameLoaderClientGtk.cpp:
2042         (WebKit::postCommitFrameViewSetup): Ditto.
2043         * webkit/webkitprivate.h: Ditto.
2044         * webkit/webkitwebview.cpp: Ditto.
2045         (destroy_menu_cb): Ditto.
2046         (webkit_web_view_forward_context_menu_event): Ditto.
2047         (webkit_web_view_key_release_event): Ditto.
2048         (webkit_web_view_button_press_event): Ditto.
2049         (webkit_web_view_button_release_event): Ditto.
2050         (webkit_web_view_focus_in_event): Ditto.
2051         (webkit_web_view_focus_out_event): Ditto.
2052         (webkit_web_view_realize): Ditto.
2053         (webkit_web_view_set_scroll_adjustments): Ditto.
2054         (webkit_web_view_dispose): Changed all g_object_unref calls to
2055         PlatformRefPtr.clear(). Although this will also be done by the manual
2056         call to the WebKitWebViewPrivate destructor, the order that these
2057         fields are zero'd in is still very sensitive.
2058         (webkit_web_view_finalize): Updated to reflect PlatformRefPtr changes.
2059         (webViewGetDPI): Ditto.
2060         (webkit_web_view_screen_changed): Ditto.
2061         (webkit_web_view_drag_end): Ditto.
2062         (webkit_web_view_drag_data_get): Ditto.
2063         (doDragLeaveLater): Ditto.
2064         (webkit_web_view_drag_leave): Ditto.
2065         (webkit_web_view_drag_motion): Ditto.
2066         (webkit_web_view_drag_data_received): Ditto.
2067         (webkit_web_view_drag_drop): Ditto.
2068         (webkit_web_view_get_im_context): Ditto.
2069         (webkit_web_view_update_settings): Ditto.
2070         (webkit_web_view_init): Ditto.
2071         (webkit_web_view_set_settings): Ditto.
2072         (webkit_web_view_get_settings): Ditto.
2073         (webkit_web_view_get_inspector): Ditto.
2074         (webkit_web_view_set_window_features): Ditto.
2075         (webkit_web_view_get_window_features): Ditto.
2076         (webkit_web_view_get_back_forward_list): Ditto.
2077         (webkit_web_view_zoom_in): Ditto.
2078         (webkit_web_view_zoom_out): Ditto.
2079         (webkit_web_view_add_resource): Ditto.
2080         (webkit_web_view_get_resource): Ditto.
2081         (webkit_web_view_get_main_resource): Ditto.
2082         (webkit_web_view_clear_resources): Ditto.
2083         (webkit_web_view_get_subresources): Ditto.
2084
2085 2010-09-14  Martin Robinson  <mrobinson@igalia.com>
2086
2087         Reviewed by Xan Lopez.
2088
2089         [GTK] fast/events/keydown-numpad-keys.html produces many GLib warnings
2090         https://bugs.webkit.org/show_bug.cgi?id=45775
2091
2092         Handle the toggle-overwrite signal on the GtkTextView used for generating editing
2093         commands. Ignore this signals, as the default handler assumes that the GtkTextView
2094         has a layout and this one does not.
2095
2096         * WebCoreSupport/EditorClientGtk.cpp:
2097         (WebKit::toggleOverwriteCallback): Added. Cancels the default handler.
2098         (WebKit::EditorClient::EditorClient): Attach a handler for toggle-overwrite.
2099
2100 2010-09-14  Pavel Feldman  <pfeldman@chromium.org>
2101
2102         Reviewed by Yury Semikhatsky.
2103
2104         Web Inspector: Provide network-based load timing.
2105         Before this change, inspector used timers taken from
2106         within WebCore notifications (that are by definition
2107         synchronous and serialized). As a result, timing was
2108         affected by the routines running on the main thread
2109         (JavaScript and such).
2110         https://bugs.webkit.org/show_bug.cgi?id=45664
2111
2112         * webkit/webkitdownload.cpp:
2113         (DownloadClient::didFinishLoading):
2114
2115 2010-09-15  Sheriff Bot  <webkit.review.bot@gmail.com>
2116
2117         Unreviewed, rolling out r67551.
2118         http://trac.webkit.org/changeset/67551
2119         https://bugs.webkit.org/show_bug.cgi?id=45816
2120
2121         "Plugin tests fail" (Requested by yurys on #webkit).
2122
2123         * webkit/webkitdownload.cpp:
2124         (DownloadClient::didFinishLoading):
2125
2126 2010-09-14  Pavel Feldman  <pfeldman@chromium.org>
2127
2128         Reviewed by Yury Semikhatsky.
2129
2130         Web Inspector: Provide network-based load timing.
2131         Before this change, inspector used timers taken from
2132         within WebCore notifications (that are by definition
2133         synchronous and serialized). As a result, timing was
2134         affected by the routines running on the main thread
2135         (JavaScript and such).
2136         https://bugs.webkit.org/show_bug.cgi?id=45664
2137
2138         * webkit/webkitdownload.cpp:
2139         (DownloadClient::didFinishLoading):
2140
2141 2010-09-13  Enrica Casucci  <enrica@apple.com>
2142
2143         Reviewed by Sam Weinig.
2144
2145         Paste should be implemented in WebCore like Copy and Cut for Mac also.
2146         https://bugs.webkit.org/show_bug.cgi?id=45494
2147         <rdar://problem/7660537>
2148
2149         On the Mac platform, the implementation of the paste operation is all done
2150         at the WebKit level. In order to support it on WebKit2 it is necessary to
2151         refactor the code and move this functionality at the level of WebCore like
2152         we already have on Windows.
2153         The original code relies on some in AppKit functions that call back into
2154         WebKit causing problems in WebKit2. All this functionality has been moved
2155         at the level of the editor client where it can be dealt with appropriately.
2156
2157         * WebCoreSupport/FrameLoaderClientGtk.cpp:
2158         (WebKit::FrameLoaderClient::canShowMIMETypeAsHTML): Added stub.
2159         * WebCoreSupport/FrameLoaderClientGtk.h:
2160
2161 2010-09-13  Mario Sanchez Prada  <msanchez@igalia.com>
2162
2163         Reviewed by Martin Robinson.
2164
2165         [GTK] Provide unit tests for AtkText's text selection functions
2166         https://bugs.webkit.org/show_bug.cgi?id=43919
2167
2168         New tests to check getting, setting and removing text selections
2169
2170         * tests/testatk.c:
2171         (testWekitAtkTextSelections): New unit tests to check all the text
2172         selection related functions altogether through a single test
2173         function.
2174         (main):
2175
2176         Make sure that code dependant on getting information from the
2177         clipboard gets executed only when there's a GDK window associated
2178         to the webview widget, as that's not the case when executing the
2179         unit tests (the wedbview is not inside of any toplevel window) and
2180         will make the tests crash if not taken into account.
2181
2182         * WebCoreSupport/EditorClientGtk.cpp:
2183         (WebKit::EditorClient::respondToChangedSelection):
2184
2185 2010-09-11  Xan Lopez  <xlopez@igalia.com>
2186
2187         Reviewed by Martin Robinson.
2188
2189         [GTK] Make introspection work with g-o-i 0.9.5
2190         https://bugs.webkit.org/show_bug.cgi?id=45590
2191
2192         Use new type syntax and bump version number of the gir file.
2193
2194         * JSCore.gir.in:
2195
2196 2010-09-10  Jesus Sanchez-Palencia  <jesus.palencia@openbossa.org>
2197
2198         Reviewed by Darin Adler.
2199
2200         Add NetworkingContext to avoid layer violations
2201         https://bugs.webkit.org/show_bug.cgi?id=42292
2202
2203         * webkit/webkitdownload.cpp:
2204         (webkit_download_start):
2205         * webkit/webkitprivate.cpp:
2206         (currentToplevelCallback):
2207
2208 2010-09-10  Sam Weinig  <sam@webkit.org>
2209
2210         Fix GTK build.
2211
2212         * webkit/webkitwebview.cpp:
2213         (webkit_web_view_set_full_content_zoom):
2214
2215 2010-09-10  Sam Weinig  <sam@webkit.org>
2216
2217         Reviewed by Darin Adler.
2218
2219         Remove unnecessary constraint in WebCore of choosing either text zoom or full page zoom.
2220         Precursor to <rdar://problem/7660657>
2221         https://bugs.webkit.org/show_bug.cgi?id=45522
2222
2223         * webkit/webkitwebview.cpp:
2224         (webkit_web_view_get_zoom_level):
2225         (webkit_web_view_apply_zoom_level):
2226         (webkit_web_view_set_full_content_zoom):
2227
2228 2010-09-10  Mario Sanchez Prada  <msanchez@igalia.com>
2229
2230         Reviewed by Gustavo Noronha Silva.
2231
2232         [GTK] Fix warnings because of bad assignments in testatk.c
2233         https://bugs.webkit.org/show_bug.cgi?id=45538
2234
2235         Use AtkObject and AtkText instances properly in the code.
2236
2237         * tests/testatk.c:
2238         (testWebkitAtkListsOfItems):
2239
2240 2010-09-10  Adam Barth  <abarth@webkit.org>
2241
2242         Reviewed by Darin Fisher.
2243
2244         Move code from WebKit-layer to DocumentLoader
2245         https://bugs.webkit.org/show_bug.cgi?id=45569
2246
2247         * WebCoreSupport/FrameLoaderClientGtk.cpp:
2248         (WebKit::FrameLoaderClient::committedLoad):
2249
2250 2010-09-10  Adam Barth  <abarth@webkit.org>
2251
2252         Reviewed by Eric Seidel.
2253
2254         Main resource bytes shouldn't bounce through FrameLoader
2255         https://bugs.webkit.org/show_bug.cgi?id=45496
2256
2257         Now return the bytes to the DocumentLoader.
2258
2259         * WebCoreSupport/FrameLoaderClientGtk.cpp:
2260         (WebKit::FrameLoaderClient::committedLoad):
2261
2262 2010-09-10  Martin Robinson  <mrobinson@igalia.com>
2263
2264         Reviewed by Xan Lopez.
2265
2266         [GTK] Placement new / manual destructor invocation should be used on private GObject memory
2267         https://bugs.webkit.org/show_bug.cgi?id=45550
2268
2269         GLib allocates and deallocates GObject private data structs itself. When
2270         those structs contain C++ members, their constructors and destructors are not
2271         called. This is not only dangerous, it makes RefPtr-type smart pointers much
2272         less useful. We can fix this problem by calling placement new on the private
2273         data struct during instance initialization and calling the destructor during
2274         finalization.
2275
2276         This patch takes that approach and switches plain char* members of
2277         WebKitWebView (with manual memory allocation) to use CString.
2278
2279         * webkit/webkitprivate.h: Switch char* members to CString.
2280         * webkit/webkitwebview.cpp:
2281         (webkit_web_view_finalize): Manually call the destructor on the private
2282         data. Remove manual deallocation of members which are now CString.
2283         (webkit_web_view_query_tooltip): Update to reflect CString change.
2284         (webkit_web_view_init): Use placement new to initialize C++ members of
2285         the private data section.
2286         (webkit_web_view_get_encoding): Update to reflect CString change.
2287         (webkit_web_view_get_custom_encoding): Ditto.
2288         (webkit_web_view_add_resource): Ditto.
2289         (webkit_web_view_get_resource): Ditto.
2290         (webkit_web_view_clear_resources): Ditto.
2291         (webkit_web_view_set_tooltip_text): Ditto.
2292         (webkit_web_view_get_icon_uri): Ditto.
2293
2294 2010-09-10  Gustavo Noronha Silva  <gns@gnome.org>
2295
2296         Reviewed by Martin Robinson.
2297
2298         [GTK] Google sites do not like WebKitGTK+
2299         https://bugs.webkit.org/show_bug.cgi?id=39617
2300
2301         Special-case Google domains, and spoof User-Agent when talking to
2302         them, to stop being treated as a second-class citizen.
2303
2304         * WebCoreSupport/FrameLoaderClientGtk.cpp:
2305         (WebKit::initializeDomainsList):
2306         (WebKit::isGoogleDomain):
2307         (WebKit::FrameLoaderClient::userAgent): If enable-site-specific-quirks
2308         is enabled, send the standard WebKit User-Agent string, disregarding the
2309         custom one set by the browser for Google domains.
2310         * webkit/webkitprivate.h:
2311         * webkit/webkitwebsettings.cpp:
2312         (webkitPlatform): Fix style, and simplify.
2313         (webkitOSVersion): Ditto.
2314         (webkitUserAgent): Add the Version/x.y string Safari has been
2315         using since Safari 2.2.
2316
2317 2010-09-10  Xan Lopez  <xlopez@igalia.com>
2318
2319         Reviewed by Martin Robinson.
2320
2321         Fix compilation with GTK+ 3.x.
2322
2323         GDK key macros were renamed from GDK_FOO to GDK_KEY_FOO, we need
2324         to include the compat header provided if we want to keep using the
2325         old names.
2326
2327         * WebCoreSupport/FullscreenVideoController.cpp: include GtkVersioning.h
2328
2329 2010-09-08  Darin Adler  <darin@apple.com>
2330
2331         Reviewed by Adam Barth.
2332
2333         Move functions from Frame to Editor as planned
2334         https://bugs.webkit.org/show_bug.cgi?id=45218
2335
2336         * webkit/webkitwebview.cpp:
2337         (webkit_web_view_set_highlight_text_matches):
2338         (webkit_web_view_get_selected_text):
2339         (webkit_web_view_set_editable):
2340         Changed call sites to use editor().
2341
2342 2010-09-09  Philippe Normand  <pnormand@igalia.com>
2343
2344         Reviewed by Martin Robinson.
2345
2346         [GTK] testmimehandling falsely succeeds testing Ogg mime type
2347         https://bugs.webkit.org/show_bug.cgi?id=45349
2348
2349         Make this actually test an ogg file request instead of pdf and
2350         changed the expected result to match libsoup content-sniffing
2351         result in the case of ogg/vorbis file.
2352
2353         * tests/testmimehandling.c:
2354         (mime_type_policy_decision_requested_cb):
2355         (test_mime_ogg):
2356
2357 2010-09-08  Mario Sanchez Prada  <msanchez@igalia.com>
2358
2359         Reviewed by Martin Robinson.
2360
2361         [Gtk] A list item's number/bullet should not be a child of that list item
2362         https://bugs.webkit.org/show_bug.cgi?id=45190
2363
2364         New unit test added.
2365
2366         * tests/testatk.c:
2367         (testWebkitAtkListsOfItems): New test to check ordered/unordered
2368         list of items are properly exposed to AT technologies.
2369         (main):
2370
2371 2010-09-07  Martin Robinson  <mrobinson@igalia.com>
2372
2373         Reviewed by Xan Lopez.
2374
2375         [GTK] Fix some feature guards
2376         https://bugs.webkit.org/show_bug.cgi?id=45302
2377
2378         * webkit/webkitprivate.h: Add ENABLE(VIDEO) guards for video specific members.
2379
2380 2010-09-03  Jesus Sanchez-Palencia  <jesus.palencia@openbossa.org>
2381
2382         Reviewed by Darin Adler.
2383
2384         Add NetworkingContext to avoid layer violations
2385         https://bugs.webkit.org/show_bug.cgi?id=42292
2386
2387         Add GTK's specific implementation of FrameNetworkingContext.
2388
2389         * WebCoreSupport/FrameLoaderClientGtk.cpp:
2390         (WebKit::FrameLoaderClient::createNetworkingContext):
2391         * WebCoreSupport/FrameLoaderClientGtk.h:
2392
2393 2010-09-03  Xan Lopez  <xlopez@igalia.com>
2394
2395         Reviewed by Gustavo Noronha.
2396
2397         Dialog separators are gone in GTK+ 3.x.
2398
2399         * webkit/webkitsoupauthdialog.c:
2400         (show_auth_dialog):
2401
2402 2010-09-02  Yury Semikhatsky  <yurys@chromium.org>
2403
2404         Reviewed by Pavel Feldman.
2405
2406         REGRESSION: Crash occurs at objc_msgSend when closing a window that is displaying the web inspector
2407         https://bugs.webkit.org/show_bug.cgi?id=44230
2408
2409         * WebCoreSupport/InspectorClientGtk.cpp:
2410         (WebKit::notifyWebViewDestroyed):
2411         (WebKit::InspectorFrontendClient::destroyInspectorWindow):
2412         (WebKit::InspectorFrontendClient::closeWindow):
2413         (WebKit::InspectorFrontendClient::disconnectFromBackend):
2414         * WebCoreSupport/InspectorClientGtk.h:
2415
2416 2010-08-31  Dave Hyatt  <hyatt@apple.com>
2417
2418         Reviewed by Sam Weinig.
2419
2420         https://bugs.webkit.org/show_bug.cgi?id=44863, disentangle style recalc from layout, so that
2421         the former can occur in more places without having to do the latter.
2422
2423         * webkit/webkitwebview.cpp:
2424         (webkit_web_view_expose_event):
2425
2426 2010-08-31  Martin Robinson  <mrobinson@igalia.com>
2427
2428         Reviewed by Gustavo Noronha Silva.
2429
2430         [GTK] Isolate all GTK+ typedefs into one file
2431         https://bugs.webkit.org/show_bug.cgi?id=44900
2432
2433         * WebCoreSupport/EditorClientGtk.h: Remove GTK+ typedefs.
2434         * WebCoreSupport/FullscreenVideoController.h: Ditto.
2435
2436 2010-08-31  Gustavo Noronha Silva  <gustavo.noronha@collabora.co.uk>
2437
2438         Reviewed by Martin Robinson.
2439
2440         Preparations for the 1.3.4 release.
2441
2442         * NEWS:
2443         * docs/webkitgtk-docs.sgml:
2444
2445 2010-08-31  Gustavo Noronha Silva  <gustavo.noronha@collabora.co.uk>
2446
2447         Unreviewed. Minor documentation typo fix.
2448
2449         * webkit/webkitwebview.cpp:
2450
2451 2010-08-30  Alejandro G. Castro  <alex@igalia.com>
2452
2453         Reviewed by Martin Robinson.
2454
2455         [Gtk] gdk_display_get_core_pointer and gdk_device_get_core_pointer
2456         are deprecated
2457         https://bugs.webkit.org/show_bug.cgi?id=44787
2458
2459         We have replaced GtkVersioning.cpp with GtkVersioning.c and
2460         created a function (getDefaultGDKPointerDevice) to get the pointer
2461         of the window with the new APIs. We added that function to DRT and
2462         copyandpaste unit test.
2463
2464         * tests/testcopyandpaste.c:
2465         (runPasteTestCallback):
2466
2467 2010-08-30  Sheriff Bot  <webkit.review.bot@gmail.com>
2468
2469         Unreviewed, rolling out r66198.
2470         http://trac.webkit.org/changeset/66198
2471         https://bugs.webkit.org/show_bug.cgi?id=44856
2472
2473         It made tests crash on Qt bot (Requested by Ossy_ on #webkit).
2474
2475         * WebCoreSupport/InspectorClientGtk.cpp:
2476         (WebKit::notifyWebViewDestroyed):
2477         (WebKit::InspectorFrontendClient::destroyInspectorWindow):
2478         (WebKit::InspectorFrontendClient::closeWindow):
2479         * WebCoreSupport/InspectorClientGtk.h:
2480
2481 2010-08-27  Yury Semikhatsky  <yurys@chromium.org>
2482
2483         Reviewed by Pavel Feldman.
2484
2485         REGRESSION: Crash occurs at objc_msgSend when closing a window that is displaying the web inspector
2486         https://bugs.webkit.org/show_bug.cgi?id=44230
2487
2488         * WebCoreSupport/InspectorClientGtk.cpp:
2489         (WebKit::notifyWebViewDestroyed):
2490         (WebKit::InspectorFrontendClient::destroyInspectorWindow):
2491         (WebKit::InspectorFrontendClient::closeWindow):
2492         (WebKit::InspectorFrontendClient::disconnectFromBackend):
2493         * WebCoreSupport/InspectorClientGtk.h:
2494
2495 2010-08-26  Yury Semikhatsky  <yurys@chromium.org>
2496
2497         Unreviewed. Revert r66103 since Qt tests are failing.
2498
2499         * WebCoreSupport/InspectorClientGtk.cpp:
2500         (WebKit::notifyWebViewDestroyed):
2501         (WebKit::InspectorFrontendClient::destroyInspectorWindow):
2502         (WebKit::InspectorFrontendClient::closeWindow):
2503         * WebCoreSupport/InspectorClientGtk.h:
2504
2505 2010-08-26  Yury Semikhatsky  <yurys@chromium.org>
2506
2507         Reviewed by Pavel Feldman.
2508
2509         REGRESSION: Crash occurs at objc_msgSend when closing a window that is displaying the web inspector
2510         https://bugs.webkit.org/show_bug.cgi?id=44230
2511
2512         * WebCoreSupport/InspectorClientGtk.cpp:
2513         (WebKit::notifyWebViewDestroyed):
2514         (WebKit::InspectorFrontendClient::destroyInspectorWindow):
2515         (WebKit::InspectorFrontendClient::closeWindow):
2516         (WebKit::InspectorFrontendClient::disconnectFromBackend):
2517         * WebCoreSupport/InspectorClientGtk.h:
2518
2519 2010-08-26  Gustavo Noronha Silva  <gns@gnome.org>
2520
2521         Reviewed by Martin Robinson.
2522
2523         [GTK] WebKitGTK+ needs proper introspection annotation for most types and methods
2524         https://bugs.webkit.org/show_bug.cgi?id=44565
2525
2526         Add documentation and introspection specific information to a few
2527         getter and setter methods.
2528
2529         * webkit/webkitwebview.cpp:
2530
2531 2010-08-25  Martin Robinson  <mrobinson@igalia.com>
2532
2533         Reviewed by Gustavo Noronha Silva.
2534
2535         confirm dialog should show OK/Cancel instead of Yes/No for consistency
2536         https://bugs.webkit.org/show_bug.cgi?id=32877
2537
2538         Change the button labels for the fallback confirmation dialog to be
2539         OK and Cancel.
2540
2541         * webkit/webkitwebview.cpp:
2542         (webkit_web_view_script_dialog):
2543
2544 2010-08-25  Martin Robinson  <mrobinson@igalia.com>
2545
2546         Reviewed by Gustavo Noronha Silva.
2547
2548         Cairo and EFL port shouldn't depend on glib.
2549         https://bugs.webkit.org/show_bug.cgi?id=44354
2550
2551         Replace occurrences of GRefPtr and adoptGRef with PlatformRefPtr and
2552         adoptPlatformRef.
2553
2554         * WebCoreSupport/ChromeClientGtk.cpp:
2555         (WebKit::ChromeClient::requestGeolocationPermissionForFrame):
2556         * WebCoreSupport/DragClientGtk.cpp:
2557         (WebKit::DragClient::startDrag):
2558         * WebCoreSupport/EditorClientGtk.h:
2559         * WebCoreSupport/FrameLoaderClientGtk.cpp:
2560         (WebKit::FrameLoaderClient::dispatchWillSendRequest):
2561         (WebKit::FrameLoaderClient::dispatchDecidePolicyForMIMEType):
2562         (WebKit::FrameLoaderClient::createPlugin):
2563         * WebCoreSupport/FullscreenVideoController.h:
2564         * WebCoreSupport/InspectorClientGtk.cpp:
2565         (WebKit::InspectorClient::populateSetting):
2566         (WebKit::InspectorClient::storeSetting):
2567         * webkit/webkitdownload.cpp:
2568         (webkit_download_error):
2569         * webkit/webkitnetworkrequest.cpp:
2570         (webkit_network_request_new_with_core_request):
2571         * webkit/webkitnetworkresponse.cpp:
2572         (webkit_network_response_new_with_core_response):
2573
2574 2010-08-24  Xan Lopez  <xlopez@igalia.com>
2575
2576         Reviewed by Martin Robinson.
2577
2578         Set a device to the event struct when building with GTK+ 3.x, it's
2579         mandatory.
2580
2581         * tests/testcopyandpaste.c:
2582         (runPasteTestCallback):
2583
2584 2010-08-24  Martin Robinson  <mrobinson@igalia.com>
2585
2586         Build fix for GTK+.
2587
2588         * tests/testkeyevents.c:
2589         (key_press_event_cb):
2590         (key_release_event_cb):
2591
2592 2010-08-24  Martin Robinson  <mrobinson@igalia.com>
2593
2594         Reviewed by Xan Lopez.
2595
2596         [GTK] GTK3: EventSender and API tests should use gdk_event_new when synthesizing events
2597         https://bugs.webkit.org/show_bug.cgi?id=44534
2598
2599         Fix an issue where a GdkEvent* is treated like a GdkEvent.
2600
2601         * tests/testkeyevents.c:
2602         (key_press_event_cb):
2603         (key_release_event_cb):
2604
2605 2010-08-24  Martin Robinson  <mrobinson@igalia.com>
2606
2607         Reviewed by Xan Lopez.
2608
2609         [GTK] GTK3: EventSender and API tests should use gdk_event_new when synthesizing events
2610         https://bugs.webkit.org/show_bug.cgi?id=44534
2611
2612         * tests/testcopyandpaste.c:
2613         (runPasteTestCallback): Switch to using gdk_event_new and also reference the GdkWindow.
2614         when setting it on the event. It is dereferenced by gdk_event_free(...).
2615         * tests/testhittestresult.c:
2616         (load_status_cb): Switch to using gdk_event_new.
2617
2618 2010-08-24  Gustavo Noronha Silva  <gustavo.noronha@collabora.co.uk>
2619
2620         Rubber-stamped by Xan Lopez.
2621
2622         Do not initialize the GSettings object in webkit_init - the other
2623         call sites are always called in the main thread so this is
2624         unnecessary.
2625
2626         * webkit/webkitprivate.cpp:
2627         (webkit_init):
2628
2629 2010-08-20  Martin Robinson  <mrobinson@igalia.com>
2630
2631         Reviewed by Pavel Feldman.
2632
2633         Web Inspector: Inspector tests were disabled for GTK.
2634         https://bugs.webkit.org/show_bug.cgi?id=43977
2635
2636         Allow setting the inspector resources path via an environment variables
2637         and add a method, inspectorFilesPath, to the WebKit InspectorClient that
2638         encapsulates this logic.
2639
2640         * WebCoreSupport/InspectorClientGtk.cpp:
2641         (WebKit::InspectorClient::openInspectorFrontend): Use the new inspectorFilesPath method to
2642         get the path to the resource.
2643         (WebKit::InspectorClient::inspectorFilesPath): Added.
2644         (WebKit::InspectorFrontendClient::localizedStringsURL): Use inspectorFilesPath method.
2645         * WebCoreSupport/InspectorClientGtk.h: Add method and member variables for caching the result.
2646
2647 2010-08-19  Gustavo Noronha Silva  <gustavo.noronha@collabora.co.uk>
2648
2649         Reviewed by Kenneth Rohde Christiansen.
2650
2651         [Gtk] Enable view mode media feature layout test
2652         https://bugs.webkit.org/show_bug.cgi?id=43278
2653
2654         Add the view-mode property, to control the WebCore page's viewMode.
2655
2656         * docs/webkitgtk-sections.txt:
2657         * webkit/webkitwebview.cpp:
2658         (webkit_web_view_get_property):
2659         (webkit_web_view_set_property):
2660         (webkit_web_view_class_init):
2661         (webkit_web_view_set_view_mode):
2662         (webkit_web_view_get_view_mode):
2663         * webkit/webkitwebview.h:
2664
2665 2010-08-19  Philippe Normand  <pnormand@igalia.com>
2666
2667         Reviewed by Gustavo Noronha Silva.
2668
2669         [GStreamer] GTK XOverlay support in GStreamerGWorld
2670         https://bugs.webkit.org/show_bug.cgi?id=39474
2671
2672         New FullscreenVideoController object, private in the webview. It
2673         is created when the user presses the fullscreen button of the
2674         video controls. Video is displayed in fullscreen, controls are
2675         displayed when the user moves the mouse and when the video is
2676         paused. There's also basic keyboard shortcuts support: F/f to
2677         leave fullscreen, space to toggle play/pause and up/down to
2678         control volume.
2679
2680         * WebCoreSupport/ChromeClientGtk.cpp:
2681         (WebKit::ChromeClient::supportsFullscreenForNode):
2682         (WebKit::ChromeClient::enterFullscreenForNode):
2683         (WebKit::ChromeClient::exitFullscreenForNode):
2684         * WebCoreSupport/ChromeClientGtk.h:
2685         * WebCoreSupport/FullscreenVideoController.cpp: Added.
2686         (hideHudCallback):
2687         (onFullscreenGtkMotionNotifyEvent):
2688         (onFullscreenGtkActiveNotification):
2689         (onFullscreenGtkConfigureEvent):
2690         (onFullscreenGtkDestroy):
2691         (togglePlayPauseActivated):
2692         (exitFullscreenActivated):
2693         (progressBarUpdateCallback):
2694         (timeScaleButtonPressed):
2695         (timeScaleButtonReleased):
2696         (timeScaleValueChanged):
2697         (volumeValueChanged):
2698         (playerVolumeChangedCallback):
2699         (playerMuteChangedCallback):
2700         (FullscreenVideoController::FullscreenVideoController):
2701         (FullscreenVideoController::~FullscreenVideoController):
2702         (FullscreenVideoController::setMediaElement):
2703         (FullscreenVideoController::gtkConfigure):
2704         (FullscreenVideoController::showHud):
2705         (FullscreenVideoController::hideHud):
2706         (onFullscreenGtkKeyPressEvent):
2707         (FullscreenVideoController::enterFullscreen):
2708         (FullscreenVideoController::updateHudPosition):
2709         (FullscreenVideoController::exitOnUserRequest):
2710         (FullscreenVideoController::exitFullscreen):
2711         (FullscreenVideoController::canPlay):
2712         (FullscreenVideoController::play):
2713         (FullscreenVideoController::pause):
2714         (FullscreenVideoController::playStateChanged):
2715         (FullscreenVideoController::togglePlay):
2716         (FullscreenVideoController::volume):
2717         (FullscreenVideoController::muted):
2718         (FullscreenVideoController::setVolume):
2719         (FullscreenVideoController::volumeChanged):
2720         (FullscreenVideoController::muteChanged):
2721         (FullscreenVideoController::currentTime):
2722         (FullscreenVideoController::setCurrentTime):
2723         (FullscreenVideoController::duration):
2724         (FullscreenVideoController::percentLoaded):
2725         (FullscreenVideoController::beginSeek):
2726         (FullscreenVideoController::doSeek):
2727         (FullscreenVideoController::endSeek):
2728         (timeToString):
2729         (FullscreenVideoController::updateHudProgressBar):
2730         (FullscreenVideoController::createHud):
2731         * WebCoreSupport/FullscreenVideoController.h: Added.
2732         (FullscreenVideoController::mediaElement):
2733         * webkit/webkitprivate.cpp:
2734         (webkit_web_view_enter_fullscreen):
2735         (webkit_web_view_exit_fullscreen):
2736         * webkit/webkitprivate.h:
2737
2738 2010-08-18  Gustavo Noronha Silva  <gustavo.noronha@collabora.co.uk>
2739
2740         Reviewed by Martin Robinson.
2741
2742         [GTK] WebBackForwardList's new_with_web_view should not be public
2743         https://bugs.webkit.org/show_bug.cgi?id=43802
2744
2745         Deprecate the constructor. The introspection annotation has been
2746         fixed in a previous commit.
2747
2748         * webkit/webkitwebbackforwardlist.cpp:
2749
2750 2010-08-17  Ilya Tikhonovsky  <loislo@chromium.org>
2751
2752         Reviewed by Yury Semikhatsky.
2753
2754         Web Inspector: replace hand written InspectorBackendStub.js with generated one.
2755         https://bugs.webkit.org/show_bug.cgi?id=43791
2756
2757         * WebCoreSupport/InspectorClientGtk.cpp:
2758         (WebKit::InspectorClient::openInspectorFrontend):
2759
2760 2010-08-18  Sheriff Bot  <webkit.review.bot@gmail.com>
2761
2762         Unreviewed, rolling out r65595.
2763         http://trac.webkit.org/changeset/65595
2764         https://bugs.webkit.org/show_bug.cgi?id=44161
2765
2766         qt build failed (Requested by loislo on #webkit).
2767
2768         * WebCoreSupport/InspectorClientGtk.cpp:
2769         (WebKit::InspectorClient::openInspectorFrontend):
2770
2771 2010-08-17  Ilya Tikhonovsky  <loislo@chromium.org>
2772
2773         Reviewed by Yury Semikhatsky.
2774
2775         Web Inspector: replace hand written InspectorBackendStub.js by generated one.
2776         https://bugs.webkit.org/show_bug.cgi?id=43791
2777
2778         * WebCoreSupport/InspectorClientGtk.cpp:
2779         (WebKit::InspectorClient::openInspectorFrontend):
2780
2781 2010-08-17  Xan Lopez  <xlopez@igalia.com>
2782
2783         Reviewed by Gustavo Noronha.
2784
2785         Fix the builds when HAVE_GSETTINGS is defined.
2786
2787         * webkit/webkitprivate.cpp:
2788         (isSchemaAvailable):
2789         (inspectorGSettings):
2790
2791 2010-08-17  Jesus Sanchez-Palencia  <jesus.palencia@openbossa.org>
2792
2793         Reviewed by Darin Adler.
2794
2795         Add NetworkingContext to avoid layer violations
2796         https://bugs.webkit.org/show_bug.cgi?id=42292
2797
2798         Preparation: Just add the files to the build system.
2799
2800         * WebCoreSupport/FrameNetworkingContextGtk.h: Added.
2801         Placeholder with tentative code that might be changed when landing
2802         the rest of it.
2803
2804 2010-08-17  Gustavo Noronha Silva  <gustavo.noronha@collabora.co.uk>
2805
2806         Reviewed by Martin Robinson.
2807
2808         WebBackForwardList.get_{back|forward}_list_with_limit not introspectable
2809         https://bugs.webkit.org/show_bug.cgi?id=43054
2810
2811         Annotate WebKitWebBackForwardList and the function that is used to fetch it from
2812         the WebKitWebView. More will follow.
2813
2814         * webkit/webkitwebbackforwardlist.cpp: Annotations, and small
2815         improvements to the docs.
2816         * webkit/webkitwebbackforwardlist.h: Couple empty lines missing.
2817         * webkit/webkitwebview.cpp:
2818
2819 2010-08-17  Martin Robinson  <mrobinson@igalia.com>
2820
2821         Reviewed by Gustavo Noronha Silva.
2822
2823         [GTK] Clean up WebCore/platform/graphics/gtk/ImageGtk.cpp
2824         https://bugs.webkit.org/show_bug.cgi?id=44069
2825
2826         * WebCoreSupport/InspectorClientGtk.cpp:
2827         (WebKit::InspectorClient::openInspectorFrontend): Switched to using the new WEBKITGTK_API_VERSION_STRING define.
2828         * webkit/webkitprivate.cpp:
2829         (inspectorGSettings): Ditto.
2830
2831 2010-08-17  Sheriff Bot  <webkit.review.bot@gmail.com>
2832
2833         Unreviewed, rolling out r65500.
2834         http://trac.webkit.org/changeset/65500
2835         https://bugs.webkit.org/show_bug.cgi?id=44108
2836
2837         Qt bots failed to compile. (Requested by loislo on #webkit).
2838
2839         * WebCoreSupport/InspectorClientGtk.cpp:
2840         (WebKit::InspectorClient::openInspectorFrontend):
2841
2842 2010-08-17  Ilya Tikhonovsky  <loislo@chromium.org>
2843
2844         Reviewed by Yury Semikhatsky.
2845
2846         Web Inspector: replace hand written InspectorBackendStub.js by generated one.
2847         https://bugs.webkit.org/show_bug.cgi?id=43791
2848
2849         * WebCoreSupport/InspectorClientGtk.cpp:
2850         (WebKit::InspectorClient::openInspectorFrontend):
2851
2852 2010-08-17  Gustavo Noronha Silva  <gustavo.noronha@collabora.co.uk>
2853
2854         Reviewed by Martin Robinson.
2855
2856         [GTK] GSettings support adds annoying warnings on systems with older glib versions
2857         https://bugs.webkit.org/show_bug.cgi?id=44056
2858
2859         Use #ifdef instead of #if for HAVE_GSETTINGS to avoid warnings
2860         when it is not defined.
2861
2862         * WebCoreSupport/InspectorClientGtk.cpp:
2863         * webkit/webkitprivate.cpp:
2864         (webkit_init):
2865         * webkit/webkitprivate.h:
2866
2867 2010-08-12  Gustavo Noronha Silva  <gustavo.noronha@collabora.co.uk>
2868
2869         Really add a missing file. I wish I would remember running git add
2870         on them after applying a patch =(.
2871
2872         * org.webkitgtk.gschema.xml.in: Added.
2873
2874 2010-08-12  Gustavo Noronha Silva  <gustavo.noronha@collabora.co.uk>
2875
2876         Reviewed by Martin Robinson.
2877
2878         [GTK] Use GSettings to save/restore Web Inspector settings
2879         https://bugs.webkit.org/show_bug.cgi?id=43512
2880
2881         Use GSettings to save/restore settings used by the Web Inspector.
2882         A few helper functions were added to have settings that use
2883         boolean values be actual booleans, and also to conform with the
2884         GSettings naming requirements for keys.
2885
2886         * WebCoreSupport/InspectorClientGtk.cpp:
2887         (WebKit::toGSettingName):
2888         (WebKit::truthStringFromVariant):
2889         (WebKit::variantFromTruthString):
2890         (WebKit::InspectorClient::populateSetting):
2891         (WebKit::InspectorClient::storeSetting):
2892         * org.webkit.gtk.gschema.xml: Added.
2893         * webkit/webkitprivate.cpp:
2894         (inspectorGSettings):
2895         (webkit_init):
2896         * webkit/webkitprivate.h:
2897
2898 2010-08-11  Joone Hur  <joone@kldp.org>
2899
2900         Reviewed by Martin Robinson.
2901
2902         [GTK] Last Hangul letter is typed again when a composition is finished with mouse press
2903         https://bugs.webkit.org/show_bug.cgi?id=40518
2904
2905         When a mouse press fires during a IME composition, the current composition character   
2906         can be entered twice at the previous editing position and a new editing position.
2907         Because the IME commit signal is emitted after the mouse press event. 
2908         This patch allows to prevent the commit signal during a composition when a mouse press fires.
2909
2910         * WebCoreSupport/EditorClientGtk.cpp: 
2911         (WebKit::imContextCommitted):
2912         (WebKit::EditorClient::handleInputMethodKeydown): Allow to accept the next composition commit.
2913         (WebKit::EditorClient::handleInputMethodMousePress): Added for handling IME when a mouse press fires.
2914         (WebKit::EditorClient::EditorClient):  Initialize m_preventNextCompositionCommit
2915         * WebCoreSupport/EditorClientGtk.h:
2916         (WebKit::EditorClient::preventNextCompositionCommit): Added for checking whether skipping a commit.
2917         * webkit/webkitwebview.cpp:
2918         (webkit_web_view_button_press_event): Call handleInputMethodMousePress() 
2919
2920 2010-08-11  Gustavo Noronha Silva  <gustavo.noronha@collabora.co.uk>
2921
2922         Reviewed by Martin Robinson.
2923
2924         Small improvement to the documentation of the geolocation-policy-decision-requested
2925         signal, stating more clearly what is expected from the handler.
2926
2927         * webkit/webkitwebview.cpp:
2928         (webkit_web_view_class_init):
2929
2930 2010-08-11  Gustavo Noronha Silva  <gustavo.noronha@collabora.co.uk>
2931
2932         Reviewed by Martin Robinson.
2933
2934         [GTK] Geolocation policy decision object leaking
2935         https://bugs.webkit.org/show_bug.cgi?id=43884
2936
2937         Use GRefPtr on the policy decision object to fix the leak.
2938
2939         * WebCoreSupport/ChromeClientGtk.cpp:
2940         (WebKit::ChromeClient::requestGeolocationPermissionForFrame):
2941
2942 2010-08-10  Chris Marrin  <cmarrin@apple.com>
2943
2944         Reviewed by Oliver Hunt.
2945
2946         Add suspendAnimations/resumeAnimation API to DRT
2947         https://bugs.webkit.org/show_bug.cgi?id=43733
2948         
2949         Gtk specific API.
2950
2951         * webkit/webkitprivate.h:
2952         * webkit/webkitwebframe.cpp:
2953         (webkit_web_frame_suspend_animations):
2954         (webkit_web_frame_resume_animations):
2955
2956 2010-08-10  Gavin Barraclough  <barraclough@apple.com>
2957
2958         Rubber stamped by Sam Weinig.
2959
2960         Bug 43786 - Move AtomicStringHash from WebCore to WTF
2961         Also remove deprecated string headers from WebCore/platform/text.
2962
2963         * gdom/ConvertToGCharPrivate.h:
2964
2965 2010-08-06  Gavin Barraclough  <barraclough@apple.com>
2966
2967         Rubber stamped by Sam Weinig
2968
2969         Bug 43594 - Add string forwards to Forward.h
2970         This allows us to remove forward declarations for these classes from
2971         WebCore/WebKit (a step in moving these class from WebCore:: to WTF::).
2972
2973         * WebCoreSupport/InspectorClientGtk.h:
2974
2975 2010-08-09  Martin Robinson  <mrobinson@igalia.com>
2976
2977         Reviewed by Xan Lopez.
2978
2979         [gtk] gtk_im_context_focus_in() should only be called when an input-able element has focus
2980         https://bugs.webkit.org/show_bug.cgi?id=43602
2981
2982         Only call gtk_im_context_focus_in() when we focus a frame which is currently in
2983         an editable node. 
2984
2985         * webkit/webkitwebview.cpp:
2986         (webkit_web_view_focus_in_event): Conditionalize call to gtk_im_context_focus_in().
2987
2988 2010-08-06  Gustavo Noronha Silva  <gustavo.noronha@collabora.co.uk>
2989
2990         Reviewed by Xan Lopez.
2991
2992         Use the new paths, and also fix the path for localizedStrings.
2993
2994         * WebCoreSupport/InspectorClientGtk.cpp:
2995         (WebKit::InspectorClient::openInspectorFrontend):
2996         (WebKit::InspectorFrontendClient::localizedStringsURL):
2997
2998 2010-08-06  Jessie Berlin  <jberlin@apple.com>
2999
3000         Roll out http://trac.webkit.org/changeset/64801, which broke the Safari Windows Build.
3001         Unreviewed.
3002
3003         * WebCoreSupport/InspectorClientGtk.h:
3004
3005 2010-08-05  Xan Lopez  <xlopez@igalia.com>
3006
3007         Reviewed by Gustavo Noronha.
3008
3009         [GTK] Use correct path for Inspector data files in GTK+3.x builds
3010         https://bugs.webkit.org/show_bug.cgi?id=43445
3011
3012         Use the right data path for inspector files when building with
3013         GTK+ 3.x.
3014
3015         * WebCoreSupport/InspectorClientGtk.cpp:
3016         (WebKit::InspectorClient::openInspectorFrontend):
3017
3018 2010-08-03  Daniel Bates  <dbates@rim.com>
3019
3020         Fix misspelled word 'teh' in the description of the XSS Auditor setting.
3021
3022         * webkit/webkitwebsettings.cpp:
3023         (webkit_web_settings_class_init):
3024
3025 2010-08-03  Xan Lopez  <xlopez@igalia.com>
3026
3027         Reviewed by Gustavo Noronha.
3028
3029         [GTK] Fix DOM event dispatch
3030         https://bugs.webkit.org/show_bug.cgi?id=40847
3031
3032         Test DOM event dispatch.
3033
3034         * tests/testdomdomwindow.c:
3035         (load_event_callback):
3036         (test_dom_domview_signals):
3037         (clicked_cb):
3038         (load_status_callback):
3039         (test_dom_domview_dispatch_event):
3040         (main):
3041
3042 2010-08-03  Gustavo Noronha Silva  <gustavo.noronha@collabora.co.uk>
3043
3044         Reviewed by Xan Lopez.
3045
3046         [GTK] Does not paint the node highlight used by the inspector
3047         https://bugs.webkit.org/show_bug.cgi?id=43429
3048
3049         Implement highlighting the nodes when using the inspector.
3050
3051         * WebCoreSupport/InspectorClientGtk.cpp:
3052         (WebKit::InspectorClient::highlight):
3053         (WebKit::InspectorClient::hideHighlight):
3054         * webkit/webkitwebview.cpp:
3055         (webkit_web_view_expose_event):
3056
3057 2010-08-03  Xan Lopez  <xlopez@igalia.com>
3058
3059         Reviewed by Gustavo Noronha.
3060
3061         Update unit tests now that string parameters are 'const char*'.
3062
3063         * tests/testdomdocument.c:
3064         (test_dom_document_title):
3065         (test_dom_document_get_elements_by_tag_name):
3066         (test_dom_document_get_elements_by_class_name):
3067         (test_dom_document_get_element_by_id):
3068         * tests/testdomnode.c:
3069         (test_dom_node_insertion):
3070
3071 2010-08-02  Martin Robinson  <mrobinson@igalia.com>
3072
3073         Reviewed by Xan Lopez.
3074
3075         [GTK] DRT implement execCommand()
3076         https://bugs.webkit.org/show_bug.cgi?id=35351
3077
3078         * webkit/webkitprivate.h: Add some private methods for the DRT which enable
3079         LayoutTestController.execCommand and LayoutTestController.isCommandEnabled:
3080         webkit_web_view_execute_core_command_by_name and webkit_web_view_is_command_enabled.
3081         * webkit/webkitwebview.cpp: 
3082         (webkit_web_view_execute_core_command_by_name): added.
3083         (webkit_web_view_is_command_enabled): Added.
3084
3085 2010-07-27  Luiz Agostini  <luiz.agostini@openbossa.org>
3086
3087         Reviewed by Darin Fisher.
3088
3089         PopupMenu refactoring in preparation to WebKit2
3090         https://bugs.webkit.org/show_bug.cgi?id=42592
3091
3092         As ChromeClient was made responsible for providing PopupMenu and SearchPopupMenu
3093         instances, concrete classes that inherit from ChromeClient needed to be changed to
3094         implement the new methods.
3095
3096         * WebCoreSupport/ChromeClientGtk.cpp:
3097         (WebKit::ChromeClient::selectItemWritingDirectionIsNatural):
3098         (WebKit::ChromeClient::createPopupMenu):
3099         (WebKit::ChromeClient::createSearchPopupMenu):
3100         * WebCoreSupport/ChromeClientGtk.h:
3101
3102 2010-08-02  Martin Robinson  <mrobinson@igalia.com>
3103
3104         Reviewed by Xan Lopez.
3105
3106         [GTK] Create a frame-created signal
3107         https://bugs.webkit.org/show_bug.cgi?id=43284
3108
3109         Add a frame-created signal, which will allow developers to track
3110         the beginning of a frame lifecycle and attach signal handlers to
3111         all new frames.
3112
3113         * WebCoreSupport/FrameLoaderClientGtk.cpp:
3114         (WebKit::FrameLoaderClient::createFrame): Emit the frame-created signal.
3115         * tests/testwebframe.c: Add a test which verifies that the correct number of
3116         frame-created signals is fired when a page with iframes loads.
3117         (createFrameSignalTestFrameCreatedCallback): Added.
3118         (createFrameSignalTestTimeout): Added.
3119         (test_webkit_web_frame_created_signal): Added.
3120         (main): Add a reference to the new test.
3121         * webkit/webkitprivate.h: De-normalize webkit_web_frame_init_with_web_view into
3122         the one place that it is used, so that the frame-created signal may be fired there.
3123         * webkit/webkitwebframe.cpp: Remove webkit_web_frame_init_with_web_view.
3124         * webkit/webkitwebview.cpp: 
3125         (webkit_web_view_class_init): Add the frame-created signal declaration.
3126
3127 2010-08-02  Mario Sanchez Prada  <msanchez@igalia.com>
3128
3129         Reviewed by Chris Fleizach.
3130
3131         [GTK] Implement support for get_character_extents and get_range_extents
3132         https://bugs.webkit.org/show_bug.cgi?id=25677
3133
3134         Added new unit tests to check get_character_extents and
3135         get_range_extents functions for the ATK_TEXT interface
3136
3137         Based on a previous patch by Joanmarie Diggs.
3138
3139         * tests/testatk.c:
3140         (test_webkit_atk_get_extents):
3141         (main):
3142
3143 2010-08-02  Jeremy Orlow  <jorlow@chromium.org>
3144
3145         Speculative revert of 64425 due to Chromium instability
3146         https://bugs.webkit.org/show_bug.cgi?id=43347
3147
3148         * WebCoreSupport/ChromeClientGtk.cpp:
3149         * WebCoreSupport/ChromeClientGtk.h:
3150
3151 2010-07-27  Luiz Agostini  <luiz.agostini@openbossa.org>
3152
3153         Reviewed by Darin Fisher.
3154
3155         PopupMenu refactoring in preparation to WebKit2
3156         https://bugs.webkit.org/show_bug.cgi?id=42592
3157
3158         As ChromeClient was made responsible for providing PopupMenu and SearchPopupMenu
3159         instances, concrete classes that inherit from ChromeClient needed to be changed to
3160         implement the new methods.
3161
3162         * WebCoreSupport/ChromeClientGtk.cpp:
3163         (WebKit::ChromeClient::selectItemWritingDirectionIsNatural):
3164         (WebKit::ChromeClient::createPopupMenu):
3165         (WebKit::ChromeClient::createSearchPopupMenu):
3166         * WebCoreSupport/ChromeClientGtk.h:
3167
3168 2010-07-31  Sheriff Bot  <webkit.review.bot@gmail.com>
3169
3170         Unreviewed, rolling out r64422.
3171         http://trac.webkit.org/changeset/64422
3172         https://bugs.webkit.org/show_bug.cgi?id=43304
3173
3174         Build fixes are needed for Snow Leopard and Windows.
3175         (Requested by lca on #webkit).
3176
3177         * WebCoreSupport/ChromeClientGtk.cpp:
3178         * WebCoreSupport/ChromeClientGtk.h:
3179
3180 2010-07-27  Luiz Agostini  <luiz.agostini@openbossa.org>
3181
3182         Reviewed by Darin Fisher.
3183
3184         PopupMenu refactoring in preparation to WebKit2
3185         https://bugs.webkit.org/show_bug.cgi?id=42592
3186
3187         As ChromeClient was made responsible for providing PopupMenu and SearchPopupMenu
3188         instances, concrete classes that inherit from ChromeClient needed to be changed to
3189         implement the new methods.
3190
3191         * WebCoreSupport/ChromeClientGtk.cpp:
3192         (WebKit::ChromeClient::selectItemWritingDirectionIsNatural):
3193         (WebKit::ChromeClient::createPopupMenu):
3194         (WebKit::ChromeClient::createSearchPopupMenu):
3195         * WebCoreSupport/ChromeClientGtk.h:
3196
3197 2010-07-30  Joseph Pecoraro  <joepeck@webkit.org>
3198
3199         Reviewed by David Kilzer.
3200
3201         Limit ApplicationCache Total and Per-Origin Storage Capacity (Quotas)
3202         https://bugs.webkit.org/show_bug.cgi?id=40627
3203
3204         * WebCoreSupport/ChromeClientGtk.cpp:
3205         (WebKit::ChromeClient::reachedApplicationCacheOriginQuota):
3206         * WebCoreSupport/ChromeClientGtk.h:
3207 2010-07-26  Steve Block  <steveblock@google.com>
3208
3209         Reviewed by Jeremy Orlow.
3210
3211         Page clients should be passed to Page constructor via structure of pointers
3212         https://bugs.webkit.org/show_bug.cgi?id=42834
3213
3214         * webkit/webkitwebview.cpp:
3215         (webkit_web_view_init):
3216
3217 2010-07-16  Zhe Su  <suzhe@chromium.org>
3218
3219         Reviewed by Darin Adler.
3220
3221         REGRESSION(r61484): Broke focus behaviour on Qt and probably other platforms
3222         https://bugs.webkit.org/show_bug.cgi?id=42253
3223
3224         Dummy implementation of EditorClient::willSetInputMethodState.
3225
3226         * WebCoreSupport/EditorClientGtk.cpp:
3227         (WebKit::EditorClient::willSetInputMethodState):
3228         * WebCoreSupport/EditorClientGtk.h:
3229
3230 2010-07-16  Nate Chapin  <japhet@chromium.org>
3231
3232         Reviewed by Darin Fisher.
3233
3234         Uncomment an assert that broke due to r63100.
3235
3236         https://bugs.webkit.org/show_bug.cgi?id=42298
3237
3238         * tests/testwebview.c:
3239
3240 2010-07-14  Sam Weinig  <sam@webkit.org>
3241
3242         Reviewed by Darin Adler.
3243
3244         Patch for https://bugs.webkit.org/show_bug.cgi?id=42232
3245         Make changing Cursors work in WebKit2.
3246
3247         * WebCoreSupport/ChromeClientGtk.cpp:
3248         (WebKit::ChromeClient::setCursor):
3249         * WebCoreSupport/ChromeClientGtk.h:
3250         Change prototype to match new one.
3251
3252 2010-07-12  Martin Robinson  <mrobinson@igalia.com>
3253
3254         Reviewed by Gustavo Noronha Silva.
3255
3256         [GTK] r63100 broke the testwebview test
3257         https://bugs.webkit.org/show_bug.cgi?id=42114
3258
3259         * tests/testwebview.c: Disable a failing assertion in test_webkit_web_view_adjustments
3260         until we can get to the bottom of the failure.
3261
3262 2010-07-12  Xan Lopez  <xlopez@igalia.com>
3263
3264         Reviewed by Gustavo Noronha.
3265
3266         Update for 1.3.3
3267
3268         * NEWS:
3269
3270 2010-07-12  Xan Lopez  <xlopez@igalia.com>
3271
3272         Reviewed by Gustavo Noronha.
3273
3274         Fix compilation with sealed GTK+.
3275
3276         * webkit/webkitwebview.cpp:
3277         (webkit_web_view_drag_motion):
3278         (webkit_web_view_drag_data_received):
3279         (webkit_web_view_drag_drop):
3280
3281 2010-07-11  Martin Robinson  <mrobinson@igalia.com>
3282
3283         Reviewed by Xan Lopez.
3284
3285         [GTK] WebKitWebView should support drops
3286         https://bugs.webkit.org/show_bug.cgi?id=39843
3287
3288         Add support for dropping content onto GTK+ WebViews.
3289
3290         * webkit/webkitprivate.h:
3291         Add a DroppingContext struct to keep track of drop data while the drop is
3292         in progress. Have WebKitWebView keep a map of GdkDragContexts to DroppingContexts.
3293         * webkit/webkitwebview.cpp: Add necessary includes and globalPointForClientPoint helper.
3294         (webkit_web_view_popup_menu_handler): Use globalPointForClientPoint helper.
3295         (webkit_web_view_dispose): Properly clean up droppingContexts member.
3296         (webkit_web_view_finalize): Delete droppingContext member.
3297         (globalPointForClientPoint): Added.
3298         (doDragLeaveLater): Added.
3299         (webkit_web_view_drag_leave): Added.
3300         (webkit_web_view_drag_motion): Added.
3301         (webkit_web_view_drag_data_received): Added.
3302         (webkit_web_view_drag_drop): Added.
3303         (webkit_web_view_class_init): Connect new drop signal handlers to the widget definition.
3304         (webkit_web_view_init): Set up the widget as a drop destination and initialize droppingContexts.
3305
3306 2010-07-07  Sam Weinig  <sam@webkit.org>
3307
3308         Reviewed by Anders Carlsson.
3309
3310         Patch for https://bugs.webkit.org/show_bug.cgi?id=41772
3311         Add basic piping for BackForwardControllerClient.
3312
3313         * webkit/webkitwebview.cpp:
3314         (webkit_web_view_init):
3315
3316 2010-06-19  Ilya Tikhonovsky  <loislo@chromium.org>
3317
3318         Reviewed by Yury Semikhatsky.
3319
3320         WebInspector: js function name was fixed.
3321         dispatchMessageToFrontend -> dispatchMessageFromBackend.
3322         https://bugs.webkit.org/show_bug.cgi?id=40675
3323
3324         * WebCoreSupport/InspectorClientGtk.cpp:
3325         (WebKit::InspectorClient::sendMessageToFrontend):
3326
3327 2010-06-23  Martin Robinson  <mrobinson@igalia.com>
3328
3329         Reviewed by Gustavo Noronha Silva.
3330
3331         [GTK] Separate DerivedSources per-project
3332         https://bugs.webkit.org/show_bug.cgi?id=41109
3333
3334         Separate WebKitGTK+ DerivedSources into per-project subdirectories to prepare
3335         for properly building WebKit2.
3336
3337         * webkitmarshal.list: Touch this file to force a rebuild of the marshaling code.
3338
3339 2010-07-01  Xan Lopez  <xlopez@igalia.com>
3340
3341         Reviewed by Gustavo Noronha.
3342
3343         [GTK] Stop using GdkRegion in 3.x mode
3344         https://bugs.webkit.org/show_bug.cgi?id=41463
3345
3346         Make us compile without using GdkRegion, since it's gone from GTK+
3347         3.x.
3348
3349         * WebCoreSupport/ChromeClientGtk.cpp:
3350         (WebKit::ChromeClient::scroll):
3351         * webkit/webkitwebview.cpp:
3352         (webkit_web_view_expose_event):
3353
3354 2010-07-01  Martin Robinson  <mrobinson@igalia.com>
3355
3356         Reviewed by Xan Lopez.
3357
3358         [GTK] Convert WebKitWebView's private draggingDataObjects member to a pointer
3359         https://bugs.webkit.org/show_bug.cgi?id=40333
3360
3361         Convert draggingDataObjects to a pointer. The constructor and destructor of the
3362         HashMap should be called explicitly. GObject memory allocation does not do this.
3363
3364         * WebCoreSupport/DragClientGtk.cpp:
3365         (WebKit::DragClient::startDrag):
3366         * webkit/webkitprivate.h:
3367         * webkit/webkitwebview.cpp:
3368         (webkit_web_view_dispose):
3369         (webkit_web_view_drag_end):
3370         (webkit_web_view_drag_data_get):
3371         (webkit_web_view_init):
3372
3373 2010-06-30  Martin Robinson  <mrobinson@igalia.com>
3374
3375         Unreviewed, rolling out r62226.
3376         http://trac.webkit.org/changeset/62226
3377         https://bugs.webkit.org/show_bug.cgi?id=40333
3378
3379         This change caused 13 new crashers.
3380
3381         * WebCoreSupport/DragClientGtk.cpp:
3382         (WebKit::DragClient::startDrag):
3383         * webkit/webkitprivate.h:
3384         * webkit/webkitwebview.cpp:
3385         (webkit_web_view_dispose):
3386         (webkit_web_view_drag_end):
3387         (webkit_web_view_drag_data_get):
3388         (webkit_web_view_init):
3389
3390 2010-06-30  Martin Robinson  <mrobinson@igalia.com>
3391
3392         Reviewed by Xan Lopez.
3393
3394         [GTK] Convert WebKitWebView's private draggingDataObjects member to a pointer
3395         https://bugs.webkit.org/show_bug.cgi?id=40333
3396
3397         Convert draggingDataObjects to a pointer. The constructor and destructor of the
3398         HashMap should be called explicitly. GObject memory allocation does not do this.
3399
3400         * WebCoreSupport/DragClientGtk.cpp:
3401         (WebKit::DragClient::startDrag):
3402         * webkit/webkitprivate.h:
3403         * webkit/webkitwebview.cpp:
3404         (webkit_web_view_dispose):
3405         (webkit_web_view_drag_end):
3406         (webkit_web_view_drag_data_get):
3407         (webkit_web_view_init):
3408
3409 2010-06-30  Xan Lopez  <xlopez@igalia.com>
3410
3411         Reviewed by Gustavo Noronha.
3412
3413         [GTK] Unit test for DOM insertion methods
3414         https://bugs.webkit.org/show_bug.cgi?id=40495
3415
3416         Add unit test for webkit_dom_node_remove_child.
3417
3418         * tests/testdomnode.c:
3419         (test_dom_node_insertion):
3420
3421 2010-06-30  Xan Lopez  <xlopez@igalia.com>
3422
3423         Reviewed by Gustavo Noronha.
3424
3425         [GTK] Unit test for DOM insertion methods
3426         https://bugs.webkit.org/show_bug.cgi?id=40495
3427
3428         Add unit tests for node insertion and replacement methods.
3429
3430         * tests/testdomnode.c:
3431         (test_dom_node_insertion):
3432         (main):
3433
3434 2010-06-30  Mario Sanchez Prada  <msanchez@igalia.com>
3435
3436         Reviewed by Xan Lopez.
3437
3438         [GTK] Random failure on 'testdownload' unit test
3439         https://bugs.webkit.org/show_bug.cgi?id=38256
3440
3441         Make sure the set_filename function is called after handling the
3442         'download-requested' signal for the asynchronous test. This change
3443         is needed because it could happen, with the current code, that the
3444         set_filename () function got called too early in the mainloop,
3445         therefore screwing the tests because the 'theDownload' global
3446         variable wouldn't be properly set yet.
3447
3448         With this modification we ensure the set_filename() function gets
3449         called always after handling the download-requested signal,
3450         therefore avoiding potentialproblems.
3451
3452         * tests/testdownload.c:
3453         (set_filename):
3454         (handle_download_requested_cb):
3455         (download_requested_cb):
3456         (download_requested_asynch_cb):
3457         (test_webkit_download_perform):
3458
3459 2010-06-30  José Millán Soto  <jmillan@igalia.com>
3460
3461         Reviewed by Xan Lopez.
3462
3463         [Gtk] Text attributes not exposed
3464         https://bugs.webkit.org/show_bug.cgi?id=25528
3465
3466         Added new tests for accessible text attributes
3467
3468         * tests/testatk.c:
3469         (compAtkAttribute):
3470         (compAtkAttributeName):
3471         (atkAttributeSetAttributeHasValue):
3472         (atkAttributeSetAreEqual):
3473         (testWebkitAtkTextAttributes):
3474         (main):
3475
3476 2010-06-28  Xan Lopez  <xlopez@igalia.com>
3477
3478         Reviewed by Gustavo Noronha.
3479
3480         Update for 1.3.2.
3481
3482         * NEWS:
3483
3484 2010-06-28  Xan Lopez  <xlopez@igalia.com>
3485
3486         Reviewed by Gustavo Noronha.
3487
3488         [GTK] Does not compile with -DGSEAL_ENABLE
3489         https://bugs.webkit.org/show_bug.cgi?id=37851
3490
3491         Fix build with GSEAL enabled.
3492
3493         * WebCoreSupport/FrameLoaderClientGtk.cpp:
3494         (WebKit::FrameLoaderClient::transitionToCommittedForNewPage):
3495         * webkit/webkitsoupauthdialog.c:
3496         (show_auth_dialog):
3497
3498 2010-06-28  Xan Lopez  <xlopez@igalia.com>
3499
3500         Reviewed by Gustavo Noronha.
3501
3502         [GTK] Add support for GTK+3
3503         https://bugs.webkit.org/show_bug.cgi?id=41253
3504
3505         Adapt build system to 3.x support, allow the JSCore gir and webkit
3506         pc files to be configurable at build time for different API
3507         versions.
3508
3509         * JSCore-1.0.gir: Removed.
3510         * JSCore.gir.in: Added.
3511         * docs/GNUmakefile.am:
3512         * webkit.pc.in:
3513
3514 2010-06-25  Mario Sanchez Prada  <msanchez@igalia.com>
3515
3516         Reviewed by Xan Lopez.
3517
3518         [Gtk] Implement atk_table_get_column_header
3519         https://bugs.webkit.org/show_bug.cgi?id=30896
3520
3521         Replace g_timeout_add() with g_idle_add().
3522
3523         * tests/testatk.c:
3524         (test_webkit_atk_get_text_at_offset_forms):
3525         (test_webkit_atk_get_text_at_offset):
3526         (test_webkit_atk_get_text_at_offset_newlines):
3527         (test_webkit_atk_get_text_at_offset_textarea):
3528         (test_webkit_atk_get_text_at_offset_text_input):
3529         (testWebkitAtkGetTextInParagraphAndBodySimple):
3530         (testWebkitAtkGetTextInParagraphAndBodyModerate):
3531         (testWebkitAtkGetTextInTable):
3532         (testWebkitAtkGetHeadersInTable):
3533
3534 2010-06-25  Mario Sanchez Prada  <msanchez@igalia.com>
3535
3536         Reviewed by Xan Lopez.
3537
3538         [Gtk] Implement atk_table_get_column_header
3539         https://bugs.webkit.org/show_bug.cgi?id=30896
3540
3541         Added new test to check whether the implementation of
3542         get_column_headers() and get_row_headers() works ok.
3543
3544         * tests/testatk.c:
3545         (testWebkitAtkGetHeadersInTable):
3546         (main):
3547
3548 2010-06-25  Martin Robinson  <mrobinson@igalia.com>
3549
3550         Reviewed by Xan Lopez.
3551
3552         [GTK] Pasteboard code does not take into account the nil character when getting and setting markup/netscape-url data
3553         https://bugs.webkit.org/show_bug.cgi?id=41221
3554
3555         Add a test for this issue which simulates a paste keyboard event into an editable
3556         body. The pasteboard should contain a markup portion containing a null terminator.
3557
3558         * tests/testcopyandpaste.c:
3559         (test_info_new):
3560         (test_info_destroy):
3561         (load_status_cb):
3562         (runPasteTestCallback):
3563         (window_object_cleared_callback):
3564         (pasting_test_get_data_callback):
3565         (pasting_test_clear_data_callback):
3566         (test_pasting_markup):
3567         (main):
3568
3569 2010-06-24  Martin Robinson  <mrobinson@igalia.com>
3570
3571         Reviewed by Xan Lopez.
3572
3573         [GTK] Cannot change the selection via the keyboard
3574         https://bugs.webkit.org/show_bug.cgi?id=41162
3575
3576         Fix issue where the selection could not be extended via the keyboard by
3577         adjusting the logic guarding against inserting text in non-editable nodes.
3578
3579         * WebCoreSupport/EditorClientGtk.cpp:
3580         (WebKit::EditorClient::handleKeyboardEvent):
3581         Allow editor commands that do not insert text in non-editable nodes. This
3582         fixes keyboard selection extension in non-editable nodes. Move the existing
3583         check to after the execution of any editor commands.
3584
3585 2010-06-15  Dumitru Daniliuc  <dumi@chromium.org>
3586
3587         Reviewed by Adam Barth.
3588
3589         Move isAvailable()/setIsAvailable() from Database/DatabaseSync to AbstractDatabase.
3590         https://bugs.webkit.org/show_bug.cgi?id=39041
3591
3592         * webkit/webkitwebview.cpp:
3593         (webkit_web_view_update_settings):
3594         (webkit_web_view_settings_notify):
3595         (webkit_get_cache_model):
3596
3597 2010-06-16  Martin Robinson  <mrobinson@igalia.com>
3598
3599         Reviewed by Gustavo Noronha Silva.
3600
3601         [GTK] Remove the abuse of GDK_CURRENT_TIME in the DRT
3602         https://bugs.webkit.org/show_bug.cgi?id=40600
3603
3604         * WebCoreSupport/DragClientGtk.cpp: 
3605         (WebKit::DragClient::startDrag): Reset the click count after a drag starts.
3606         * WebCoreSupport/FrameLoaderClientGtk.cpp:
3607         (WebKit::postCommitFrameViewSetup): Reset the click count after a load is committed.
3608         * webkit/webkitprivate.h: Move static click counting variables to be per-view.
3609         * webkit/webkitwebview.cpp: 
3610         (getEventTime): Added.
3611         (webkit_web_view_button_press_event): If the event time is zero, use the current time.
3612         (webkit_web_view_finalize): Clean up click counting member.
3613         (webkit_web_view_init): Initialize click counting member.
3614
3615 2010-06-15  Xan Lopez  <xlopez@igalia.com>
3616
3617         Fix compilation with older GTK+.
3618
3619         * WebCoreSupport/ChromeClientGtk.cpp:
3620         (WebKit::ChromeClient::pageRect):
3621
3622 2010-06-15  Xan Lopez  <xlopez@igalia.com>
3623
3624         Reviewed by Gustavo Noronha.
3625
3626         [GTK] Does not compile with -DGSEAL_ENABLE
3627         https://bugs.webkit.org/show_bug.cgi?id=37851
3628
3629         Fix compilation with GSEAL_ENABLE.
3630
3631         * WebCoreSupport/ChromeClientGtk.cpp:
3632         (WebKit::ChromeClient::pageRect):
3633         (WebKit::ChromeClient::contentsSizeChanged):
3634         * tests/testdomnode.c:
3635         (test_dom_node_insertion):
3636         * webkit/webkitwebview.cpp:
3637         (webkit_web_view_realize):
3638         (webkit_web_view_script_dialog):
3639         (webkit_web_view_drag_end):
3640         (webkit_web_view_init):
3641
3642 2010-06-14  Ilya Tikhonovsky  <loislo@chromium.org>
3643
3644         Unreviewed build fix.
3645
3646         This is a fix for flaky inspector tests at gtk-debug bots.
3647