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