Add StringBuilder::appendNumber() and use it
https://bugs.webkit.org/show_bug.cgi?id=96030

Reviewed by Eric Seidel.

Also fix a bunch of append() vs. appendLiteral() issues in the surrounding code.

Source/JavaScriptCore:

* API/JSContextRef.cpp:
(JSContextCreateBacktrace):
* JavaScriptCore.vcproj/JavaScriptCore/JavaScriptCore.def:
* interpreter/Interpreter.h:
(JSC::StackFrame::toString):

Source/WebCore:

* Modules/websockets/WebSocketHandshake.cpp:
(WebCore::hostName):
* bindings/v8/ScriptController.cpp:
(WebCore::resourceString):
* css/FontFeatureValue.cpp:
(WebCore::FontFeatureValue::customCssText):
* dom/Position.cpp:
(WebCore::Position::formatForDebugger):
* dom/Range.cpp:
(WebCore::Range::formatForDebugger):
* html/FormController.cpp:
(WebCore::FormKeyGenerator::formKey):
* html/HTMLAnchorElement.cpp:
(WebCore::appendServerMapMousePosition):
* html/canvas/CanvasRenderingContext2D.cpp:
(WebCore::CanvasRenderingContext2D::font):
* inspector/IdentifiersFactory.cpp:
(WebCore::IdentifiersFactory::addProcessIdPrefixTo):
* page/FrameTree.cpp:
(WebCore::FrameTree::uniqueChildName):
* page/SecurityOrigin.cpp:
(WebCore::SecurityOrigin::toRawString):
* platform/Decimal.cpp:
(WebCore::Decimal::toString):
* platform/network/ProxyServer.cpp:
(WebCore::appendProxyServerString):
* platform/text/TextStream.cpp:
(WebCore::TextStream::operator<<):
* platform/text/TextStream.h:
* rendering/RenderTreeAsText.cpp:
(WebCore::nodePosition):
* xml/XMLErrors.cpp:
(WebCore::XMLErrors::appendErrorMessage):

Source/WTF:

* wtf/DateMath.cpp:
(WTF::makeRFC2822DateString):
* wtf/text/StringBuilder.cpp:
(WTF::StringBuilder::appendNumber):
* wtf/text/StringBuilder.h:
(StringBuilder):


git-svn-id: http://svn.webkit.org/repository/webkit/trunk@128014 268f45cc-cd09-0410-ab3c-d52691b4dbfc
Source/JavaScriptCore: Fixed a few llint C++ interpreter bugs.
https://bugs.webkit.org/show_bug.cgi?id=96127.

Patch by Mark Lam <mark.lam@apple.com> on 2012-09-09
Reviewed by Geoffrey Garen.

* llint/LLIntCLoop.h:
    CLoop::execute()'s bootstrapOpcodeId does not need a default
    value. There is no case when this function is called without
    that parameter being specified.
* llint/LowLevelInterpreter.asm:
    Moved the dispatchAfterCall() call to where it is needed.
    For the C_LOOP back-end, it generates unreachable code.
* llint/LowLevelInterpreter.cpp:
    #include <wtf/Assertions.h> because LLIntAssembly.h needs it.
(JSC):
    Fixed bug in SIGN_BIT32() macro.
    Placate a MSVC warning for t0, and t1 being uninitialized.
(JSC::CLoop::execute):
    The bootstrapOpcodeId arg should always be specified.
    MSVC doesn't like UNUSED_PARAM() for labels. Switch to using
        the new UNUSED_LABEL() macro.
* offlineasm/cloop.rb:
* offlineasm/generate_offset_extractor.rb:
    Resolved a compiler warning found via MSVC.

Source/WTF: Fixed ASSERT() and ASSERT_AT() macros so that they can be used in
comma expressions. Also, added UNUSED_LABEL().
https://bugs.webkit.org/show_bug.cgi?id=96127.

Patch by Mark Lam <mark.lam@apple.com> on 2012-09-09
Reviewed by Geoffrey Garen.

* wtf/Assertions.h:
* wtf/UnusedParam.h: Added UNUSED_LABEL(). Removed an obsolete comment.

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