Cleanup namespace usage in platform/fonts/.

Cleanup namespace usage in platform/fonts/.

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

[kangil_, 2014-08-13 01:33:10]

Some changes in this CL added comments to show that this is end block of
'namespace blink'. I am clearly ok to add that comment in which 'blink::'
clean-up is already happened. But, I am not sure for others. IMO, we should ask
owner for the direction of this clean-up.

@haraken: Would you give feedback? Thanks!

https://codereview.chromium.org/468443002/diff/1/Source/platform/fonts/FontCa...
File Source/platform/fonts/FontCache.cpp (left):

https://codereview.chromium.org/468443002/diff/1/Source/platform/fonts/FontCa...
Source/platform/fonts/FontCache.cpp:53: using namespace WTF;
Name space WTF is alive. I don't think we can delete this.

https://codereview.chromium.org/468443002/diff/1/Source/platform/fonts/FontCa...
Source/platform/fonts/FontCache.cpp:117: typedef HashMap<FontCache::FontFileKey,
RefPtr<OpenTypeVerticalData>, WTF::IntHash<FontCache::FontFileKey>,
WTF::UnsignedWithZeroKeyHashTraits<FontCache::FontFileKey> >
FontVerticalDataCache;
Rather IMO we can delete WTF:: here.

https://codereview.chromium.org/468443002/diff/1/Source/platform/fonts/FontDa...
File Source/platform/fonts/FontDataCache.cpp (left):

https://codereview.chromium.org/468443002/diff/1/Source/platform/fonts/FontDa...
Source/platform/fonts/FontDataCache.cpp:36: using namespace WTF;
I don't think we can delete this line.

[haraken, 2014-08-13 01:35:20]

LGTM

[heeyoun.lee, 2014-08-13 02:57:27]

https://codereview.chromium.org/468443002/diff/1/Source/platform/fonts/FontCa...
File Source/platform/fonts/FontCache.cpp (left):

https://codereview.chromium.org/468443002/diff/1/Source/platform/fonts/FontCa...
Source/platform/fonts/FontCache.cpp:53: using namespace WTF;
On 2014/08/13 01:33:10, kangil_ wrote:
> Name space WTF is alive. I don't think we can delete this.

Done.

https://codereview.chromium.org/468443002/diff/1/Source/platform/fonts/FontCa...
Source/platform/fonts/FontCache.cpp:117: typedef HashMap<FontCache::FontFileKey,
RefPtr<OpenTypeVerticalData>, WTF::IntHash<FontCache::FontFileKey>,
WTF::UnsignedWithZeroKeyHashTraits<FontCache::FontFileKey> >
FontVerticalDataCache;
On 2014/08/13 01:33:10, kangil_ wrote:
> Rather IMO we can delete WTF:: here.

Done.

https://codereview.chromium.org/468443002/diff/1/Source/platform/fonts/FontDa...
File Source/platform/fonts/FontDataCache.cpp (left):

https://codereview.chromium.org/468443002/diff/1/Source/platform/fonts/FontDa...
Source/platform/fonts/FontDataCache.cpp:36: using namespace WTF;
On 2014/08/13 01:33:10, kangil_ wrote:
> I don't think we can delete this line.

Done.

[heeyoun.lee, 2014-08-13 04:41:50]

The CQ bit was checked by heeyoun.lee@samsung.com

[commit-bot: I haz the power, 2014-08-13 04:42:32]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/heeyoun.lee@samsung.com/468443002/20001

[commit-bot: I haz the power, 2014-08-13 05:44:08]

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

[commit-bot: I haz the power, 2014-08-13 05:47:15]

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

[commit-bot: I haz the power, 2014-08-13 05:47:16]

Try jobs failed on following builders:
  blink_presubmit on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/blink_presubmit/builds/1...)

[heeyoun.lee, 2014-08-13 06:13:22]

PTAL

[tkent, 2014-08-14 00:42:32]

The CQ bit was checked by tkent@chromium.org

[tkent, 2014-08-14 00:42:33]

lgtm

[commit-bot: I haz the power, 2014-08-14 00:43:19]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/heeyoun.lee@samsung.com/468443002/20001

[commit-bot: I haz the power, 2014-08-14 03:14:48]

Committed patchset #2 (20001) as 180224