Call toWebCoreAtomicString() in V8Window::namedSecurityCheckCustom()

Call toWebCoreAtomicString() in V8Window::namedSecurityCheckCustom()

Call toWebCoreAtomicString() in V8Window::namedSecurityCheckCustom() instead
of toWebCoreString(). This avoids uselessly creating a String and then
convert it to an AtomicString since FrameTree::scopedChild() takes an
AtomicString in argument.

R=haraken
BUG=323739
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=162731

[Inactive, 2013-11-27 01:11:41]

https://codereview.chromium.org/90153002/diff/1/Source/bindings/v8/custom/V8W...
File Source/bindings/v8/custom/V8WindowCustom.cpp (right):

https://codereview.chromium.org/90153002/diff/1/Source/bindings/v8/custom/V8W...
Source/bindings/v8/custom/V8WindowCustom.cpp:478: DEFINE_STATIC_LOCAL(const
AtomicString, nameOfProtoProperty, ("__proto__",
AtomicString::ConstructFromLiteral));
Not strictly related but since I was editing this code...

https://codereview.chromium.org/90153002/diff/1/Source/bindings/v8/custom/V8W...
Source/bindings/v8/custom/V8WindowCustom.cpp:480: AtomicString name =
toWebCoreAtomicString(key.As<v8::String>());
Note: This one is safe (cannot throw) as it takes a v8::String as argument. It
is the other one (taking a v8::Value in argument) that we want to get rid of.

[haraken, 2013-11-27 01:14:35]

LGTM

[commit-bot: I haz the power, 2013-11-27 01:17:20]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/ch.dumez@samsung.com/90153002/1

[commit-bot: I haz the power, 2013-11-27 02:59:37]

Change committed as 162731