Don't allow direct const char* trace value

Source/web/WebViewImpl.cpp

Index: Source/web/WebViewImpl.cpp
diff --git a/Source/web/WebViewImpl.cpp b/Source/web/WebViewImpl.cpp
index c12b34326755f4c749bdbf89cd9d4c9c6ca70930..3b019013042b879345b89b5ec3d55c37d5494a17 100644
--- a/Source/web/WebViewImpl.cpp
+++ b/Source/web/WebViewImpl.cpp
@@ -1861,7 +1861,7 @@ bool WebViewImpl::handleInputEvent(const WebInputEvent& inputEvent)
     }
 
     const AtomicString* inputEventName = inputTypeToName(inputEvent.type);
-    TRACE_EVENT1("input", "WebViewImpl::handleInputEvent", "type", inputEventName ? TRACE_STR_COPY(inputEventName->ascii().data()) : "unknown");

[Xianzhu, 2014/06/17 19:47:13]

Because of the original "const char*" operator of TraceStringWithCopy, the
compiler actually pass a "const char*" parameter to TRACE_EVENT1, causing
inputEventName->ascii().data() not copied.

Change to use inputEventName->ascii() directly so that the compiler will pass
CString instead (for which TRACE_EVENT will copy the parameter).

+    TRACE_EVENT1("input", "WebViewImpl::handleInputEvent", "type", inputEventName ? inputEventName->ascii() : "unknown");
     // If we've started a drag and drop operation, ignore input events until
     // we're done.
     if (m_doingDragAndDrop)