Extension Error Piping - Blink: WebKit Side

Source/core/page/FormatConsoleMessage.cpp

 /*
  * Copyright (C) 2013 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 11 matching lines @@
  * OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
  * SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
  * LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
  * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
  * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
  * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
 #include "config.h"
-#include "modules/crypto/Algorithm.h"
+#include "core/page/FormatConsoleMessage.h"
 
-#include "modules/crypto/AesCbcParams.h"
-#include "modules/crypto/AesKeyGenParams.h"
-#include "modules/crypto/HmacParams.h"
-#include "modules/crypto/RsaKeyGenParams.h"
-#include "modules/crypto/RsaSsaParams.h"
+#include "core/inspector/ScriptCallFrame.h"
+#include "core/inspector/ScriptCallStack.h"
+#include "wtf/NotFound.h"
 #include "wtf/text/WTFString.h"
 
 namespace WebCore {
 
-PassRefPtr<Algorithm> Algorithm::create(const WebKit::WebCryptoAlgorithm& algorithm)
+// If there is already a callstack in the message (which can happen when we execute internal code with try-catch and error reporting), we don't want to include another.
+static bool containsStackTrace(const String& message)
 {
-    switch (algorithm.paramsType()) {
-    case WebKit::WebCryptoAlgorithmParamsTypeNone:
-        return adoptRef(new Algorithm(algorithm));
-    case WebKit::WebCryptoAlgorithmParamsTypeAesCbcParams:
-        return AesCbcParams::create(algorithm);
-    case WebKit::WebCryptoAlgorithmParamsTypeAesKeyGenParams:
-        return AesKeyGenParams::create(algorithm);
-    case WebKit::WebCryptoAlgorithmParamsTypeHmacParams:
-        return HmacParams::create(algorithm);
-    case WebKit::WebCryptoAlgorithmParamsTypeRsaSsaParams:
-        return RsaSsaParams::create(algorithm);
-    case WebKit::WebCryptoAlgorithmParamsTypeRsaKeyGenParams:
-        return RsaKeyGenParams::create(algorithm);
-    }
-    ASSERT_NOT_REACHED();
-    return 0;
+    return message.find("\n    at ") != WTF::notFound;

[pfeldman, 2013/08/14 09:53:00]

This looks like a very fragile heuristic.

[Devlin, 2013/08/14 17:00:40]

It is fragile, but there's not really a solution (we have no knowledge of
whether or not we tacked on a stack trace, since it's just part of the message
string in the js implementation when we catch the error).  But if we have the
stack trace as a separate string, we'll be sending both stack traces along
anyway, so we don't really need this.  Removed.

 }
 
-Algorithm::Algorithm(const WebKit::WebCryptoAlgorithm& algorithm)
-    : m_algorithm(algorithm)
+String formatConsoleMessage(const String& originalMessage, PassRefPtr<ScriptCallStack> callStack)
 {
-    ScriptWrappable::init(this);
+    String formattedMessage = originalMessage;
+    if (containsStackTrace(formattedMessage))
+        return formattedMessage;
+
+    for (size_t i = 0; i < callStack->size(); ++i) {
+        const ScriptCallFrame& frame = callStack->at(i);
+        formattedMessage.append("\n    at " + (frame.functionName().length() ? frame.functionName() : String("(anonymous function)")) + " (" + frame.sourceURL() + ":" + String::number(frame.lineNumber()) + ":" + String::number(frame.columnNumber()) + ")");
+    }
+
+    return formattedMessage;
 }
 
-String Algorithm::name()
+String formatConsoleMessage(const WTF::String& originalMessage, const WTF::String& url, unsigned lineNumber, unsigned columnNumber)

[pfeldman, 2013/08/14 09:53:00]

Console message already has url, line number and column plumbed. Adding empty
stack based on this information looks redundant.

[Devlin, 2013/08/14 17:00:40]

Mostly it was for completeness purposes - this way, we would know that we report
some extra details if more details are requested.  Also, ConsoleMessage doesn't
have column - just line.  But, assuming no one's using this on minified code
(and they'd be crazy to do so, anyway), column's not really critical most of the
time.  Removed this variant.

 {
-    return m_algorithm.name();
+    String formattedMessage = originalMessage;
+    if (containsStackTrace(formattedMessage))
+        return formattedMessage;
+
+    formattedMessage.append("\n    at (anonymous function) (" + url + ":" + String::number(lineNumber) + ":" + String::number(columnNumber) + ")");
+
+    return formattedMessage;
 }
 
 } // namespace WebCore