Unreviewed, rolling out r218373.
authorzalan@apple.com <zalan@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 28 Jun 2017 16:45:14 +0000 (16:45 +0000)
committerzalan@apple.com <zalan@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 28 Jun 2017 16:45:14 +0000 (16:45 +0000)
Output is not right

Reverted changeset:

"Use WTFLogAlways for debug logging so that it shows up in
device system logs"
https://bugs.webkit.org/show_bug.cgi?id=173450
http://trac.webkit.org/changeset/218373

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

35 files changed:
Source/WebCore/ChangeLog
Source/WebCore/Modules/webaudio/AudioContext.cpp
Source/WebCore/Modules/webaudio/AudioNode.cpp
Source/WebCore/bridge/objc/objc_instance.mm
Source/WebCore/bridge/objc/objc_utility.mm
Source/WebCore/css/StyleProperties.cpp
Source/WebCore/dom/DocumentMarkerController.cpp
Source/WebCore/dom/Node.cpp
Source/WebCore/dom/Position.cpp
Source/WebCore/dom/Range.cpp
Source/WebCore/editing/FrameSelection.cpp
Source/WebCore/editing/VisiblePosition.cpp
Source/WebCore/editing/VisibleSelection.cpp
Source/WebCore/history/HistoryItem.cpp
Source/WebCore/html/parser/HTMLFormattingElementList.cpp
Source/WebCore/inspector/DOMPatchSupport.cpp
Source/WebCore/page/scrolling/ScrollingStateTree.cpp
Source/WebCore/platform/audio/ios/AudioDestinationIOS.cpp
Source/WebCore/platform/graphics/GraphicsLayer.cpp
Source/WebCore/platform/graphics/displaylists/DisplayList.cpp
Source/WebCore/platform/ios/wak/WAKView.mm
Source/WebCore/platform/ios/wak/WKUtilities.c
Source/WebCore/platform/ios/wak/WKView.mm
Source/WebCore/platform/text/TextEncodingRegistry.cpp
Source/WebCore/rendering/CounterNode.cpp
Source/WebCore/rendering/InlineBox.cpp
Source/WebCore/rendering/InlineTextBox.cpp
Source/WebCore/rendering/RenderCounter.cpp
Source/WebCore/rendering/RenderLayer.cpp
Source/WebCore/rendering/RenderObject.cpp
Source/WebCore/rendering/SimpleLineLayoutFunctions.cpp
Source/WebCore/rendering/svg/SVGResources.cpp
Source/WebCore/rendering/svg/SVGResourcesCycleSolver.cpp
Source/WebCore/rendering/svg/SVGTextLayoutAttributes.cpp
Source/WebCore/rendering/svg/SVGTextLayoutEngine.cpp

index 88f47e7..63d28d0 100644 (file)
@@ -1,3 +1,16 @@
+2017-06-28  Zalan Bujtas  <zalan@apple.com>
+
+        Unreviewed, rolling out r218373.
+
+        Output is not right
+
+        Reverted changeset:
+
+        "Use WTFLogAlways for debug logging so that it shows up in
+        device system logs"
+        https://bugs.webkit.org/show_bug.cgi?id=173450
+        http://trac.webkit.org/changeset/218373
+
 2017-06-28  Carlos Garcia Campos  <cgarcia@igalia.com>
 
         [GTK] Some web pages disappear immediately after rendering
index 2bd7017..8de2f23 100644 (file)
@@ -190,7 +190,7 @@ void AudioContext::constructCommon()
 AudioContext::~AudioContext()
 {
 #if DEBUG_AUDIONODE_REFERENCES
-    WTFLogAlways("%p: AudioContext::~AudioContext()\n", this);
+    fprintf(stderr, "%p: AudioContext::~AudioContext()\n", this);
 #endif
     ASSERT(!m_isInitialized);
     ASSERT(m_isStopScheduled);
index da8f70b..da59eb1 100644 (file)
@@ -70,7 +70,7 @@ AudioNode::~AudioNode()
     ASSERT(isMainThread());
 #if DEBUG_AUDIONODE_REFERENCES
     --s_nodeCount[nodeType()];
-    WTFLogAlways("%p: %d: AudioNode::~AudioNode() %d %d\n", this, nodeType(), m_normalRefCount.load(), m_connectionRefCount);
+    fprintf(stderr, "%p: %d: AudioNode::~AudioNode() %d %d\n", this, nodeType(), m_normalRefCount.load(), m_connectionRefCount);
 #endif
 }
 
@@ -404,7 +404,7 @@ void AudioNode::ref(RefType refType)
     }
 
 #if DEBUG_AUDIONODE_REFERENCES
-    WTFLogAlways("%p: %d: AudioNode::ref(%d) %d %d\n", this, nodeType(), refType, m_normalRefCount, m_connectionRefCount);
+    fprintf(stderr, "%p: %d: AudioNode::ref(%d) %d %d\n", this, nodeType(), refType, m_normalRefCount, m_connectionRefCount);
 #endif
 
     // See the disabling code in finishDeref() below. This handles the case where a node
@@ -467,7 +467,7 @@ void AudioNode::finishDeref(RefType refType)
     }
     
 #if DEBUG_AUDIONODE_REFERENCES
-    WTFLogAlways("%p: %d: AudioNode::deref(%d) %d %d\n", this, nodeType(), refType, m_normalRefCount, m_connectionRefCount);
+    fprintf(stderr, "%p: %d: AudioNode::deref(%d) %d %d\n", this, nodeType(), refType, m_normalRefCount, m_connectionRefCount);
 #endif
 
     if (!m_connectionRefCount) {
@@ -493,15 +493,15 @@ int AudioNode::s_nodeCount[NodeTypeEnd];
 
 void AudioNode::printNodeCounts()
 {
-    WTFLogAlways("\n\n");
-    WTFLogAlways("===========================\n");
-    WTFLogAlways("AudioNode: reference counts\n");
-    WTFLogAlways("===========================\n");
+    fprintf(stderr, "\n\n");
+    fprintf(stderr, "===========================\n");
+    fprintf(stderr, "AudioNode: reference counts\n");
+    fprintf(stderr, "===========================\n");
 
     for (unsigned i = 0; i < NodeTypeEnd; ++i)
-        WTFLogAlways("%d: %d\n", i, s_nodeCount[i]);
+        fprintf(stderr, "%d: %d\n", i, s_nodeCount[i]);
 
-    WTFLogAlways("===========================\n\n\n");
+    fprintf(stderr, "===========================\n\n\n");
 }
 
 #endif // DEBUG_AUDIONODE_REFERENCES
index 164b694..780c235 100644 (file)
 #import <wtf/NeverDestroyed.h>
 #import <wtf/ThreadSpecific.h>
 
+#ifdef NDEBUG
+#define OBJC_LOG(formatAndArgs...) ((void)0)
+#else
+#define OBJC_LOG(formatAndArgs...) { \
+    fprintf (stderr, "%s:%d -- %s:  ", __FILE__, __LINE__, __FUNCTION__); \
+    fprintf(stderr, formatAndArgs); \
+}
+#endif
+
 using namespace JSC::Bindings;
 using namespace JSC;
 
@@ -293,7 +302,7 @@ JSC::JSValue ObjcInstance::invokeObjcMethod(ExecState* exec, ObjcMethod* method)
                     // Should never get here.  Argument types are filtered (and
                     // the assert above should have fired in the impossible case
                     // of an invalid type anyway).
