[Webkit-unassigned] [Bug 188174] [WinCairo] Search terms are not saved for <input type="search">
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Sun Oct 14 19:31:35 PDT 2018
https://bugs.webkit.org/show_bug.cgi?id=188174
Fujii Hironori <Hironori.Fujii at sony.com> changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #352094|review? |review-
Flags| |
--- Comment #17 from Fujii Hironori <Hironori.Fujii at sony.com> ---
Comment on attachment 352094
--> https://bugs.webkit.org/attachment.cgi?id=352094
Save search inputs to filesystem
View in context: https://bugs.webkit.org/attachment.cgi?id=352094&action=review
> Source/WebCore/platform/win/SearchPopupMenuWin.cpp:25
> +#include <WebCore/UserAgent.h>
Do you need to include <WebCore/UserAgent.h> in this file?
> Source/WebCore/platform/win/SearchPopupMenuWin.cpp:58
> + file.append(name);
Can `name` contain '/', '\', or '..'? How much is the maximum length?
How do you think the idea using single SQLite database, for example 'search.db' instead of many files?
> Source/WebCore/platform/win/SearchPopupMenuWin.cpp:72
> +void saveRecentSearches(const String& name, const Vector<WebCore::RecentSearch>& searchItems)
I think you shouldn't use 'WebCore::' in WebCore.
> Source/WebCore/platform/win/SearchPopupMenuWin.cpp:91
> + auto nullTerminatedSearchItem = WTF::stringToNullTerminatedWChar(searchItems[i].string);
Remove 'WTF::' because there is using statement.
https://github.com/WebKit/webkit/blob/2fde85661b4bf2a3ad6344694401d91e9220e99e/Source/WTF/wtf/text/win/WCharStringExtras.h#L68
--
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-unassigned/attachments/20181015/a714ca6e/attachment.html>
More information about the webkit-unassigned
mailing list