On 2015/04/07 20:07:04, arv wrote:
LGTM
Consider adding the following before CQ'ing:
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_chromium_rel_ng;tryserver.blink:linux_blink_rel
to the CL description since it is likely that there are Blink tests that
needs
to be updated
thanks, will do.
I did run the layout tests locally and none of the failures seemed related
to
the change though (and failed without the patch applied in the same way).
So I'm
not sure any of the layout tests need fixes from this
https://codereview.chromium.org/1027283004/
--
--
v8-dev mailing list
v8-dev@googlegroups.com
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 v8-dev+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.