Quote WebPreferences in namespace webkit_glue.

Quote WebPreferences in namespace webkit_glue.

To avoid name conflict when linking DumpRenderTree which has its own
implementation of the class with the same name.

BUG=127811
TEST=Build all without errors
TBR=kalman@chromium.org,dpapad@chromium.org,ben@chromium.org,battre@chromium.org


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=136707

[tony, 2012-05-11 20:57:18]

It would probably be easier to put the other WebPreferences in a namespace since
it's only used by DRT.  Maybe put it in the DumpRenderTree namespace.

[Xianzhu, 2012-05-11 21:12:05]

On 2012/05/11 20:57:18, tony wrote:
> It would probably be easier to put the other WebPreferences in a namespace
since
> it's only used by DRT.  Maybe put it in the DumpRenderTree namespace.

I thought of this, it seems better to me to keep consistency. About 4/5 of the
source files under webkit/glue are already using webkit_glue namespace and we
already did that on CppBoundClass and CppVariant. Using DumpRenderTree namespace
in only one of the DumpRenderTree source files looks a bit weird.

[tony, 2012-05-11 21:44:04]

On 2012/05/11 21:12:05, Xianzhu wrote:
> On 2012/05/11 20:57:18, tony wrote:
> > It would probably be easier to put the other WebPreferences in a namespace
> since
> > it's only used by DRT.  Maybe put it in the DumpRenderTree namespace.
> 
> I thought of this, it seems better to me to keep consistency. About 4/5 of the
> source files under webkit/glue are already using webkit_glue namespace and we
> already did that on CppBoundClass and CppVariant. Using DumpRenderTree
namespace
> in only one of the DumpRenderTree source files looks a bit weird.

Good point. LGTM.

[commit-bot: I haz the power, 2012-05-11 21:58:54]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/wangxianzhu@chromium.org/10384128/4003

[commit-bot: I haz the power, 2012-05-11 21:59:38]

Presubmit check for 10384128-4003 failed and returned exit status 1.

Running presubmit commit checks ...

** Presubmit Messages **
--tbr was specified, skipping OWNERS check

** Presubmit Warnings **
License must match:
.*? Copyright \(c\) 2012 The Chromium Authors\. All rights reserved\.\n.*? Use
of this source code is governed by a BSD-style license that can be\n.*? found in
the LICENSE file\.(?: \*/)?\n
Found a bad license header in these files:
  webkit/tools/test_shell/test_shell.h

Presubmit checks took 11.1s to calculate.

[commit-bot: I haz the power, 2012-05-11 22:09:52]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/wangxianzhu@chromium.org/10384128/10001

[commit-bot: I haz the power, 2012-05-11 23:31:28]

Change committed as 136707