LGTM. https://codereview.appspot.com/322760043/diff/1/source/common/ucasemap.cpp File source/common/ucasemap.cpp (right): https://codereview.appspot.com/322760043/diff/1/source/common/ucasemap.cpp#ne... source/common/ucasemap.cpp:471: (src[titleStart] == 0x0049 || src[titleStart] == 0x0069)) { could use Unicode single character literals here, u'I', u'j', etc. https://codereview.appspot.com/322760043/diff/1/source/common/ucasemap_titlec... File source/common/ucasemap_titlecase_brkiter.cpp (right): https://codereview.appspot.com/322760043/diff/1/source/common/ucasemap_titlec... source/common/ucasemap_titlecase_brkiter.cpp:59: utext_close(&utext); If iter was passed in from the caller, we return it with a dangling reference for it's text. A little odd, but probably ok.
LGTM