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