Remove unused support for NFC normalization during text encoding

Remove unused support for NFC normalization during text encoding

Previous CLs have updated all of the paths that were incorrectly
performing normalization during text encoding, so this code is
now unused.

BUG=545678
R=pdr@chromium.org
TBR=jochen@chromium.org

Committed: https://crrev.com/859bb51120f88fd7e1b53e052be488f31dd4a926
Cr-Commit-Position: refs/heads/master@{#358360}

[jsbell, 2015-10-29 19:48:49]

This depends on:

https://codereview.chromium.org/1420483005/
https://codereview.chromium.org/1424503006/

[jsbell, 2015-11-02 23:45:17]

This is ready to go now. jshin@ - please take a look?

[jsbell, 2015-11-04 18:56:17]

jsbell@chromium.org changed reviewers:
+ pdr@chromium.org

[jsbell, 2015-11-04 18:56:17]

Since this is just code deletion could you take a look at this one too, pdr@ ?

[jsbell, 2015-11-04 18:56:27]

Description was changed from

==========
Remove unused support for NFC normalization during text encoding

Previous CLs have updated all of the paths that were incorrectly
performing normalization during text encoding, so this code is
now unused.

BUG=545678
R=jshin@chromium.org
==========

to

==========
Remove unused support for NFC normalization during text encoding

Previous CLs have updated all of the paths that were incorrectly
performing normalization during text encoding, so this code is
now unused.

BUG=545678
==========

[pdr., 2015-11-06 02:55:08]

On 2015/11/04 at 18:56:17, jsbell wrote:
> Since this is just code deletion could you take a look at this one too, pdr@ ?

LGTM

[jsbell, 2015-11-06 17:17:44]

The CQ bit was checked by jsbell@chromium.org

[commit-bot: I haz the power, 2015-11-06 17:18:24]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1424303002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1424303002/1

[commit-bot: I haz the power, 2015-11-06 17:26:41]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-11-06 17:26:42]

Try jobs failed on following builders:
  chromium_presubmit on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[jsbell, 2015-11-06 17:29:19]

jsbell@chromium.org changed reviewers:
+ jochen@chromium.org
- jshin@chromium.org, pdr@chromium.org

[jsbell, 2015-11-06 17:29:19]

*sigh* wtf OWNERS; jochen@ - can you approve?

[pdr., 2015-11-06 18:15:50]

On 2015/11/06 at 17:29:19, jsbell wrote:
> *sigh* wtf OWNERS; jochen@ - can you approve?

Ooh yeah, I'm not a wtf OWNER any more :P

I would recommend TBR=jochen

[jsbell, 2015-11-06 18:22:55]

Description was changed from

==========
Remove unused support for NFC normalization during text encoding

Previous CLs have updated all of the paths that were incorrectly
performing normalization during text encoding, so this code is
now unused.

BUG=545678
==========

to

==========
Remove unused support for NFC normalization during text encoding

Previous CLs have updated all of the paths that were incorrectly
performing normalization during text encoding, so this code is
now unused.

BUG=545678
R=pdr@chromium.org
TBR=jochen@chromium.org
==========

[jsbell, 2015-11-06 18:22:55]

jsbell@chromium.org changed reviewers:
+ pdr@chromium.org

[jsbell, 2015-11-06 18:23:05]

The CQ bit was checked by jsbell@chromium.org

[jsbell, 2015-11-06 18:23:43]

On 2015/11/06 18:15:50, pdr wrote:
> On 2015/11/06 at 17:29:19, jsbell wrote:
> > *sigh* wtf OWNERS; jochen@ - can you approve?
> 
> Ooh yeah, I'm not a wtf OWNER any more :P
> 
> I would recommend TBR=jochen

Sure, that's just what a non-OWNER would recommend. ¬_¬

[commit-bot: I haz the power, 2015-11-06 18:24:05]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1424303002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1424303002/1

[commit-bot: I haz the power, 2015-11-06 18:44:10]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-11-06 18:45:08]

Patchset 1 (id:??) landed as
https://crrev.com/859bb51120f88fd7e1b53e052be488f31dd4a926
Cr-Commit-Position: refs/heads/master@{#358360}