-                    WTFLogAlways("%s: invalid type (%d)\n", __PRETTY_FUNCTION__, (int)objcValueType);
+                    fprintf(stderr, "%s: invalid type (%d)\n", __PRETTY_FUNCTION__, (int)objcValueType);
                     ASSERT_NOT_REACHED();
             }
         }
index f899fbc..87f6dbe 100644 (file)
@@ -229,7 +229,7 @@ JSValue convertObjcValueToValue(ExecState* exec, void* buffer, ObjcValueType typ
             return jsNumber(*(double*)buffer);
         default:
             // Should never get here. Argument types are filtered.
-            WTFLogAlways("%s: invalid type (%d)\n", __PRETTY_FUNCTION__, (int)type);
+            fprintf(stderr, "%s: invalid type (%d)\n", __PRETTY_FUNCTION__, (int)type);
             ASSERT_NOT_REACHED();
     }
     
index 544c533..903bed2 100644 (file)
@@ -1363,7 +1363,7 @@ COMPILE_ASSERT(sizeof(StyleProperties) == sizeof(SameSizeAsStyleProperties), sty
 #ifndef NDEBUG
 void StyleProperties::showStyle()
 {
-    WTFLogAlways("%s\n", asText().ascii().data());
+    fprintf(stderr, "%s\n", asText().ascii().data());
 }
 #endif
 
index dccc5bd..165a1d2 100644 (file)
@@ -818,14 +818,14 @@ void DocumentMarkerController::clearDescriptionOnMarkersIntersectingRange(Range&
 #if ENABLE(TREE_DEBUGGING)
 void DocumentMarkerController::showMarkers() const
 {
-    WTFLogAlways("%d nodes have markers:\n", m_markers.size());
+    fprintf(stderr, "%d nodes have markers:\n", m_markers.size());
     for (auto& marker : m_markers) {
         Node* node = marker.key.get();
-        WTFLogAlways("%p", node);
+        fprintf(stderr, "%p", node);
         for (auto& documentMarker : *marker.value)
-            WTFLogAlways(" %d:[%d:%d](%d)", documentMarker.type(), documentMarker.startOffset(), documentMarker.endOffset(), documentMarker.isActiveMatch());
+            fprintf(stderr, " %d:[%d:%d](%d)", documentMarker.type(), documentMarker.startOffset(), documentMarker.endOffset(), documentMarker.isActiveMatch());
 
-        WTFLogAlways("\n");
+        fprintf(stderr, "\n");
     }
 }
 #endif
index bc18912..2d69b29 100644 (file)
@@ -1729,12 +1729,12 @@ void Node::showNode(const char* prefix) const
         String value = nodeValue();
         value.replaceWithLiteral('\\', "\\\\");
         value.replaceWithLiteral('\n', "\\n");
-        WTFLogAlways("%s%s\t%p \"%s\"\n", prefix, nodeName().utf8().data(), this, value.utf8().data());
+        fprintf(stderr, "%s%s\t%p \"%s\"\n", prefix, nodeName().utf8().data(), this, value.utf8().data());
     } else {
         StringBuilder attrs;
         appendAttributeDesc(this, attrs, classAttr, " CLASS=");
         appendAttributeDesc(this, attrs, styleAttr, " STYLE=");
-        WTFLogAlways("%s%s\t%p (renderer %p) %s%s%s\n", prefix, nodeName().utf8().data(), this, renderer(), attrs.toString().utf8().data(), needsStyleRecalc() ? " (needs style recalc)" : "", childNeedsStyleRecalc() ? " (child needs style recalc)" : "");
+        fprintf(stderr, "%s%s\t%p (renderer %p) %s%s%s\n", prefix, nodeName().utf8().data(), this, renderer(), attrs.toString().utf8().data(), needsStyleRecalc() ? " (needs style recalc)" : "", childNeedsStyleRecalc() ? " (child needs style recalc)" : "");
     }
 }
 
@@ -1757,13 +1757,13 @@ void Node::showNodePathForThis() const
             int count = 0;
             for (const ShadowRoot* shadowRoot = downcast<ShadowRoot>(node); shadowRoot && shadowRoot != node; shadowRoot = shadowRoot->shadowRoot())
                 ++count;
-            WTFLogAlways("/#shadow-root[%d]", count);
+            fprintf(stderr, "/#shadow-root[%d]", count);
             continue;
         }
 
         switch (node->nodeType()) {
         case ELEMENT_NODE: {
-            WTFLogAlways("/%s", node->nodeName().utf8().data());
+            fprintf(stderr, "/%s", node->nodeName().utf8().data());
 
             const Element& element = downcast<Element>(*node);
             const AtomicString& idattr = element.getIdAttribute();
@@ -1774,39 +1774,39 @@ void Node::showNodePathForThis() const
                     if (previous->nodeName() == node->nodeName())
                         ++count;
                 if (hasIdAttr)
-                    WTFLogAlways("[@id=\"%s\" and position()=%d]", idattr.string().utf8().data(), count);
+                    fprintf(stderr, "[@id=\"%s\" and position()=%d]", idattr.string().utf8().data(), count);
                 else
-                    WTFLogAlways("[%d]", count);
+                    fprintf(stderr, "[%d]", count);
             } else if (hasIdAttr)
-                WTFLogAlways("[@id=\"%s\"]", idattr.string().utf8().data());
+                fprintf(stderr, "[@id=\"%s\"]", idattr.string().utf8().data());
             break;
         }
         case TEXT_NODE:
-            WTFLogAlways("/text()");
+            fprintf(stderr, "/text()");
             break;
         case ATTRIBUTE_NODE:
-            WTFLogAlways("/@%s", node->nodeName().utf8().data());
+            fprintf(stderr, "/@%s", node->nodeName().utf8().data());
             break;
         default:
             break;
         }
     }
