Revert 255397 "Fill out the rest of the StringPiece functions fo..."

Revert 255397 "Fill out the rest of the StringPiece functions fo..."

> Fill out the rest of the StringPiece functions for 16-bit.
> 
> I was originally opposed to these since we didn't need them and they were complicated. But I'm wanting to use some of these functions in a different patch, so it seems like a good time to fill out the std::string-like finding functions for StringPiece16.
> 
> This deletes the old StringPieceDetails for which the only point was to share the common stuff between the two BasicStringPiece specializations.
> 
> I used the pattern of having two versions of each function declared in the header and then expanding the template in the .cc file, to avoid template bloat in the header.
> 
> This replaces all of the size_type goop with size_t. Chrome code assumes these are the same and we encourage people to just use size_t in loops, for example, rather than using the size_type of the template they're iterating over. This makes the code more readable in many places. It also solves a problem with declaration ordering since most of the functions that used size_type are now moved above where the size_type is actually declared.
> 
> R=viettrungluu@chromium.org
> TBR=akalin, ben
> 
> Review URL: https://codereview.chromium.org/187793004

TBR=brettw@chromium.org

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=255492

[brettw, 2014-03-07 01:45:04]

Committed patchset #1 manually as r255492 (tree was closed).