[Webkit-unassigned] [Bug 27543] Add platform/wince/ files for WINCE port

Wed Aug 12 12:17:24 PDT 2009

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

George Staikos <staikos at kde.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #34374|review?                     |review-
               Flag|                            |

--- Comment #56 from George Staikos <staikos at kde.org>  2009-08-12 12:17:21 PDT ---
(From update of attachment 34374)

> +static bool getWebLocData(IDataObject* dataObject, String& url, String* title)
> +{
> +    return false;
> +}

Should probably add some UNUSED_PARAM() here

> +    CString cf_html = "Version:0.9";
> +    CString startHTML = "\nStartHTML:";
> +    CString endHTML = "\nEndHTML:";
> +    CString startFragment = "\nStartFragment:";
> +    CString endFragment = "\nEndFragment:";
> +    CString sourceURL = "\nSourceURL:";
> +
> +    append(result, cf_html);
> +
> +    bool shouldFillSourceURL = !srcURL.isEmpty() && (srcURL != "about:blank");
> +
> +    CString startMarkup = "\n<HTML>\n<BODY>\n<!--StartFragment-->\n";
> +    CString endMarkup = "\n<!--EndFragment-->\n</BODY>\n</HTML>";

   all these CString should be "const" right?  static too maybe?

> +void ClipboardWince::clearData(const String& type)
> +{
> +    notImplemented();
> +}

This file can use lots of UNUSED_PARAM()

The rest seems fine

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