Add BreakIterator::getRuleStatus

The CL https://codereview.chromium.org/23618052/ depends of an API that will be added by libicu 52, BreakIterator::getRuleStatus. This patch is just a copy and paste of the libicu 52 API into chromium libicu.

BUG=N/A

[igoroliveira, 2013-09-17 22:06:19]

On 2013/09/17 21:52:22, Igor Oliveira wrote:

Adding icu OWNER.

[Mark Mentovai, 2013-09-17 22:58:04]

→jungshik

[jungshik at Google, 2013-09-18 00:28:34]

On 2013/09/17 22:58:04, Mark Mentovai wrote:
> →jungshik

I'm going to update ICU to 52.1 so that we can nuke this change soon. Anyway,
for the record, could you follow the usual pattern of patching ICU locally? 

1. Add an entry to README.chromium
2. Add a patch file to patches directory

Thank you

[igoroliveira, 2013-09-18 20:41:29]

On 2013/09/18 00:28:34, Jungshik Shin wrote:
> On 2013/09/17 22:58:04, Mark Mentovai wrote:
> > →jungshik
> 
> I'm going to update ICU to 52.1 so that we can nuke this change soon. Anyway,
> for the record, could you follow the usual pattern of patching ICU locally? 
> 
> 1. Add an entry to README.chromium
> 2. Add a patch file to patches directory
> 
> Thank you

Done!

[jungshik at Google, 2013-09-18 20:55:19]

LGTM with one nit below addressed. 

BTW, we have to keep this patch even after my on-going because I'm upgrading to
ICU 51.2 (ICU 52 will be out next week and perhaps we want to wait for ICU 52.1
to get a more stable version).

https://codereview.chromium.org/23480090/diff/10001/patches/breakiterator.patch
File patches/breakiterator.patch (right):

https://codereview.chromium.org/23480090/diff/10001/patches/breakiterator.pat...
patches/breakiterator.patch:5: an API that will be added by libicu 5.2,
nit: ICU changed its versioning scheme. Instead of libicu 5.2, you may just say
ICU 52. You can change the CL description and elsewhere accordingly.

[igoroliveira, 2013-09-19 14:06:22]

On 2013/09/18 20:55:19, Jungshik Shin wrote:
> LGTM with one nit below addressed. 
> 
> BTW, we have to keep this patch even after my on-going because I'm upgrading
to
> ICU 51.2 (ICU 52 will be out next week and perhaps we want to wait for ICU
52.1
> to get a more stable version).
> 
>
https://codereview.chromium.org/23480090/diff/10001/patches/breakiterator.patch
> File patches/breakiterator.patch (right):
> 
>
https://codereview.chromium.org/23480090/diff/10001/patches/breakiterator.pat...
> patches/breakiterator.patch:5: an API that will be added by libicu 5.2,
> nit: ICU changed its versioning scheme. Instead of libicu 5.2, you may just
say
> ICU 52. You can change the CL description and elsewhere accordingly.

Ok fixed. Someone can kick the CQ?

[igoroliveira, 2013-09-19 17:55:27]

Adding third_party owners.

[jungshik at Google, 2013-09-19 19:38:39]

On 2013/09/19 17:55:27, Igor Oliveira wrote:
> Adding third_party owners.

After this goes in, you have to roll icu to this version by updating DEPS file.

[igoroliveira, 2013-09-19 19:39:49]

On 2013/09/19 19:38:39, Jungshik Shin wrote:
> On 2013/09/19 17:55:27, Igor Oliveira wrote:
> > Adding third_party owners.
> 
> After this goes in, you have to roll icu to this version by updating DEPS
file.

I am not a commit so i probably can not commit this patch neither roll deps :).

[jungshik at Google, 2013-09-23 16:43:40]

On 2013/09/19 19:39:49, Igor Oliveira wrote:
> On 2013/09/19 19:38:39, Jungshik Shin wrote:
> > On 2013/09/19 17:55:27, Igor Oliveira wrote:
> > > Adding third_party owners.
> > 
> > After this goes in, you have to roll icu to this version by updating DEPS
> file.
> 
> I am not a commit so i probably can not commit this patch neither roll deps
:).

Hmm... somehow, checking 'commit' box didn't do anything so far. I'll check that
in and roll icu after that.

[jungshik at Google, 2013-09-23 22:42:34]

Landed manually with a little change in the patch file and README.chromium. See
https://codereview.chromium.org/23532086/