-    WTFLogAlways("\n");
+    fprintf(stderr, "\n");
 }
 
 static void traverseTreeAndMark(const String& baseIndent, const Node* rootNode, const Node* markedNode1, const char* markedLabel1, const Node* markedNode2, const char* markedLabel2)
 {
     for (const Node* node = rootNode; node; node = NodeTraversal::next(*node)) {
         if (node == markedNode1)
-            WTFLogAlways("%s", markedLabel1);
+            fprintf(stderr, "%s", markedLabel1);
         if (node == markedNode2)
-            WTFLogAlways("%s", markedLabel2);
+            fprintf(stderr, "%s", markedLabel2);
 
         StringBuilder indent;
         indent.append(baseIndent);
         for (const Node* tmpNode = node; tmpNode && tmpNode != rootNode; tmpNode = tmpNode->parentOrShadowHostNode())
             indent.append('\t');
-        WTFLogAlways("%s", indent.toString().utf8().data());
+        fprintf(stderr, "%s", indent.toString().utf8().data());
         node->showNode();
         indent.append('\t');
         if (!node->isShadowRoot()) {
index a14c8be..b6f5575 100644 (file)
@@ -1398,9 +1398,9 @@ TextDirection Position::primaryDirection() const
 void Position::debugPosition(const char* msg) const
 {
     if (isNull())
-        WTFLogAlways("Position [%s]: null\n", msg);
+        fprintf(stderr, "Position [%s]: null\n", msg);
     else
-        WTFLogAlways("Position [%s]: %s [%p] at %d\n", msg, deprecatedNode()->nodeName().utf8().data(), deprecatedNode(), m_offset);
+        fprintf(stderr, "Position [%s]: %s [%p] at %d\n", msg, deprecatedNode()->nodeName().utf8().data(), deprecatedNode(), m_offset);
 }
 
 void Position::formatForDebugger(char* buffer, unsigned length) const
@@ -1442,7 +1442,7 @@ void Position::showAnchorTypeAndOffset() const
         fputs("after", stderr);
         break;
     }
-    WTFLogAlways(", offset:%d\n", m_offset);
+    fprintf(stderr, ", offset:%d\n", m_offset);
 }
 
 void Position::showTreeForThis() const
index 40df4ac..8e95565 100644 (file)
@@ -1837,7 +1837,7 @@ void showTree(const WebCore::Range* range)
 {
     if (range && range->boundaryPointsValid()) {
         range->startContainer().showTreeAndMark(&range->startContainer(), "S", &range->endContainer(), "E");
-        WTFLogAlways("start offset: %d, end offset: %d\n", range->startOffset(), range->endOffset());
+        fprintf(stderr, "start offset: %d, end offset: %d\n", range->startOffset(), range->endOffset());
     }
 }
 
index 17b9a57..c777b92 100644 (file)
@@ -1754,11 +1754,11 @@ void FrameSelection::debugRenderer(RenderObject* renderer, bool selected) const
 {
     if (is<Element>(*renderer->node())) {
         Element& element = downcast<Element>(*renderer->node());
-        WTFLogAlways("%s%s\n", selected ? "==> " : "    ", element.localName().string().utf8().data());
+        fprintf(stderr, "%s%s\n", selected ? "==> " : "    ", element.localName().string().utf8().data());
     } else if (is<RenderText>(*renderer)) {
         RenderText& textRenderer = downcast<RenderText>(*renderer);
         if (!textRenderer.textLength() || !textRenderer.firstTextBox()) {
-            WTFLogAlways("%s#text (empty)\n", selected ? "==> " : "    ");
+            fprintf(stderr, "%s#text (empty)\n", selected ? "==> " : "    ");
             return;
         }
         
@@ -1800,17 +1800,17 @@ void FrameSelection::debugRenderer(RenderObject* renderer, bool selected) const
             
             show.replace('\n', ' ');
             show.replace('\r', ' ');
-            WTFLogAlways("==> #text : \"%s\" at offset %d\n", show.utf8().data(), pos);
-            WTFLogAlways("           ");
+            fprintf(stderr, "==> #text : \"%s\" at offset %d\n", show.utf8().data(), pos);
+            fprintf(stderr, "           ");
             for (int i = 0; i < caret; i++)
-                WTFLogAlways(" ");
-            WTFLogAlways("^\n");
+                fprintf(stderr, " ");
+            fprintf(stderr, "^\n");
         } else {
             if ((int)text.length() > max)
                 text = text.left(max - 3) + "...";
             else
                 text = text.left(max);
-            WTFLogAlways("    #text : \"%s\"\n", text.utf8().data());
+            fprintf(stderr, "    #text : \"%s\"\n", text.utf8().data());
         }
     }
 }
index fbe52ec..62a5833 100644 (file)
@@ -689,9 +689,9 @@ int VisiblePosition::lineDirectionPointForBlockDirectionNavigation() const
 void VisiblePosition::debugPosition(const char* msg) const
 {
     if (isNull())
-        WTFLogAlways("Position [%s]: null\n", msg);
+        fprintf(stderr, "Position [%s]: null\n", msg);
     else {
-        WTFLogAlways("Position [%s]: %s, ", msg, m_deepPosition.deprecatedNode()->nodeName().utf8().data());
+        fprintf(stderr, "Position [%s]: %s, ", msg, m_deepPosition.deprecatedNode()->nodeName().utf8().data());
         m_deepPosition.showAnchorTypeAndOffset();
     }
 }
index 3248ab1..5901ef3 100644 (file)
@@ -671,21 +671,21 @@ bool VisibleSelection::isInPasswordField() const
 
 void VisibleSelection::debugPosition() const
 {
-    WTFLogAlways("VisibleSelection ===============\n");
+    fprintf(stderr, "VisibleSelection ===============\n");
 
     if (!m_start.anchorNode())
         fputs("pos:   null", stderr);
     else if (m_start == m_end) {
-        WTFLogAlways("pos:   %s ", m_start.anchorNode()->nodeName().utf8().data());
+        fprintf(stderr, "pos:   %s ", m_start.anchorNode()->nodeName().utf8().data());
         m_start.showAnchorTypeAndOffset();
     } else {
-        WTFLogAlways("start: %s ", m_start.anchorNode()->nodeName().utf8().data());
+        fprintf(stderr, "start: %s ", m_start.anchorNode()->nodeName().utf8().data());
         m_start.showAnchorTypeAndOffset();
-        WTFLogAlways("end:   %s ", m_end.anchorNode()->nodeName().utf8().data());
+        fprintf(stderr, "end:   %s ", m_end.anchorNode()->nodeName().utf8().data());
         m_end.showAnchorTypeAndOffset();
     }
 
-    WTFLogAlways("================================\n");
+    fprintf(stderr, "================================\n");
 }
 
 void VisibleSelection::formatForDebugger(char* buffer, unsigned length) const
index 2c07c22..d99c006 100644 (file)
@@ -494,7 +494,7 @@ int HistoryItem::showTreeWithIndent(unsigned indentLevel) const
         prefix.append("  ", 2);
     prefix.append("\0", 1);
 
-    WTFLogAlways("%s+-%s (%p)\n", prefix.data(), m_urlString.utf8().data(), this);
+    fprintf(stderr, "%s+-%s (%p)\n", prefix.data(), m_urlString.utf8().data(), this);
     
     int totalSubItems = 0;
     for (unsigned i = 0; i < m_children.size(); ++i)
index 1a075e8..d35101b 100644 (file)
@@ -202,7 +202,7 @@ void HTMLFormattingElementList::show()
     for (unsigned i = 1; i <= m_entries.size(); ++i) {
         const Entry& entry = m_entries[m_entries.size() - i];
         if (entry.isMarker())
-            WTFLogAlways("marker\n");
+            fprintf(stderr, "marker\n");
         else
             entry.element().showNode();
     }
index 07ff5c7..643fdd7 100644 (file)
@@ -505,9 +505,9 @@ static String nodeName(Node* node)
 
 void DOMPatchSupport::dumpMap(const ResultMap& map, const String& name)
 {
-    WTFLogAlways("\n\n");
+    fprintf(stderr, "\n\n");
     for (size_t i = 0; i < map.size(); ++i)
-        WTFLogAlways("%s[%lu]: %s (%p) - [%lu]\n", name.utf8().data(), i, map[i].first ? nodeName(map[i].first->m_node).utf8().data() : "", map[i].first, map[i].second);
+        fprintf(stderr, "%s[%lu]: %s (%p) - [%lu]\n", name.utf8().data(), i, map[i].first ? nodeName(map[i].first->m_node).utf8().data() : "", map[i].first, map[i].second);
 }
 
 #endif
index a4e56f4..c05c51d 100644 (file)
@@ -272,12 +272,12 @@ void showScrollingStateTree(const WebCore::ScrollingStateTree* tree)
 
     auto rootNode = tree->rootStateNode();
     if (!rootNode) {
-        WTFLogAlways("Scrolling state tree %p with no root node\n", tree);
+        fprintf(stderr, "Scrolling state tree %p with no root node\n", tree);
         return;
     }
 
     String output = rootNode->scrollingStateTreeAsText(WebCore::ScrollingStateTreeAsTextBehaviorDebug);
-    WTFLogAlways("%s\n", output.utf8().data());
+    fprintf(stderr, "%s\n", output.utf8().data());
 }
 
 void showScrollingStateTree(const WebCore::ScrollingStateNode* node)
