Source/WebCore: Modify log method in V8DOMActivityLogger so that the apiName and
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 6 Mar 2013 22:56:30 +0000 (22:56 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 6 Mar 2013 22:56:30 +0000 (22:56 +0000)
extraInfo arguments are of type const String& instead of const char*.
This change gets rid of some of the memory ownership issues that arise
with char* pointers.
https://bugs.webkit.org/show_bug.cgi?id=111483

Patch by Ankur Taly <ataly@google.com> on 2013-03-06
Reviewed by Adam Barth.

There are no new tests as there is no change in behavior.

* bindings/v8/V8DOMActivityLogger.h:
(WebCore::V8DOMActivityLogger::log):

Source/WebKit/chromium: Modify log method in WebDOMActivityLogger so that the apiName and
extraInfo arguments are of type const WebString& instead of const char*.
This change gets rid of some of the memory ownership issues that arise
with char* pointers.
https://bugs.webkit.org/show_bug.cgi?id=111483

Patch by Ankur Taly <ataly@google.com> on 2013-03-06
Reviewed by Adam Barth.

There are no new tests as there is no change in behavior.

* public/WebDOMActivityLogger.h: Added property svn:eol-style.
(WebKit::WebDOMActivityLogger::log):
* src/WebDOMActivityLogger.cpp: Added property svn:eol-style.
(WebKit::DOMActivityLoggerContainer::log):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@144988 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebCore/ChangeLog
Source/WebCore/bindings/v8/V8DOMActivityLogger.h
Source/WebKit/chromium/ChangeLog
Source/WebKit/chromium/public/WebDOMActivityLogger.h
Source/WebKit/chromium/src/WebDOMActivityLogger.cpp

index daadc3d..d152af2 100644 (file)
@@ -1,3 +1,18 @@
+2013-03-06  Ankur Taly  <ataly@google.com>
+
+        Modify log method in V8DOMActivityLogger so that the apiName and
+        extraInfo arguments are of type const String& instead of const char*.
+        This change gets rid of some of the memory ownership issues that arise
+        with char* pointers.
+        https://bugs.webkit.org/show_bug.cgi?id=111483
+
+        Reviewed by Adam Barth.
+
+        There are no new tests as there is no change in behavior.
+
+        * bindings/v8/V8DOMActivityLogger.h:
+        (WebCore::V8DOMActivityLogger::log):
+
 2013-03-06  Mike Reed  <reed@google.com>
 
         Use SkTypeface API directly, rather than (soon to be private) SkFontHost
index ea7176b..0265876 100644 (file)
@@ -31,7 +31,8 @@
 #ifndef V8DOMActivityLogger_h
 #define V8DOMActivityLogger_h
 
-#include "v8.h"
+#include <v8.h>
+#include <wtf/text/WTFString.h>
 
 namespace WebCore {
 
@@ -39,7 +40,7 @@ class V8DOMActivityLogger {
 public:
     virtual ~V8DOMActivityLogger() { }
 
-    virtual void log(const char* apiName, int argc, const v8::Handle<v8::Value>* argv, const char* extraInfo) { }
+    virtual void log(const String& apiName, int argc, const v8::Handle<v8::Value>* argv, const String& extraInfo) { }
 };
 
 } // namespace WebCore
index cefd1d6..577e569 100644 (file)
@@ -1,3 +1,20 @@
+2013-03-06  Ankur Taly  <ataly@google.com>
+
+        Modify log method in WebDOMActivityLogger so that the apiName and
+        extraInfo arguments are of type const WebString& instead of const char*.
+        This change gets rid of some of the memory ownership issues that arise
+        with char* pointers.
+        https://bugs.webkit.org/show_bug.cgi?id=111483
+
+        Reviewed by Adam Barth.
+
+        There are no new tests as there is no change in behavior.
+
+        * public/WebDOMActivityLogger.h: Added property svn:eol-style.
+        (WebKit::WebDOMActivityLogger::log):
+        * src/WebDOMActivityLogger.cpp: Added property svn:eol-style.
+        (WebKit::DOMActivityLoggerContainer::log):
+
 2013-03-06  Sheriff Bot  <webkit.review.bot@gmail.com>
 
         Unreviewed.  Rolled Chromium DEPS to r186511.  Requested by
index d7e7a2d..7420551 100644 (file)
@@ -32,6 +32,7 @@
 #define WebDOMActivityLogger_h
 
 #include "../../../Platform/chromium/public/WebCommon.h"
+#include "../../../Platform/chromium/public/WebString.h"
 #include <v8.h>
 
 namespace WebKit {
@@ -39,7 +40,7 @@ namespace WebKit {
 class WebDOMActivityLogger {
 public:
     virtual ~WebDOMActivityLogger() { }
-    virtual void log(const char* apiName, int argc, const v8::Handle<v8::Value>* argv, const char* extraInfo) { }
+    virtual void log(const WebString& apiName, int argc, const v8::Handle<v8::Value>* argv, const WebString& extraInfo) { }
 };
 
 // Checks if a logger already exists for the world identified
index e874f71..158a92d 100644 (file)
@@ -34,6 +34,7 @@
 #include "DOMWrapperWorld.h"
 #include "V8DOMActivityLogger.h"
 #include <wtf/PassRefPtr.h>
+#include <wtf/text/WTFString.h>
 
 using namespace WebCore;
 
@@ -46,9 +47,9 @@ public:
     {
     }
 
-    virtual void log(const char* apiName, int argc, const v8::Handle<v8::Value>* argv, const char* extraInfo)
+    virtual void log(const String& apiName, int argc, const v8::Handle<v8::Value>* argv, const String& extraInfo)
     {   
-        m_domActivityLogger->log(apiName, argc, argv, extraInfo); 
+        m_domActivityLogger->log(WebString(apiName), argc, argv, WebString(extraInfo)); 
     }
 
 private: