[Webkit-unassigned] [Bug 89872] [WK2] Add new C API to generate MHTML data from the UI process

[bugzilla-daemon at webkit.org]

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





--- Comment #9 from Carlos Garcia Campos <cgarcia at igalia.com>  2012-07-18 03:50:20 PST ---
(From update of attachment 149472)
View in context: https://bugs.webkit.org/attachment.cgi?id=149472&action=review

Patch looks good to me in general, but since it adds new C API it would be great if Anders or Sam could take a look at it too.

> Source/WebKit2/WebProcess/WebPage/WebPage.cpp:1845
> +    if (buffer)
> +        dataReference = CoreIPC::DataReference(reinterpret_cast<const uint8_t*>(buffer->data()), buffer->size());

I don't think buffer can be null

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