index 6a0d1d0..074d8ef 100644 (file)
@@ -268,7 +268,7 @@ void AudioDestinationIOS::frameSizeChangedProc(void *inRefCon, AudioUnit, AudioU
     UInt32 bufferSize = 0;
     UInt32 dataSize = sizeof(bufferSize);
     AudioUnitGetProperty(audioOutput->m_outputUnit, kAudioUnitProperty_MaximumFramesPerSlice, kAudioUnitScope_Global, 0, (void*)&bufferSize, &dataSize);
-    WTFLogAlways(">>>> frameSizeChanged = %lu\n", static_cast<unsigned long>(bufferSize));
+    fprintf(stderr, ">>>> frameSizeChanged = %lu\n", static_cast<unsigned long>(bufferSize));
 }
 
 } // namespace WebCore
index 16c67f4..d12193b 100644 (file)
@@ -970,6 +970,6 @@ void showGraphicsLayerTree(const WebCore::GraphicsLayer* layer)
         return;
 
     String output = layer->layerTreeAsText(WebCore::LayerTreeAsTextDebug | WebCore::LayerTreeAsTextIncludeVisibleRects | WebCore::LayerTreeAsTextIncludeTileCaches | WebCore::LayerTreeAsTextIncludeContentLayers);
-    WTFLogAlways("%s\n", output.utf8().data());
+    fprintf(stderr, "%s\n", output.utf8().data());
 }
 #endif
index eb0cd97..cd9c933 100644 (file)
@@ -44,7 +44,7 @@ WTF::CString DisplayList::description() const
 
 void DisplayList::dump() const
 {
-    WTFLogAlways("%s", description().data());
+    fprintf(stderr, "%s", description().data());
 }
 #endif
 
index a024efe..fd2f852 100644 (file)
@@ -211,7 +211,7 @@ static void invalidateGStateCallback(WKViewRef view)
     ASSERT(_viewRef);
     if (_viewRef->isa.classInfo == &WKViewClassInfo)
         return [[[WAKView alloc] _initWithViewRef:_viewRef] autorelease];
-    WKError("unable to create wrapper for %s\n", _viewRef->isa.classInfo->name);
+    WKError ("unable to create wrapper for %s\n", _viewRef->isa.classInfo->name);
     return nil;
 }
 
