RESOLVED FIXED 16329
[GTK] Two small cleanups 
https://bugs.webkit.org/show_bug.cgi?id=16329
Summary [GTK] Two small cleanups
Xan Lopez
Reported 2007-12-06 08:23:58 PST
Rename m_webPage to m_webView in ChromeClient and chain to parent class in WebKitWebView to activate bindings.
Attachments
Patch (8.89 KB, patch)
2007-12-06 08:25 PST, Xan Lopez 		no flags
DetailsFormatted DiffDiff
Cleanup (9.33 KB, patch)
2007-12-06 11:55 PST, Xan Lopez 		alp: review+
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Xan Lopez
Comment 1 2007-12-06 08:25:12 PST
Created attachment 17749 [details] Patch
Xan Lopez
Comment 2 2007-12-06 08:29:24 PST
+ return GTK_WIDGET_CLASS(webkit_web_view_parent_class)->key_press_event (widget, event); Extra space, blah :)
Xan Lopez
Comment 3 2007-12-06 11:55:45 PST
Created attachment 17753 [details] Cleanup
Alp Toker
Comment 4 2007-12-06 12:08:31 PST
Comment on attachment 17753 [details] Cleanup Good, apart from a whitespace tweak (not your fault, it was that way before the change) and chaining up key_release to key_press (will fix up before landing).
Alp Toker
Comment 5 2007-12-10 13:09:39 PST
Landed in r28490.
Note You need to log in before you can comment on or make changes to this bug.