[Webkit-unassigned] [Bug 27028] [gtk] Pasteboard/GtkClipboard can't handle the "text/html" target.

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Fri Aug 14 23:04:54 PDT 2009


https://bugs.webkit.org/show_bug.cgi?id=27028


Jan Alonzo <jmalonzo at gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #32560|review-                     |review+
               Flag|                            |




--- Comment #17 from Jan Alonzo <jmalonzo at gmail.com>  2009-08-14 23:04:53 PDT ---
(From update of attachment 32560)
> -    if ((gint)info == WEBKIT_WEB_VIEW_TARGET_INFO_HTML) {
> +    if ((gint)info == Pasteboard::generalPasteboard()->m_helper->getWebViewTargetInfoHtml()) {
>          gtk_selection_data_set(selection_data, selection_data->target, 8,
>                                 reinterpret_cast<const guchar*>(clipboardData->markup()),
>                                 g_utf8_strlen(clipboardData->markup(), -1));

The braces should be removed as there's only one statement here.

I've looked into this again and seems I overlooked a few things in my review.
This patch is fine. r=me.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.



More information about the webkit-unassigned mailing list