@@ -542,7 +542,7 @@ static CGInterpolationQuality toCGInterpolationQuality(WebCore::InterpolationQua
 {
     CGContextRef context = WKGetCurrentGraphicsContext();
     if (!context) {
-        WKError("unable to get context for view");
+        WKError ("unable to get context for view");
         return;
     }
 
@@ -557,7 +557,7 @@ static CGInterpolationQuality toCGInterpolationQuality(WebCore::InterpolationQua
 - (void)displayRectIgnoringOpacity:(NSRect)rect inContext:(CGContextRef)context
 {
     if (!context) {
-        WKError("invalid parameter: context must not be NULL");
+        WKError ("invalid parameter: context must not be NULL");
         return;
     }
 
index eca2227..cb04ed3 100644 (file)
@@ -59,7 +59,7 @@ void WKRelease(const void *o)
     WKObjectRef object = (WKObjectRef)(uintptr_t)o;
 
     if (object->referenceCount == 0) {
-        WKError("attempt to release invalid object");
+        WKError ("attempt to release invalid object");
         return;
     }
     
index 2c8b17e..a7c7172 100644 (file)
@@ -117,7 +117,7 @@ WKViewRef WKViewCreateWithFrame (CGRect frame, WKViewContext *context)
 void _WKViewSetViewContext (WKViewRef view, WKViewContext *context)
 {
     if (!view) {
-        WKError("invalid parameter");
+        WKError ("invalid parameter");
         return;
     }
     view->context = context;
@@ -126,7 +126,7 @@ void _WKViewSetViewContext (WKViewRef view, WKViewContext *context)
 CGRect WKViewGetBounds (WKViewRef view)
 {
     if (!view) {
-        WKError("invalid parameter");
+        WKError ("invalid parameter");
         return CGRectZero;
     }
     
@@ -136,7 +136,7 @@ CGRect WKViewGetBounds (WKViewRef view)
 CGRect WKViewGetFrame (WKViewRef view)
 {
     if (!view) {
-        WKError("invalid parameter");
+        WKError ("invalid parameter");
         return CGRectZero;
     }
     
@@ -156,7 +156,7 @@ CGPoint WKViewGetOrigin(WKViewRef view)
 static void _WKViewRecursivelyInvalidateGState(WKViewRef view)
 {
     if (!view) {
-        WKError("invalid parameter");
+        WKError ("invalid parameter");
         return;
     }
 
@@ -173,7 +173,7 @@ static void _WKViewRecursivelyInvalidateGState(WKViewRef view)
 void WKViewSetFrameOrigin (WKViewRef view, CGPoint newOrigin)
 {
     if (!view) {
-        WKError("invalid parameter");
+        WKError ("invalid parameter");
         return;
     }
     
@@ -322,7 +322,7 @@ static void _WKViewAutoresizeChildren(WKViewRef view, const CGRect *oldSuperFram
 void WKViewSetFrameSize (WKViewRef view, CGSize newSize)
 {
     if (!view) {
-        WKError("invalid parameter");
+        WKError ("invalid parameter");
         return;
     }
     
@@ -386,7 +386,7 @@ float WKViewGetScale(WKViewRef view)
 WAKWindow *WKViewGetWindow (WKViewRef view)
 {
     if (!view) {
-        WKError("invalid parameter");
+        WKError ("invalid parameter");
         return 0;
     }
     
@@ -396,7 +396,7 @@ WAKWindow *WKViewGetWindow (WKViewRef view)
 CFArrayRef WKViewGetSubviews (WKViewRef view)
 {
     if (!view) {
-        WKError("invalid parameter");
+        WKError ("invalid parameter");
         return 0;
     }
     
@@ -406,7 +406,7 @@ CFArrayRef WKViewGetSubviews (WKViewRef view)
 void WKViewAddSubview (WKViewRef view, WKViewRef subview)
 {
     if (!view || !subview) {
-        WKError("invalid parameter");
+        WKError ("invalid parameter");
         return;
     }
     
@@ -423,7 +423,7 @@ void WKViewAddSubview (WKViewRef view, WKViewRef subview)
 void WKViewRemoveFromSuperview (WKViewRef view)
 {
     if (!view) {
-        WKError("invalid parameter");
+        WKError ("invalid parameter");
         return;
     }
 
@@ -435,13 +435,13 @@ void WKViewRemoveFromSuperview (WKViewRef view)
     
     CFMutableArrayRef svs = view->superview->subviews;
     if (!svs) {
-        WKError("superview has no subviews");
+        WKError ("superview has no subviews");
         return;
     }
 
     CFIndex index = WKArrayIndexOfValue (svs, view);
     if (index < 0) {
-        WKError("view not in superview subviews");
+        WKError ("view not in superview subviews");
         return;
     }
 
@@ -455,7 +455,7 @@ void WKViewRemoveFromSuperview (WKViewRef view)
 WKViewRef WKViewFirstChild (WKViewRef view)
 {
     if (!view) {
-        WKError("invalid parameter");
+        WKError ("invalid parameter");
         return 0;
     }
 
@@ -474,7 +474,7 @@ WKViewRef WKViewFirstChild (WKViewRef view)
 WKViewRef WKViewNextSibling (WKViewRef view)
 {
     if (!view) {
-        WKError("invalid parameter");
+        WKError ("invalid parameter");
         return 0;
     }
 
@@ -487,7 +487,7 @@ WKViewRef WKViewNextSibling (WKViewRef view)
         
     CFIndex thisIndex = WKArrayIndexOfValue (svs, view);
     if (thisIndex < 0) {
-        WKError("internal error, view is not present in superview subviews");
+        WKError ("internal error, view is not present in superview subviews");
         return 0;
     }
     
@@ -502,7 +502,7 @@ WKViewRef WKViewNextSibling (WKViewRef view)
 WKViewRef WKViewTraverseNext(WKViewRef view)
 {
     if (!view) {
-        WKError("invalid parameter");
+        WKError ("invalid parameter");
         return 0;
     }
 
@@ -538,7 +538,7 @@ CGAffineTransform _WKViewGetTransform(WKViewRef view)
 CGRect WKViewGetVisibleRect(WKViewRef viewRef)
 {
     if (!viewRef) {
-        WKError("invalid parameter");
+        WKError ("invalid parameter");
         return CGRectZero;
     }
 
@@ -564,7 +564,7 @@ CGRect WKViewGetVisibleRect(WKViewRef viewRef)
 CGRect WKViewConvertRectToSuperview(WKViewRef view, CGRect r)
 {
     if (!view) {
-        WKError("invalid parameter");
+        WKError ("invalid parameter");
         return CGRectZero;
     }
     
@@ -574,7 +574,7 @@ CGRect WKViewConvertRectToSuperview(WKViewRef view, CGRect r)
 CGRect WKViewConvertRectToBase(WKViewRef view, CGRect r)
 {
     if (!view) {
-        WKError("invalid parameter");
+        WKError ("invalid parameter");
         return CGRectZero;
     }
 
@@ -591,7 +591,7 @@ CGRect WKViewConvertRectToBase(WKViewRef view, CGRect r)
 CGPoint WKViewConvertPointToSuperview(WKViewRef view, CGPoint p)
 {
     if (!view) {
-        WKError("invalid parameter");
+        WKError ("invalid parameter");
         return CGPointZero;
     }
 
@@ -601,7 +601,7 @@ CGPoint WKViewConvertPointToSuperview(WKViewRef view, CGPoint p)
 CGPoint WKViewConvertPointFromSuperview(WKViewRef view, CGPoint p)
 {
     if (!view) {
-        WKError("invalid parameter");
+        WKError ("invalid parameter");
         return CGPointZero;
     }
     
@@ -612,7 +612,7 @@ CGPoint WKViewConvertPointFromSuperview(WKViewRef view, CGPoint p)
 CGPoint WKViewConvertPointToBase(WKViewRef view, CGPoint p)
 {
     if (!view) {
-        WKError("invalid parameter");
+        WKError ("invalid parameter");
         return CGPointZero;
     }
 
@@ -637,7 +637,7 @@ static void _WKViewGetAncestorViewsIncludingView (WKViewRef view, WKViewRef *vie
     while (superview) {
         views[count++] = superview;
         if (count >= maxViews) {
-            WKError("Exceeded maxViews, use malloc/realloc");
+            WKError ("Exceeded maxViews, use malloc/realloc");
             *viewCount = 0;
             return;
         }
@@ -649,7 +649,7 @@ static void _WKViewGetAncestorViewsIncludingView (WKViewRef view, WKViewRef *vie
 CGPoint WKViewConvertPointFromBase(WKViewRef view, CGPoint p)
 {
     if (!view) {
-        WKError("invalid parameter");
+        WKError ("invalid parameter");
         return CGPointZero;
     }
 
@@ -672,7 +672,7 @@ CGPoint WKViewConvertPointFromBase(WKViewRef view, CGPoint p)
 CGRect WKViewConvertRectFromSuperview(WKViewRef view, CGRect r)
 {
     if (!view) {
-        WKError("invalid parameter");
+        WKError ("invalid parameter");
         return CGRectZero;
     }
     
@@ -683,7 +683,7 @@ CGRect WKViewConvertRectFromSuperview(WKViewRef view, CGRect r)
 CGRect WKViewConvertRectFromBase(WKViewRef view, CGRect r)
 {
     if (!view) {
-        WKError("invalid parameter");
+        WKError ("invalid parameter");
         return CGRectZero;
     }
 
@@ -730,7 +730,7 @@ bool WKViewResignFirstResponder (WKViewRef view)
 unsigned int WKViewGetAutoresizingMask(WKViewRef view)
 {
     if (!view) {
-        WKError("invalid parameter");
+        WKError ("invalid parameter");
         return 0;
     }    
     return view->autoresizingMask;
@@ -739,7 +739,7 @@ unsigned int WKViewGetAutoresizingMask(WKViewRef view)
 void WKViewSetAutoresizingMask (WKViewRef view, unsigned int mask)
 {
     if (!view) {
-        WKError("invalid parameter");
+        WKError ("invalid parameter");
         return;
     }    
     view->autoresizingMask = mask;
index 79ce7ee..031ccc5 100644 (file)
@@ -391,14 +391,14 @@ String defaultTextEncodingNameForSystemLanguage()
 void dumpTextEncodingNameMap()
 {
     unsigned size = textEncodingNameMap->size();
-    WTFLogAlways("Dumping %u entries in WebCore::textEncodingNameMap...\n", size);
+    fprintf(stderr, "Dumping %u entries in WebCore::textEncodingNameMap...\n", size);
 
     std::lock_guard<StaticLock> lock(encodingRegistryMutex);
 
     TextEncodingNameMap::const_iterator it = textEncodingNameMap->begin();
     TextEncodingNameMap::const_iterator end = textEncodingNameMap->end();
     for (; it != end; ++it)
-        WTFLogAlways("'%s' => '%s'\n", it->key, it->value);
+        fprintf(stderr, "'%s' => '%s'\n", it->key, it->value);
 }
 #endif
 
index e4962da..dc0726a 100644 (file)
@@ -336,10 +336,10 @@ static void showTreeAndMark(const CounterNode* node)
         root = root->parent();
 
     for (const CounterNode* current = root; current; current = current->nextInPreOrder()) {
-        WTFLogAlways("%c", (current == node) ? '*' : ' ');
+        fprintf(stderr, "%c", (current == node) ? '*' : ' ');
         for (const CounterNode* parent = current; parent && parent != root; parent = parent->parent())
-            WTFLogAlways("    ");
-        WTFLogAlways("%p %s: %d %d P:%p PS:%p NS:%p R:%p\n",
+            fprintf(stderr, "    ");
+        fprintf(stderr, "%p %s: %d %d P:%p PS:%p NS:%p R:%p\n",
             current, current->actsAsReset() ? "reset____" : "increment", current->value(),
             current->countInParent(), current->parent(), current->previousSibling(),
             current->nextSibling(), &current->owner());
index 11c58ad..d771bfc 100644 (file)
@@ -107,15 +107,15 @@ void InlineBox::showLineTreeAndMark(const InlineBox* markedBox, int depth) const
 
 void InlineBox::showLineBox(bool mark, int depth) const
 {
-    WTFLogAlways("-------- %c-", isDirty() ? 'D' : '-');
+    fprintf(stderr, "-------- %c-", isDirty() ? 'D' : '-');
     int printedCharacters = 0;
     if (mark) {
-        WTFLogAlways("*");
+        fprintf(stderr, "*");
         ++printedCharacters;
     }
     while (++printedCharacters <= depth * 2)
         fputc(' ', stderr);
-    WTFLogAlways("%s  (%.2f, %.2f) (%.2f, %.2f) (%p) renderer->(%p)\n", boxName(), x(), y(), width(), height(), this, &renderer());
+    fprintf(stderr, "%s  (%.2f, %.2f) (%.2f, %.2f) (%p) renderer->(%p)\n", boxName(), x(), y(), width(), height(), this, &renderer());
 }
 
 #endif // ENABLE(TREE_DEBUGGING)
index 7b7d351..a37702a 100644 (file)
@@ -1125,11 +1125,11 @@ const char* InlineTextBox::boxName() const
 
 void InlineTextBox::showLineBox(bool mark, int depth) const
 {
-    WTFLogAlways("-------- %c-", isDirty() ? 'D' : '-');
+    fprintf(stderr, "-------- %c-", isDirty() ? 'D' : '-');
 
     int printedCharacters = 0;
     if (mark) {
-        WTFLogAlways("*");
+        fprintf(stderr, "*");
         ++printedCharacters;
     }
     while (++printedCharacters <= depth * 2)
@@ -1139,7 +1139,7 @@ void InlineTextBox::showLineBox(bool mark, int depth) const
     value = value.substring(start(), len());
     value.replaceWithLiteral('\\', "\\\\");
     value.replaceWithLiteral('\n', "\\n");
-    WTFLogAlways("%s  (%.2f, %.2f) (%.2f, %.2f) (%p) renderer->(%p) run(%d, %d) \"%s\"\n", boxName(), x(), y(), width(), height(), this, &renderer(), start(), start() + len(), value.utf8().data());
+    fprintf(stderr, "%s  (%.2f, %.2f) (%.2f, %.2f) (%p) renderer->(%p) run(%d, %d) \"%s\"\n", boxName(), x(), y(), width(), height(), this, &renderer(), start(), start() + len(), value.utf8().data());
 }
 
 #endif
index 4da1424..5f14673 100644 (file)
@@ -610,10 +610,10 @@ void showCounterRendererTree(const WebCore::RenderObject* renderer, const char*
     for (const WebCore::RenderObject* current = root; current; current = current->nextInPreOrder()) {
         if (!is<WebCore::RenderElement>(*current))
             continue;
-        WTFLogAlways("%c", (current == renderer) ? '*' : ' ');
+        fprintf(stderr, "%c", (current == renderer) ? '*' : ' ');
         for (const WebCore::RenderObject* parent = current; parent && parent != root; parent = parent->parent())
-            WTFLogAlways("    ");
-        WTFLogAlways("%p N:%p P:%p PS:%p NS:%p C:%p\n",
+            fprintf(stderr, "    ");
+        fprintf(stderr, "%p N:%p P:%p PS:%p NS:%p C:%p\n",
             current, current->node(), current->parent(), current->previousSibling(),
             current->nextSibling(), downcast<WebCore::RenderElement>(*current).hasCounterNodeMap() ?
             counterName ? WebCore::counterMaps().get(downcast<WebCore::RenderElement>(current))->get(identifier) : (WebCore::CounterNode*)1 : (WebCore::CounterNode*)0);
index 7a0ebaa..51bd194 100644 (file)
@@ -7232,7 +7232,7 @@ void showLayerTree(const WebCore::RenderLayer* layer)
         return;
 
     WTF::String output = externalRepresentation(&layer->renderer().frame(), WebCore::RenderAsTextShowAllLayers | WebCore::RenderAsTextShowLayerNesting | WebCore::RenderAsTextShowCompositedLayers | WebCore::RenderAsTextShowAddresses | WebCore::RenderAsTextShowIDAndClass | WebCore::RenderAsTextDontUpdateLayout | WebCore::RenderAsTextShowLayoutState | WebCore::RenderAsTextShowOverflow | WebCore::RenderAsTextShowSVGGeometry | WebCore::RenderAsTextShowLayerFragments);
-    WTFLogAlways("\n%s\n", output.utf8().data());
+    fprintf(stderr, "\n%s\n", output.utf8().data());
 }
 
 void showLayerTree(const WebCore::RenderObject* renderer)
index 0d2c5bc..269ac97 100644 (file)
@@ -1009,7 +1009,7 @@ FloatRect RenderObject::computeFloatRectForRepaint(const FloatRect&, const Rende
 
 static void showRenderTreeLegend()
 {
-    WTFLogAlways("\n(B)lock/(I)nline/I(N)line-block, (A)bsolute/Fi(X)ed/(R)elative/Stic(K)y, (F)loating, (O)verflow clip, Anon(Y)mous, (G)enerated, has(L)ayer, (C)omposited, (+)Dirty style, (+)Dirty layout\n");
+    fprintf(stderr, "\n(B)lock/(I)nline/I(N)line-block, (A)bsolute/Fi(X)ed/(R)elative/Stic(K)y, (F)loating, (O)verflow clip, Anon(Y)mous, (G)enerated, has(L)ayer, (C)omposited, (+)Dirty style, (+)Dirty layout\n");
 }
 
 void RenderObject::showNodeTreeForThis() const
@@ -1072,7 +1072,7 @@ void RenderObject::showRegionsInformation() const
     RenderRegion* startRegion = nullptr;
     RenderRegion* endRegion = nullptr;
     ftcb->getRegionRangeForBox(downcast<RenderBox>(this), startRegion, endRegion);
-    WTFLogAlways(" [Rs:%p Re:%p]", startRegion, endRegion);
+    fprintf(stderr, " [Rs:%p Re:%p]", startRegion, endRegion);
 }
 
 void RenderObject::showRenderObject(bool mark, int depth) const
@@ -1142,7 +1142,7 @@ void RenderObject::showRenderObject(bool mark, int depth) const
 
     int printedCharacters = 0;
     if (mark) {
-        WTFLogAlways("*");
+        fprintf(stderr, "*");
         ++printedCharacters;
     }
 
@@ -1150,33 +1150,33 @@ void RenderObject::showRenderObject(bool mark, int depth) const
         fputc(' ', stderr);
 
     if (node())
-        WTFLogAlways("%s ", node()->nodeName().utf8().data());
+        fprintf(stderr, "%s ", node()->nodeName().utf8().data());
 
     String name = renderName();
     // FIXME: Renderer's name should not include property value listing.
     int pos = name.find('(');
     if (pos > 0)
-        WTFLogAlways("%s", name.left(pos - 1).utf8().data());
+        fprintf(stderr, "%s", name.left(pos - 1).utf8().data());
     else
-        WTFLogAlways("%s", name.utf8().data());
+        fprintf(stderr, "%s", name.utf8().data());
 
     if (is<RenderBox>(*this)) {
         auto& renderBox = downcast<RenderBox>(*this);
         FloatRect boxRect = renderBox.frameRect();
         if (renderBox.isInFlowPositioned())
             boxRect.move(renderBox.offsetForInFlowPosition());
-        WTFLogAlways("  (%.2f, %.2f) (%.2f, %.2f)", boxRect.x(), boxRect.y(), boxRect.width(), boxRect.height());
+        fprintf(stderr, "  (%.2f, %.2f) (%.2f, %.2f)", boxRect.x(), boxRect.y(), boxRect.width(), boxRect.height());
     } else if (is<RenderInline>(*this) && isInFlowPositioned()) {
         FloatSize inlineOffset = downcast<RenderInline>(*this).offsetForInFlowPosition();
-        WTFLogAlways("  (%.2f, %.2f)", inlineOffset.width(), inlineOffset.height());
+        fprintf(stderr, "  (%.2f, %.2f)", inlineOffset.width(), inlineOffset.height());
     }
 
-    WTFLogAlways(" renderer->(%p)", this);
+    fprintf(stderr, " renderer->(%p)", this);
     if (node()) {
-        WTFLogAlways(" node->(%p)", node());
+        fprintf(stderr, " node->(%p)", node());
         if (node()->isTextNode()) {
             String value = node()->nodeValue();
-            WTFLogAlways(" length->(%u)", value.length());
+            fprintf(stderr, " length->(%u)", value.length());
 
             value.replaceWithLiteral('\\', "\\\\");
             value.replaceWithLiteral('\n', "\\n");
@@ -1184,31 +1184,31 @@ void RenderObject::showRenderObject(bool mark, int depth) const
             const int maxPrintedLength = 80;
             if (value.length() > maxPrintedLength) {
                 String substring = value.substring(0, maxPrintedLength);
-                WTFLogAlways(" \"%s\"...", substring.utf8().data());
+                fprintf(stderr, " \"%s\"...", substring.utf8().data());
             } else
-                WTFLogAlways(" \"%s\"", value.utf8().data());
+                fprintf(stderr, " \"%s\"", value.utf8().data());
         }
     }
     if (is<RenderBoxModelObject>(*this)) {
         auto& renderer = downcast<RenderBoxModelObject>(*this);
         if (renderer.hasContinuation())
-            WTFLogAlways(" continuation->(%p)", renderer.continuation());
+            fprintf(stderr, " continuation->(%p)", renderer.continuation());
     }
     showRegionsInformation();
     if (needsLayout()) {
-        WTFLogAlways(" layout->");
+        fprintf(stderr, " layout->");
         if (selfNeedsLayout())
-            WTFLogAlways("[self]");
+            fprintf(stderr, "[self]");
         if (normalChildNeedsLayout())
-            WTFLogAlways("[normal child]");
+            fprintf(stderr, "[normal child]");
         if (posChildNeedsLayout())
-            WTFLogAlways("[positioned child]");
+            fprintf(stderr, "[positioned child]");
         if (needsSimplifiedNormalFlowLayout())
-            WTFLogAlways("[simplified]");
+            fprintf(stderr, "[simplified]");
         if (needsPositionedMovementLayout())
-            WTFLogAlways("[positioned movement]");
+            fprintf(stderr, "[positioned movement]");
     }
-    WTFLogAlways("\n");
+    fprintf(stderr, "\n");
 }
 
 void RenderObject::showRenderSubTreeAndMark(const RenderObject* markedObject, int depth) const
@@ -2017,8 +2017,8 @@ void printRenderTreeForLiveDocuments()
         if (!document->renderView())
             continue;
         if (document->frame() && document->frame()->isMainFrame())
-            WTFLogAlways("----------------------main frame--------------------------\n");
-        WTFLogAlways("%s", document->url().string().utf8().data());
+            fprintf(stderr, "----------------------main frame--------------------------\n");
+        fprintf(stderr, "%s", document->url().string().utf8().data());
         showRenderTree(document->renderView());
     }
 }
@@ -2029,8 +2029,8 @@ void printLayerTreeForLiveDocuments()
         if (!document->renderView())
             continue;
         if (document->frame() && document->frame()->isMainFrame())
-            WTFLogAlways("----------------------main frame--------------------------\n");
-        WTFLogAlways("%s", document->url().string().utf8().data());
+            fprintf(stderr, "----------------------main frame--------------------------\n");
+        fprintf(stderr, "%s", document->url().string().utf8().data());
         showLayerTree(document->renderView());
     }
 }
index 590a6c1..83b12f1 100644 (file)
@@ -263,7 +263,7 @@ const RenderObject& rendererForPosition(const FlowContents& flowContents, unsign
 #if ENABLE(TREE_DEBUGGING)
 static void printPrefix(int& printedCharacters, int depth)
 {
-    WTFLogAlways("-------- --");
+    fprintf(stderr, "-------- --");
     printedCharacters = 0;
     while (++printedCharacters <= depth * 2)
         fputc(' ', stderr);
@@ -274,18 +274,18 @@ void showLineLayoutForFlow(const RenderBlockFlow& flow, const Layout& layout, in
     int printedCharacters = 0;
     printPrefix(printedCharacters, depth);
 
-    WTFLogAlways("SimpleLineLayout (%u lines, %u runs) (%p)\n", layout.lineCount(), layout.runCount(), &layout);
+    fprintf(stderr, "SimpleLineLayout (%u lines, %u runs) (%p)\n", layout.lineCount(), layout.runCount(), &layout);
     ++depth;
 
     for (auto run : runResolver(flow, layout)) {
         FloatRect rect = run.rect();
         printPrefix(printedCharacters, depth);
         if (run.start() < run.end()) {
-            WTFLogAlways("line %u run(%u, %u) (%.2f, %.2f) (%.2f, %.2f) \"%s\"\n", run.lineIndex(), run.start(), run.end(),
+            fprintf(stderr, "line %u run(%u, %u) (%.2f, %.2f) (%.2f, %.2f) \"%s\"\n", run.lineIndex(), run.start(), run.end(),
                 rect.x(), rect.y(), rect.width(), rect.height(), run.text().toStringWithoutCopying().utf8().data());
         } else {
             ASSERT(run.start() == run.end());
-            WTFLogAlways("line break %u run(%u, %u) (%.2f, %.2f) (%.2f, %.2f)\n", run.lineIndex(), run.start(), run.end(), rect.x(), rect.y(), rect.width(), rect.height());
+            fprintf(stderr, "line break %u run(%u, %u) (%.2f, %.2f) (%.2f, %.2f)\n", run.lineIndex(), run.start(), run.end(), rect.x(), rect.y(), rect.width(), rect.height());
         }
     }
 }
index 9163944..ea9b452 100644 (file)
@@ -656,38 +656,38 @@ void SVGResources::dump(const RenderObject* object)
     ASSERT(object);
     ASSERT(object->node());
 
-    WTFLogAlways("-> this=%p, SVGResources(renderer=%p, node=%p)\n", this, object, object->node());
-    WTFLogAlways(" | DOM Tree:\n");
+    fprintf(stderr, "-> this=%p, SVGResources(renderer=%p, node=%p)\n", this, object, object->node());
+    fprintf(stderr, " | DOM Tree:\n");
     object->node()->showTreeForThis();
 
-    WTFLogAlways("\n | List of resources:\n");
+    fprintf(stderr, "\n | List of resources:\n");
     if (m_clipperFilterMaskerData) {
         if (RenderSVGResourceClipper* clipper = m_clipperFilterMaskerData->clipper)
-            WTFLogAlways(" |-> Clipper    : %p (node=%p)\n", clipper, &clipper->clipPathElement());
+            fprintf(stderr, " |-> Clipper    : %p (node=%p)\n", clipper, &clipper->clipPathElement());
         if (RenderSVGResourceFilter* filter = m_clipperFilterMaskerData->filter)
-            WTFLogAlways(" |-> Filter     : %p (node=%p)\n", filter, &filter->filterElement());
+            fprintf(stderr, " |-> Filter     : %p (node=%p)\n", filter, &filter->filterElement());
         if (RenderSVGResourceMasker* masker = m_clipperFilterMaskerData->masker)
-            WTFLogAlways(" |-> Masker     : %p (node=%p)\n", masker, &masker->maskElement());
+            fprintf(stderr, " |-> Masker     : %p (node=%p)\n", masker, &masker->maskElement());
     }
 
     if (m_markerData) {
         if (RenderSVGResourceMarker* markerStart = m_markerData->markerStart)
-            WTFLogAlways(" |-> MarkerStart: %p (node=%p)\n", markerStart, &markerStart->markerElement());
+            fprintf(stderr, " |-> MarkerStart: %p (node=%p)\n", markerStart, &markerStart->markerElement());
         if (RenderSVGResourceMarker* markerMid = m_markerData->markerMid)
-            WTFLogAlways(" |-> MarkerMid  : %p (node=%p)\n", markerMid, &markerMid->markerElement());
+            fprintf(stderr, " |-> MarkerMid  : %p (node=%p)\n", markerMid, &markerMid->markerElement());
         if (RenderSVGResourceMarker* markerEnd = m_markerData->markerEnd)
-            WTFLogAlways(" |-> MarkerEnd  : %p (node=%p)\n", markerEnd, &markerEnd->markerElement());
+            fprintf(stderr, " |-> MarkerEnd  : %p (node=%p)\n", markerEnd, &markerEnd->markerElement());
     }
 
     if (m_fillStrokeData) {
         if (RenderSVGResourceContainer* fill = m_fillStrokeData->fill)
-            WTFLogAlways(" |-> Fill       : %p (node=%p)\n", fill, &fill->element());
+            fprintf(stderr, " |-> Fill       : %p (node=%p)\n", fill, &fill->element());
         if (RenderSVGResourceContainer* stroke = m_fillStrokeData->stroke)
-            WTFLogAlways(" |-> Stroke     : %p (node=%p)\n", stroke, &stroke->element());
+            fprintf(stderr, " |-> Stroke     : %p (node=%p)\n", stroke, &stroke->element());
     }
 
     if (m_linkedResource)
-        WTFLogAlways(" |-> xlink:href : %p (node=%p)\n", m_linkedResource, &m_linkedResource->element());
+        fprintf(stderr, " |-> xlink:href : %p (node=%p)\n", m_linkedResource, &m_linkedResource->element());
 }
 #endif
 
index 877b2a3..be76c1f 100644 (file)
@@ -142,7 +142,7 @@ void SVGResourcesCycleSolver::resolveCycles()
         for (RenderObject* resource : localResources)
             LOG_DEBUG_CYCLE("|> %s : %p (node %p)\n", resource->renderName(), resource, resource->node());
 
-        WTFLogAlways("Parent resources:\n");
+        fprintf(stderr, "Parent resources:\n");
         for (RenderObject* resource : ancestorResources)
             LOG_DEBUG_CYCLE("|> %s : %p (node %p)\n", resource->renderName(), resource, resource->node());
     }
index 15ad2eb..6441272 100644 (file)
@@ -45,29 +45,29 @@ float SVGTextLayoutAttributes::emptyValue()
 static inline void dumpSVGCharacterDataMapValue(const char* identifier, float value, bool appendSpace = true)
 {
     if (value == SVGTextLayoutAttributes::emptyValue()) {
-        WTFLogAlways("%s=x", identifier);
+        fprintf(stderr, "%s=x", identifier);
         if (appendSpace)
-            WTFLogAlways(" ");
+            fprintf(stderr, " ");
         return;
     }
-    WTFLogAlways("%s=%lf", identifier, value);
+    fprintf(stderr, "%s=%lf", identifier, value);
     if (appendSpace)
-        WTFLogAlways(" ");
+        fprintf(stderr, " ");
 }
 
 void SVGTextLayoutAttributes::dump() const
 {
-    WTFLogAlways("context: %p\n", &m_context);
+    fprintf(stderr, "context: %p\n", &m_context);
     const SVGCharacterDataMap::const_iterator end = m_characterDataMap.end();
     for (SVGCharacterDataMap::const_iterator it = m_characterDataMap.begin(); it != end; ++it) {
         const SVGCharacterData& data = it->value;
-        WTFLogAlways(" ---> pos=%i, data={", it->key);
+        fprintf(stderr, " ---> pos=%i, data={", it->key);
         dumpSVGCharacterDataMapValue("x", data.x);
         dumpSVGCharacterDataMapValue("y", data.y);
         dumpSVGCharacterDataMapValue("dx", data.dx);
         dumpSVGCharacterDataMapValue("dy", data.dy);
         dumpSVGCharacterDataMapValue("rotate", data.rotate, false);
-        WTFLogAlways("}\n");
+        fprintf(stderr, "}\n");
     }
 }
 
index 1f4c7d1..660e86a 100644 (file)
@@ -236,14 +236,14 @@ void SVGTextLayoutEngine::layoutInlineTextBox(SVGInlineTextBox& textBox)
 static inline void dumpTextBoxes(Vector<SVGInlineTextBox*>& boxes)
 {
     unsigned boxCount = boxes.size();
-    WTFLogAlways("Dumping all text fragments in text sub tree, %i boxes\n", boxCount);
+    fprintf(stderr, "Dumping all text fragments in text sub tree, %i boxes\n", boxCount);
 
     for (unsigned boxPosition = 0; boxPosition < boxCount; ++boxPosition) {
         SVGInlineTextBox* textBox = boxes.at(boxPosition);
         Vector<SVGTextFragment>& fragments = textBox->textFragments();
-        WTFLogAlways("-> Box %i: Dumping text fragments for SVGInlineTextBox, textBox=%p, textRenderer=%p\n", boxPosition, textBox, textBox->renderer());
-        WTFLogAlways("        textBox properties, start=%i, len=%i, box direction=%i\n", textBox->start(), textBox->len(), textBox->direction());
-        WTFLogAlways("   textRenderer properties, textLength=%i\n", textBox->renderer()->textLength());
+        fprintf(stderr, "-> Box %i: Dumping text fragments for SVGInlineTextBox, textBox=%p, textRenderer=%p\n", boxPosition, textBox, textBox->renderer());
+        fprintf(stderr, "        textBox properties, start=%i, len=%i, box direction=%i\n", textBox->start(), textBox->len(), textBox->direction());
+        fprintf(stderr, "   textRenderer properties, textLength=%i\n", textBox->renderer()->textLength());
 
         const UChar* characters = textBox->renderer()->characters();
 
@@ -251,7 +251,7 @@ static inline void dumpTextBoxes(Vector<SVGInlineTextBox*>& boxes)
         for (unsigned i = 0; i < fragmentCount; ++i) {
             SVGTextFragment& fragment = fragments.at(i);
             String fragmentString(characters + fragment.characterOffset, fragment.length);
-            WTFLogAlways("    -> Fragment %i, x=%lf, y=%lf, width=%lf, height=%lf, characterOffset=%i, length=%i, characters='%s'\n"
+            fprintf(stderr, "    -> Fragment %i, x=%lf, y=%lf, width=%lf, height=%lf, characterOffset=%i, length=%i, characters='%s'\n"
                           , i, fragment.x, fragment.y, fragment.width, fragment.height, fragment.characterOffset, fragment.length, fragmentString.utf8().data());
         }
     }
@@ -291,7 +291,7 @@ void SVGTextLayoutEngine::finishLayout()
     // Finalize transform matrices, after the chunk layout corrections have been applied, and all fragment x/y positions are finalized.
     if (!m_lineLayoutBoxes.isEmpty()) {
 #if DUMP_TEXT_FRAGMENTS > 0
-        WTFLogAlways("Line layout: ");
+        fprintf(stderr, "Line layout: ");
         dumpTextBoxes(m_lineLayoutBoxes);
 #endif
 
@@ -300,7 +300,7 @@ void SVGTextLayoutEngine::finishLayout()
 
     if (!m_pathLayoutBoxes.isEmpty()) {
 #if DUMP_TEXT_FRAGMENTS > 0
-        WTFLogAlways("Path layout: ");
+        fprintf(stderr, "Path layout: ");
         dumpTextBoxes(m_pathLayoutBoxes);
 #endif