On 2014/06/10 15:18:16, yurys wrote:
On 2014/06/10 15:02:56, marja wrote:
> Yeah okay, let's match the ContentSearchUtils.cpp. The latest patch set
does
> that.
ContentSearchUtils.cpp would return empty string if there is a whitespace
inside
url as if there were no source url at all while implementation in PS#9
will
accept that and return that value with trimmed whitespaces.
Ah true, I missed that. Will fix tomorrow. Oh well, it has other "disallowed
chars" too, and this will be a can of worms if we start to discuss what
chars
exactly we want to allow here...
https://codereview.chromium.org/316173002/
--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
---
You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
For more options, visit https://groups.google.com/d/optout.
