[Webkit-unassigned] [Bug 96001] [BlackBerry] Implement a color picker

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Thu Sep 6 13:03:50 PDT 2012


--- Comment #8 from Christophe Dumez <christophe.dumez at intel.com>  2012-09-06 13:04:04 PST ---
(From update of attachment 162560)
View in context: https://bugs.webkit.org/attachment.cgi?id=162560&action=review

> Source/WebKit/blackberry/WebCoreSupport/ColorPickerClient.cpp:105
> +    writer.addData(m_source.utf8().data(), m_source.utf8().length());

My comment about caching m_source.utf8() in a local variable was still valid I believe. Any reason you did not fix it?
Your copying the string twice here.

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