Removing "using" declarations that import names in the C++ Standard library.(Source/platform/[fonts…

Removing "using" declarations that import names in the C++ Standard library.(Source/platform/[fonts,text])

This is as per the the coding guidelines for blink.
Do not use "using" declarations of any kind to import names
in the standard template library. Directly qualify the names
at the point they're used instead.

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=181130

[shreeramk, 2014-08-28 19:45:14]

Please review.
Thanks!

[jamesr, 2014-08-28 19:57:21]

jamesr@chromium.org changed reviewers:
+ jamesr@chromium.org

[jamesr, 2014-08-28 19:57:21]

lgtm

https://codereview.chromium.org/516953003/diff/1/Source/platform/fonts/Font.cpp
File Source/platform/fonts/Font.cpp (right):

https://codereview.chromium.org/516953003/diff/1/Source/platform/fonts/Font.c...
Source/platform/fonts/Font.cpp:372: return std::make_pair(data, page);
given the number of make_pair calls i think adding

using std::make_pair;

to the top of this .cpp would be clearer

[shreeramk, 2014-08-29 02:34:18]

On 2014/08/28 19:57:21, jamesr wrote:
> lgtm
> 
>
https://codereview.chromium.org/516953003/diff/1/Source/platform/fonts/Font.cpp
> File Source/platform/fonts/Font.cpp (right):
> 
>
https://codereview.chromium.org/516953003/diff/1/Source/platform/fonts/Font.c...
> Source/platform/fonts/Font.cpp:372: return std::make_pair(data, page);
> given the number of make_pair calls i think adding
> 
> using std::make_pair;
> 
> to the top of this .cpp would be clearer

I am getting following Presubmit error while I am trying to submit the cl with
using std::make_pair.

** Presubmit ERRORS **
check-webkit-style failed
  Source/platform/fonts/Font.cpp:45:  Use 'using namespace std;' instead of
'using std::make_pair;'.  [build/using_std] [4]

[jamesr, 2014-08-29 21:11:15]

Hmm, then go back to the std::make_pair<> version.  Looks like the style checker
and style guide both need some love.

[shreeramk, 2014-08-30 07:05:30]

The CQ bit was checked by shreeram.k@samsung.com

[commit-bot: I haz the power, 2014-08-30 07:05:55]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/shreeram.k@samsung.com/516953003/1

[commit-bot: I haz the power, 2014-08-30 08:07:51]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  linux_blink_dbg on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/linux_blink_dbg/builds/2...)

[commit-bot: I haz the power, 2014-08-30 09:09:42]

Committed patchset #1 (id:1) as 181130