Tidy string16.h after https://codereview.chromium.org/809563003/

Tidy string16.h after https://codereview.chromium.org/809563003/

R=mark@chromium.org

Committed: https://chromium.googlesource.com/chromium/mini_chromium/+/e87cd9d4dff52a0bd77f8473002bed48f0f3df2d

[Mark Mentovai, 2014-12-16 18:34:07]

LGTM with this one more refinement, too. Thanks for following up.

https://codereview.chromium.org/812553004/diff/20001/base/strings/string16.h
File base/strings/string16.h (right):

https://codereview.chromium.org/812553004/diff/20001/base/strings/string16.h#...
base/strings/string16.h:19: typedef std::wstring string16;
This is better. The one thing I can think of to improve it would be to move
these definitions of string16 and string16_char_traits down to line 32 or so, so
that string16 is defined for both variants at the same place in the file.

[scottmg, 2014-12-16 18:56:39]

https://codereview.chromium.org/812553004/diff/20001/base/strings/string16.h
File base/strings/string16.h (right):

https://codereview.chromium.org/812553004/diff/20001/base/strings/string16.h#...
base/strings/string16.h:19: typedef std::wstring string16;
On 2014/12/16 18:34:07, Mark Mentovai wrote:
> This is better. The one thing I can think of to improve it would be to move
> these definitions of string16 and string16_char_traits down to line 32 or so,
so
> that string16 is defined for both variants at the same place in the file.

Done.

[scottmg, 2014-12-16 18:57:06]

Committed patchset #3 (id:40001) manually as
e87cd9d4dff52a0bd77f8473002bed48f0f3df2d (presubmit successful).