Reviewed by Kevin McCulough.
authorggaren <ggaren@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 26 Mar 2007 20:08:53 +0000 (20:08 +0000)
committerggaren <ggaren@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 26 Mar 2007 20:08:53 +0000 (20:08 +0000)
        Global replace of "assert(" with "ASSERT(" to assist debugging. Removed
        one instance of commented-out code containing "assert(" instead of replacing it.

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

56 files changed:
WebCore/ChangeLog
WebCore/bindings/js/kjs_css.cpp
WebCore/bindings/js/kjs_dom.cpp
WebCore/bindings/js/kjs_events.cpp
WebCore/bindings/js/kjs_html.cpp
WebCore/bindings/js/kjs_navigator.cpp
WebCore/bindings/js/kjs_proxy.cpp
WebCore/bindings/js/kjs_window.cpp
WebCore/bindings/objc/WebScriptObject.mm
WebCore/css/cssparser.cpp
WebCore/css/cssstyleselector.cpp
WebCore/dom/Attr.cpp
WebCore/dom/ContainerNode.cpp
WebCore/dom/Document.cpp
WebCore/dom/Element.cpp
WebCore/dom/NamedAttrMap.h
WebCore/dom/Node.cpp
WebCore/dom/Range.cpp
WebCore/dom/StyledElement.cpp
WebCore/dom/XMLTokenizer.cpp
WebCore/editing/Editor.cpp
WebCore/editing/InsertTextCommand.cpp
WebCore/editing/ReplaceSelectionCommand.cpp
WebCore/editing/TextIterator.cpp
WebCore/editing/VisiblePosition.h
WebCore/editing/htmlediting.cpp
WebCore/editing/markup.cpp
WebCore/editing/visible_units.cpp
WebCore/html/HTMLAppletElement.cpp
WebCore/html/HTMLCollection.cpp
WebCore/html/HTMLDocument.cpp
WebCore/html/HTMLElement.cpp
WebCore/html/HTMLEmbedElement.cpp
WebCore/html/HTMLGenericFormElement.cpp
WebCore/html/HTMLInputElement.cpp
WebCore/html/HTMLLIElement.cpp
WebCore/html/HTMLObjectElement.cpp
WebCore/html/HTMLScriptElement.cpp
WebCore/html/HTMLTableElement.cpp
WebCore/loader/CachedXBLDocument.cpp
WebCore/page/Frame.cpp
WebCore/platform/Arena.cpp
WebCore/platform/Arena.h
WebCore/platform/Font.cpp
WebCore/platform/FontFallbackList.cpp
WebCore/platform/KURL.cpp
WebCore/platform/SegmentedString.cpp
WebCore/platform/SegmentedString.h
WebCore/rendering/DataRef.h
WebCore/rendering/InlineBox.cpp
WebCore/rendering/InlineTextBox.cpp
WebCore/rendering/RenderArena.cpp
WebCore/rendering/RenderImage.cpp
WebCore/rendering/RenderLayer.cpp
WebCore/rendering/RenderView.cpp
WebCore/rendering/bidi.cpp

index 67aaf0fa3fc2d31b07f51b75c567c3cc24ea2794..0f10288f0ef7171a38c02229487fb794675a7403 100644 (file)
@@ -1,3 +1,10 @@
+2007-03-26  Geoffrey Garen  <ggaren@apple.com>
+
+        Reviewed by Kevin McCulough.
+
+        Global replace of "assert(" with "ASSERT(" to assist debugging. Removed
+        one instance of commented-out code containing "assert(" instead of replacing it.
+
 2007-03-25  Antti Koivisto  <antti@apple.com>
 
         Reviewed by Adam.
index d21f9b7db395fcb5a96166e85c7361bb7a5c3dcc..f9fa87a1d1506e6b27cd02afa8fa7f417cbc3442 100644 (file)
@@ -201,7 +201,7 @@ JSValue* DOMCSSStyleDeclaration::getValueProperty(ExecState* exec, int token)
   case ParentRule:
     return toJS(exec, m_impl->parentRule());
   default:
-    assert(0);
+    ASSERT(0);
     return jsUndefined();
   }
 }
@@ -384,7 +384,7 @@ JSValue* DOMStyleSheetList::getValueProperty(ExecState* exec, int token) const
     case Length:
       return jsNumber(m_impl->length());
     default:
-      assert(0);
+      ASSERT(0);
       return jsUndefined();
     }
 }
@@ -505,7 +505,7 @@ JSValue* DOMMediaList::getValueProperty(ExecState* exec, int token)
   case Length:
     return jsNumber(m_impl->length());
   default:
-    assert(0);
+    ASSERT(0);
     return jsUndefined();
   }
 }
@@ -612,7 +612,7 @@ JSValue* DOMCSSStyleSheet::getValueProperty(ExecState* exec, int token) const
   case Rules:
     return toJS(exec, static_cast<CSSStyleSheet*>(impl())->cssRules(true));
   default:
-    assert(0);
+    ASSERT(0);
     return jsUndefined();
   }
 }
@@ -791,7 +791,7 @@ JSValue* DOMCSSRule::getValueProperty(ExecState* exec, int token) const
     return jsStringOrNull(static_cast<CSSCharsetRule*>(m_impl.get())->encoding());
 
   default:
-    assert(0);
+    ASSERT(0);
   }
   return jsUndefined();
 }
@@ -901,7 +901,7 @@ JSValue* DOMCSSValue::getValueProperty(ExecState* exec, int token) const
   case CssValueType:
     return jsNumber(cssValue.cssValueType());
   default:
-    assert(0);
+    ASSERT(0);
     return jsUndefined();
   }
 }
index 457e4b5eebba3fa2c5b3869ba1dbcced62e143be..14f8d2b4220fa84b713da20004be3cf24a66b7e2 100644 (file)
@@ -124,7 +124,7 @@ DOMNode::~DOMNode()
 
 void DOMNode::mark()
 {
-  assert(!marked());
+  ASSERT(!marked());
 
   Node* node = m_impl.get();
 
@@ -169,7 +169,7 @@ void DOMNode::mark()
   root->m_inSubtreeMark = false;
 
   // Double check that we actually ended up marked. This assert caught problems in the past.
-  assert(marked());
+  ASSERT(marked());
 }
 
 bool DOMNode::toBoolean(ExecState* ) const
@@ -717,7 +717,7 @@ JSValue* DOMNodeList::toPrimitive(ExecState* exec, JSType) const
 
 JSValue* DOMNodeList::getValueProperty(ExecState* exec, int token) const
 {
-  assert(token == Length);
+  ASSERT(token == Length);
   return jsNumber(m_impl->length());
 }
 
index 6f90c6519b3ba86f4d7d430d85c7b1fa9c22ab09..f95b860432f6039f16fc38e83ecdf95603ae8f07 100644 (file)
@@ -572,10 +572,10 @@ JSValue *Clipboard::getValueProperty(ExecState *exec, int token) const
 {
     switch (token) {
         case DropEffect:
-            assert(clipboard->isForDragging() || clipboard->dropEffect().isNull());
+            ASSERT(clipboard->isForDragging() || clipboard->dropEffect().isNull());
             return jsStringOrUndefined(clipboard->dropEffect());
         case EffectAllowed:
-            assert(clipboard->isForDragging() || clipboard->effectAllowed().isNull());
+            ASSERT(clipboard->isForDragging() || clipboard->effectAllowed().isNull());
             return jsStringOrUndefined(clipboard->effectAllowed());
         case Types:
         {
index f1768aa58db91a071627a8168bfffc8fc65bf76d..b0d8718ee12d7e8ab76960e3692c8cad7fc73326 100644 (file)
@@ -293,7 +293,7 @@ JSValue *JSHTMLDocument::getValueProperty(ExecState* exec, int token) const
   case DesignMode:
     return jsString(doc.inDesignMode() ? "on" : "off");
   default:
-    assert(0);
+    ASSERT(0);
     return jsUndefined();
   }
 }
index 0dac225163c323ea9fcf6978a559cf3826d0bc6f..e58a68eb329ef8c8dbe73fbf222a682d08aa8499 100644 (file)
@@ -300,7 +300,7 @@ KJS_IMPLEMENT_PROTOTYPE_FUNCTION(PluginsFunc)
 
 JSValue *Plugins::getValueProperty(ExecState *exec, int token) const
 {
-  assert(token == Length);
+  ASSERT(token == Length);
   return jsNumber(plugins->size());
 }
 
@@ -363,7 +363,7 @@ bool Plugins::getOwnPropertySlot(ExecState *exec, const Identifier& propertyName
 
 JSValue *MimeTypes::getValueProperty(ExecState *exec, int token) const
 {
-  assert(token == Length);
+  ASSERT(token == Length);
   return jsNumber(mimes->size());
 }
 
@@ -437,7 +437,7 @@ JSValue *Plugin::getValueProperty(ExecState *exec, int token) const
     case Length: 
         return jsNumber(m_info->mimes.size());
     default:
-        assert(0);
+        ASSERT(0);
         return jsUndefined();
     }
 }
index 75a0a8506048d8bffeeb4613490cd5ee79022bdd..2b65f05d4b66890443f2a826777524750e6b141a 100644 (file)
@@ -131,7 +131,7 @@ void KJSProxy::finishedWithEvent(Event* event)
 ScriptInterpreter* KJSProxy::interpreter()
 {
   initScriptIfNeeded();
-  assert(m_script);
+  ASSERT(m_script);
   return m_script.get();
 }
 
index 392fa9174fc1e7dc17c3618864f91bad82ce18be..d6fca91de9cbc88d50676954c8ff1a016b93df53 100644 (file)
@@ -1519,7 +1519,7 @@ static void parseWindowFeatures(const String& features, WindowFeatures& windowFe
             i++;
         valueEnd = i;
         
-        assert(i <= length);
+        ASSERT(i <= length);
 
         String keyString(buffer.substring(keyBegin, keyEnd - keyBegin));
         String valueString(buffer.substring(valueBegin, valueEnd - valueBegin));
index 955fe7156bc1d556214da446d019cbf600196561..5beafc968916624d60c86a5e72330814c92f9c95 100644 (file)
@@ -606,7 +606,7 @@ static List listFromNSArray(ExecState *exec, NSArray *array)
 
 - (void)dealloc
 {
-    assert(false);
+    ASSERT(false);
     return;
     [super dealloc]; // make -Wdealloc-check happy
 }
index 15a4f823d706a369d1beb9ca1950611427d15ca3..b133b1fb5c4736b3ef7bc5a36ba616a3c72a313c 100644 (file)
@@ -2774,7 +2774,7 @@ struct BorderImageParseContext
         else if (!m_bottom)
             m_bottom = val;
         else {
-            assert(!m_left);
+            ASSERT(!m_left);
             m_left = val;
         }
         
@@ -2790,7 +2790,7 @@ struct BorderImageParseContext
         else if (!m_borderBottom)
             m_borderBottom = val;
         else {
-            assert(!m_borderLeft);
+            ASSERT(!m_borderLeft);
             m_borderLeft = val;
         }
 
index bb5c2ca1c100118aa269ad117c01ea9b6ad78d67..71796970c8ee92eb183035c3b37c90d61f35dd15 100644 (file)
@@ -1610,7 +1610,7 @@ bool CSSStyleSelector::checkOneSelector(CSSSelector* sel, Element* e, bool isSub
                 dynamicPseudo = RenderStyle::SEARCH_RESULTS_BUTTON;
                 return true;
             case CSSSelector::PseudoNotParsed:
-                assert(false);
+                ASSERT(false);
                 break;
         }
         return false;
index 6cc83ce91b669baeafdb735b9d44eec117e3a172..7fcc3ad1218b7246b660a7a8dfb94471c72f59d4 100644 (file)
@@ -38,14 +38,14 @@ Attr::Attr(Element* element, Document* docPtr, Attribute* a)
       m_attribute(a),
       m_ignoreChildrenChanged(0)
 {
-    assert(!m_attribute->attr());
+    ASSERT(!m_attribute->attr());
     m_attribute->m_impl = this;
     m_specified = true;
 }
 
 Attr::~Attr()
 {
-    assert(m_attribute->attr() == this);
+    ASSERT(m_attribute->attr() == this);
     m_attribute->m_impl = 0;
 }
 
index f511ca06a4ac2bc64943e183044832bf091c939d..c6db06e61ebad14a474a8f4fca8bbf55e8a09369 100644 (file)
@@ -184,20 +184,20 @@ bool ContainerNode::insertBefore(PassRefPtr<Node> newChild, Node* refChild, Exce
         if (child->parentNode())
             break;
 
-        assert(!child->nextSibling());
-        assert(!child->previousSibling());
+        ASSERT(!child->nextSibling());
+        ASSERT(!child->previousSibling());
 
         // Add child before "next".
         forbidEventDispatch();
         Node* prev = next->previousSibling();
-        assert(m_lastChild != prev);
+        ASSERT(m_lastChild != prev);
         next->setPreviousSibling(child.get());
         if (prev) {
-            assert(m_firstChild != next);
-            assert(prev->nextSibling() == next);
+            ASSERT(m_firstChild != next);
+            ASSERT(prev->nextSibling() == next);
             prev->setNextSibling(child.get());
         } else {
-            assert(m_firstChild == next);
+            ASSERT(m_firstChild == next);
             m_firstChild = child.get();
         }
         child->setParent(this);
@@ -282,26 +282,26 @@ bool ContainerNode::replaceChild(PassRefPtr<Node> newChild, Node* oldChild, Exce
         if (child->parentNode())
             break;
 
-        assert(!child->nextSibling());
-        assert(!child->previousSibling());
+        ASSERT(!child->nextSibling());
+        ASSERT(!child->previousSibling());
 
         // Add child after "prev".
         forbidEventDispatch();
         Node* next;
         if (prev) {
             next = prev->nextSibling();
-            assert(m_firstChild != next);
+            ASSERT(m_firstChild != next);
             prev->setNextSibling(child.get());
         } else {
             next = m_firstChild;
             m_firstChild = child.get();
         }
         if (next) {
-            assert(m_lastChild != prev);
-            assert(next->previousSibling() == prev);
+            ASSERT(m_lastChild != prev);
+            ASSERT(next->previousSibling() == prev);
             next->setPreviousSibling(child.get());
         } else {
-            assert(m_lastChild == prev);
+            ASSERT(m_lastChild == prev);
             m_lastChild = child.get();
         }
         child->setParent(this);
@@ -878,7 +878,7 @@ Node *ContainerNode::childNode(unsigned index) const
 
 static void dispatchChildInsertionEvents(Node* child, ExceptionCode& ec)
 {
-    assert(!eventDispatchForbidden());
+    ASSERT(!eventDispatchForbidden());
 
     RefPtr<Node> c = child;
     RefPtr<Document> doc = child->document();
index 9bb8526fa72c588034a0cc31427f4131fedb65c0..e1f3ea884642a40c420f81d141ee30e9fdd4c84f 100644 (file)
@@ -231,8 +231,8 @@ static Widget* widgetForNode(Node* focusedNode)
 
 static bool relinquishesEditingFocus(Node *node)
 {
-    assert(node);
-    assert(node->isContentEditable());
+    ASSERT(node);
+    ASSERT(node->isContentEditable());
 
     Node *root = node->rootEditableElement();
     Frame* frame = node->document()->frame();
@@ -244,8 +244,8 @@ static bool relinquishesEditingFocus(Node *node)
 
 static bool acceptsEditingFocus(Node *node)
 {
-    assert(node);
-    assert(node->isContentEditable());
+    ASSERT(node);
+    ASSERT(node->isContentEditable());
 
     Node *root = node->rootEditableElement();
     Frame* frame = node->document()->frame();
@@ -1098,8 +1098,8 @@ void Document::updateLayoutIgnorePendingStylesheets()
 
 void Document::attach()
 {
-    assert(!attached());
-    assert(!m_inPageCache);
+    ASSERT(!attached());
+    ASSERT(!m_inPageCache);
 
     if (!m_renderArena)
         m_renderArena = new RenderArena();
@@ -1493,7 +1493,7 @@ void Document::write(const String& text)
     
     if (!m_tokenizer) {
         open();
-        assert(m_tokenizer);
+        ASSERT(m_tokenizer);
         if (!m_tokenizer)
             return;
         write(DeprecatedString("<html>"));
@@ -1705,7 +1705,7 @@ Node* Document::previousFocusableNode(Node* start, KeyboardEvent* event)
 
 int Document::nodeAbsIndex(Node *node)
 {
-    assert(node->document() == this);
+    ASSERT(node->document() == this);
 
     int absIndex = 0;
     for (Node *n = node; n && n != this; n = n->traversePreviousNode())
@@ -1724,7 +1724,7 @@ Node *Document::nodeWithAbsIndex(int absIndex)
 
 void Document::processHttpEquiv(const String &equiv, const String &content)
 {
-    assert(!equiv.isNull() && !content.isNull());
+    ASSERT(!equiv.isNull() && !content.isNull());
 
     Frame *frame = this->frame();
 
@@ -1920,7 +1920,7 @@ void Document::setSelectedStylesheetSet(const String& aString)
 void Document::stylesheetLoaded()
 {
     // Make sure we knew this sheet was pending, and that our count isn't out of sync.
-    assert(m_pendingStylesheets > 0);
+    ASSERT(m_pendingStylesheets > 0);
 
     m_pendingStylesheets--;
     
@@ -2688,7 +2688,7 @@ void Document::passwordFieldAdded()
 
 void Document::passwordFieldRemoved()
 {
-    assert(m_passwordFields > 0);
+    ASSERT(m_passwordFields > 0);
     m_passwordFields--;
 }
 
@@ -2704,7 +2704,7 @@ void Document::secureFormAdded()
 
 void Document::secureFormRemoved()
 {
-    assert(m_secureForms > 0);
+    ASSERT(m_secureForms > 0);
     m_secureForms--;
 }
 
@@ -2806,7 +2806,7 @@ void Document::removeMarkers(Range *range, DocumentMarker::MarkerType markerType
 
 void Document::addMarker(Node *node, DocumentMarker newMarker) 
 {
-    assert(newMarker.endOffset >= newMarker.startOffset);
+    ASSERT(newMarker.endOffset >= newMarker.startOffset);
     if (newMarker.endOffset == newMarker.startOffset)
         return;
     
@@ -3177,7 +3177,7 @@ void Document::shiftMarkers(Node *node, unsigned startOffset, int delta, Documen
     for (size_t i = 0; i != markers.size(); ++i) {
         DocumentMarker &marker = markers[i];
         if (marker.startOffset >= startOffset && (markerType == DocumentMarker::AllMarkers || marker.type == markerType)) {
-            assert((int)marker.startOffset + delta >= 0);
+            ASSERT((int)marker.startOffset + delta >= 0);
             marker.startOffset += delta;
             marker.endOffset += delta;
             docDirty = true;
index fa0ea819ae65703a49c2b9216690e9c6076f42e5..59e1c81c75c12363fab07bc2bf92071d6906f2f1 100644 (file)
@@ -66,7 +66,7 @@ PassRefPtr<Node> Element::cloneNode(bool deep)
 {
     ExceptionCode ec = 0;
     RefPtr<Element> clone = document()->createElementNS(namespaceURI(), nodeName(), ec);
-    assert(!ec);
+    ASSERT(!ec);
     
     // clone attributes
     if (namedAttrMap)
@@ -670,7 +670,7 @@ bool Element::childTypeAllowed(NodeType type)
 
 void Element::dispatchAttrRemovalEvent(Attribute*)
 {
-    assert(!eventDispatchForbidden());
+    ASSERT(!eventDispatchForbidden());
 #if 0
     if (!document()->hasListenerType(Document::DOMATTRMODIFIED_LISTENER))
         return;
@@ -682,7 +682,7 @@ void Element::dispatchAttrRemovalEvent(Attribute*)
 
 void Element::dispatchAttrAdditionEvent(Attribute *attr)
 {
-    assert(!eventDispatchForbidden());
+    ASSERT(!eventDispatchForbidden());
 #if 0
     if (!document()->hasListenerType(Document::DOMATTRMODIFIED_LISTENER))
         return;
index 65fb73642fd585fc829dd9b23a946a21ea839eff..dfc297d6b37f7713b5740fe0454e7a072a68edc2 100644 (file)
@@ -69,7 +69,7 @@ public:
     // used during parsing: only inserts if not already there
     // no error checking!
     void insertAttribute(Attribute* newAttribute) {
-        assert(!element);
+        ASSERT(!element);
         if (!getAttributeItem(newAttribute->name()))
             addAttribute(newAttribute);
         else
index 74403a66f16e800360f30429a64513e621e5eb3c..8e275b77cc5e29825c69bb358e4936ae8d69323b 100644 (file)
@@ -486,20 +486,20 @@ Node *Node::childNode(unsigned /*index*/) const
 Node *Node::traverseNextNode(const Node *stayWithin) const
 {
     if (firstChild()) {
-        assert(!stayWithin || firstChild()->isDescendantOf(stayWithin));
+        ASSERT(!stayWithin || firstChild()->isDescendantOf(stayWithin));
         return firstChild();
     }
     if (this == stayWithin)
         return 0;
     if (nextSibling()) {
-        assert(!stayWithin || nextSibling()->isDescendantOf(stayWithin));
+        ASSERT(!stayWithin || nextSibling()->isDescendantOf(stayWithin));
         return nextSibling();
     }
     const Node *n = this;
     while (n && !n->nextSibling() && (!stayWithin || n->parentNode() != stayWithin))
         n = n->parentNode();
     if (n) {
-        assert(!stayWithin || !n->nextSibling() || n->nextSibling()->isDescendantOf(stayWithin));
+        ASSERT(!stayWithin || !n->nextSibling() || n->nextSibling()->isDescendantOf(stayWithin));
         return n->nextSibling();
     }
     return 0;
@@ -510,14 +510,14 @@ Node *Node::traverseNextSibling(const Node *stayWithin) const
     if (this == stayWithin)
         return 0;
     if (nextSibling()) {
-        assert(!stayWithin || nextSibling()->isDescendantOf(stayWithin));
+        ASSERT(!stayWithin || nextSibling()->isDescendantOf(stayWithin));
         return nextSibling();
     }
     const Node *n = this;
     while (n && !n->nextSibling() && (!stayWithin || n->parentNode() != stayWithin))
         n = n->parentNode();
     if (n) {
-        assert(!stayWithin || !n->nextSibling() || n->nextSibling()->isDescendantOf(stayWithin));
+        ASSERT(!stayWithin || !n->nextSibling() || n->nextSibling()->isDescendantOf(stayWithin));
         return n->nextSibling();
     }
     return 0;
@@ -539,20 +539,20 @@ Node *Node::traversePreviousNode(const Node *stayWithin) const
 Node *Node::traversePreviousNodePostOrder(const Node *stayWithin) const
 {
     if (lastChild()) {
-        assert(!stayWithin || lastChild()->isDescendantOf(stayWithin));
+        ASSERT(!stayWithin || lastChild()->isDescendantOf(stayWithin));
         return lastChild();
     }
     if (this == stayWithin)
         return 0;
     if (previousSibling()) {
-        assert(!stayWithin || previousSibling()->isDescendantOf(stayWithin));
+        ASSERT(!stayWithin || previousSibling()->isDescendantOf(stayWithin));
         return previousSibling();
     }
     const Node *n = this;
     while (n && !n->previousSibling() && (!stayWithin || n->parentNode() != stayWithin))
         n = n->parentNode();
     if (n) {
-        assert(!stayWithin || !n->previousSibling() || n->previousSibling()->isDescendantOf(stayWithin));
+        ASSERT(!stayWithin || !n->previousSibling() || n->previousSibling()->isDescendantOf(stayWithin));
         return n->previousSibling();
     }
     return 0;
@@ -810,8 +810,8 @@ void Node::dump(TextStream* stream, DeprecatedString ind) const
 
 void Node::attach()
 {
-    assert(!attached());
-    assert(!renderer() || (renderer()->style() && renderer()->parent()));
+    ASSERT(!attached());
+    ASSERT(!renderer() || (renderer()->style() && renderer()->parent()));
     document()->incDOMTreeVersion();
     m_attached = true;
 }
@@ -888,7 +888,7 @@ Node *Node::previousEditable() const
 // the last child, it specifies to start at next sibling.
 Node *Node::nextEditable(int offset) const
 {
-    assert(offset>=0);
+    ASSERT(offset>=0);
     Node *node;
     if (hasChildNodes())
         node = (offset >= (int)childNodeCount()) ? nextSibling() : childNode(offset)->nextLeafNode();
@@ -994,11 +994,11 @@ void Node::createRendererIfNeeded()
     if (!document()->shouldCreateRenderers())
         return;
     
-    assert(!attached());
-    assert(!renderer());
+    ASSERT(!attached());
+    ASSERT(!renderer());
     
     Node *parent = parentNode();    
-    assert(parent);
+    ASSERT(parent);
     
     RenderObject *parentRenderer = parent->renderer();
     if (parentRenderer && parentRenderer->canHaveChildren()
@@ -1034,7 +1034,7 @@ bool Node::rendererIsNeeded(RenderStyle *style)
 
 RenderObject *Node::createRenderer(RenderArena *arena, RenderStyle *style)
 {
-    assert(false);
+    ASSERT(false);
     return 0;
 }
 
index 546cdfedfebcc09ffc5114cb34f34ebe50ec9903..1b1c3bcc71ce40382d6aa57a2cffb465a200576b 100644 (file)
@@ -556,7 +556,7 @@ short Range::compareBoundaryPoints( Node *containerA, int offsetA, Node *contain
     }
 
     // Should never reach this point.
-    assert(0);
+    ASSERT(0);
     return 0;
 }
 
index 3dbc5c4fc95b6856de36bae05a2d2532684dda6b..d7f22d29cb91f38182e7dc2753ef46d74b2c076d 100644 (file)
@@ -382,7 +382,7 @@ void StyledElement::addCSSColor(MappedAttribute* attr, int id, const String &c)
             colors[0] >>= 4*maxDigit;
             colors[1] >>= 4*maxDigit;
             colors[2] >>= 4*maxDigit;
-            // assert(colors[0] < 0x100 && colors[1] < 0x100 && colors[2] < 0x100);
+            // ASSERT(colors[0] < 0x100 && colors[1] < 0x100 && colors[2] < 0x100);
             
             color = String::format("#%02x%02x%02x", colors[0], colors[1], colors[2]);
             if (attr->decl()->setProperty(id, color, false))
@@ -419,7 +419,7 @@ unsigned MappedAttributeHash::hash(const MappedAttributeKey& key)
     tmp = (p[1] << 11) ^ hash;
     hash = (hash << 16) ^ tmp;
     hash += hash >> 11;
-    assert(sizeof(key.name) == 4 || sizeof(key.name) == 8);
+    ASSERT(sizeof(key.name) == 4 || sizeof(key.name) == 8);
     if (sizeof(key.name) == 8) {
         p += 2;
         hash += p[0];
@@ -433,7 +433,7 @@ unsigned MappedAttributeHash::hash(const MappedAttributeKey& key)
     tmp = (p[1] << 11) ^ hash;
     hash = (hash << 16) ^ tmp;
     hash += hash >> 11;
-    assert(sizeof(key.value) == 4 || sizeof(key.value) == 8);
+    ASSERT(sizeof(key.value) == 4 || sizeof(key.value) == 8);
     if (sizeof(key.value) == 8) {
         p += 2;
         hash += p[0];
index 2419bc6710612353aba9c9cfd7495c92cf8abbf6..febdc607e55b0c3336274cad08a3b03cca81cbed 100644 (file)
@@ -1066,7 +1066,7 @@ static xmlEntityPtr getXHTMLEntity(const xmlChar* name)
         return 0;
 
     CString value = String(&c, 1).utf8();
-    assert(value.length() < 5);
+    ASSERT(value.length() < 5);
     sharedXHTMLEntity.length = value.length();
     sharedXHTMLEntity.name = name;
     memcpy(sharedXHTMLEntityResult, value.data(), sharedXHTMLEntity.length + 1);
index bc7991e50a39c3dbec50bde8da6baa9edb618990..c88835689cd72477e19ac2a60630116407b8621d 100644 (file)
@@ -665,7 +665,7 @@ bool Editor::selectionStartHasStyle(CSSStyleDeclaration* style) const
     if (nodeToRemove) {
         ExceptionCode ec = 0;
         nodeToRemove->remove(ec);
-        assert(ec == 0);
+        ASSERT(ec == 0);
     }
     
     return match;
index 887457ecb757285831e26d448beacea9aa470963..8c88b0abd87be7950386ee01c6f26be136f24740 100644 (file)
@@ -94,7 +94,7 @@ Position InsertTextCommand::prepareForTextInsertion(const Position& p)
 
 void InsertTextCommand::input(const String &text, bool selectInsertedText)
 {
-    assert(text.find('\n') == -1);
+    ASSERT(text.find('\n') == -1);
 
     if (endingSelection().isNone())
         return;
index f5b9f6b1729bcc85a526b3fbf224ed8f63d74d8a..93c728cfb947ca73cf3cd8761e24fba4313ba9d7 100644 (file)
@@ -749,7 +749,7 @@ void ReplaceSelectionCommand::completeHTMLReplacement(const Position &lastPositi
         rebalanceWhitespaceAt(end);
 
         if (m_matchStyle) {
-            assert(m_insertionStyle);
+            ASSERT(m_insertionStyle);
             applyStyle(m_insertionStyle.get(), start, end);
         }    
         
index afefb8238cc6d019aacdb5373e4d98d52b9a226c..0ceaaf40b56a5f3b57a4ffd6f2d40a7738f9238d 100644 (file)
@@ -569,7 +569,7 @@ void TextIterator::exitNode()
             emitCharacter('\n', baseNode->parentNode(), baseNode, 1, 1);
 
             // remember whether to later add a newline for the current node
-            assert(!m_needAnotherNewline);
+            ASSERT(!m_needAnotherNewline);
             m_needAnotherNewline = addNewline;
         } else if (addNewline) {
             // insert a newline with a position following this block's contents.
@@ -696,7 +696,7 @@ SimplifiedBackwardsTextIterator::SimplifiedBackwardsTextIterator(const Range *r)
 
 void SimplifiedBackwardsTextIterator::advance()
 {
-    assert(m_positionNode);
+    ASSERT(m_positionNode);
 
     m_positionNode = 0;
     m_textLength = 0;
@@ -876,11 +876,11 @@ PassRefPtr<Range> CharacterIterator::range() const
     RefPtr<Range> r = m_textIterator.range();
     if (!m_textIterator.atEnd()) {
         if (m_textIterator.length() <= 1) {
-            assert(m_runOffset == 0);
+            ASSERT(m_runOffset == 0);
         } else {
             int exception = 0;
             Node *n = r->startContainer(exception);
-            assert(n == r->endContainer(exception));
+            ASSERT(n == r->endContainer(exception));
             int offset = r->startOffset(exception) + m_runOffset;
             r->setStart(n, offset, exception);
             r->setEnd(n, offset + 1, exception);
@@ -975,7 +975,7 @@ void WordAwareIterator::advance()
 
     // If last time we did a look-ahead, start with that looked-ahead chunk now
     if (!m_didLookAhead) {
-        assert(!m_textIterator.atEnd());
+        ASSERT(!m_textIterator.atEnd());
         m_textIterator.advance();
     }
     m_didLookAhead = false;
@@ -1038,7 +1038,7 @@ const UChar* WordAwareIterator::characters() const
 CircularSearchBuffer::CircularSearchBuffer(const String& s, bool isCaseSensitive)
     : m_target(s)
 {
-    assert(!s.isEmpty());
+    ASSERT(!s.isEmpty());
 
     if (!isCaseSensitive)
         m_target = s.foldCase();
@@ -1069,8 +1069,8 @@ void CircularSearchBuffer::append(int count, const UChar* characters)
 {
     int tailSpace = m_buffer + length() - m_cursor;
 
-    assert(!m_bufferFull);
-    assert(count <= tailSpace);
+    ASSERT(!m_bufferFull);
+    ASSERT(count <= tailSpace);
 
     if (m_isCaseSensitive) {
         for (int i = 0; i != count; ++i) {
@@ -1098,7 +1098,7 @@ int CircularSearchBuffer::neededCharacters() const
 
 bool CircularSearchBuffer::isMatch() const
 {
-    assert(m_bufferFull);
+    ASSERT(m_bufferFull);
 
     int headSpace = m_cursor - m_buffer;
     int tailSpace = length() - headSpace;
@@ -1154,9 +1154,9 @@ PassRefPtr<Range> TextIterator::rangeFromLocationAndLength(Element *scope, int r
         textRunRange = it.range();
         
         resultRange->setStart(textRunRange->startContainer(exception), 0, exception);
-        assert(exception == 0);
+        ASSERT(exception == 0);
         resultRange->setEnd(textRunRange->startContainer(exception), 0, exception);
-        assert(exception == 0);
+        ASSERT(exception == 0);
         
         return resultRange.release();
     }
index 4f6dae3e258f9be9445a4795c71a0bd7f6e804d4..c82f76017e1db6026cc8820c93a7e7f775fe856c 100644 (file)
@@ -57,7 +57,7 @@ public:
     bool isNotNull() const { return m_deepPosition.isNotNull(); }
 
     Position deepEquivalent() const { return m_deepPosition; }
-    EAffinity affinity() const { assert(m_affinity == UPSTREAM || m_affinity == DOWNSTREAM); return m_affinity; }
+    EAffinity affinity() const { ASSERT(m_affinity == UPSTREAM || m_affinity == DOWNSTREAM); return m_affinity; }
     void setAffinity(EAffinity affinity) { m_affinity = affinity; }
 
     // next() and previous() will increment/decrement by a character cluster.
index 57f528892a856f182195de95093739fed089abf1..bc307cb48acf2200e99d49b7e17517c2395dcc9b 100644 (file)
@@ -844,7 +844,7 @@ PassRefPtr<Element> createTabSpanElement(Document* document, PassRefPtr<Node> ta
     // make the span to hold the tab
     ExceptionCode ec = 0;
     RefPtr<Element> spanElement = document->createElementNS(xhtmlNamespaceURI, "span", ec);
-    assert(ec == 0);
+    ASSERT(ec == 0);
     spanElement->setAttribute(classAttr, AppleTabSpanClass);
     spanElement->setAttribute(styleAttr, "white-space:pre");
 
@@ -852,7 +852,7 @@ PassRefPtr<Element> createTabSpanElement(Document* document, PassRefPtr<Node> ta
     if (!tabTextNode)
         tabTextNode = document->createEditingTextNode("\t");
     spanElement->appendChild(tabTextNode, ec);
-    assert(ec == 0);
+    ASSERT(ec == 0);
 
     return spanElement.release();
 }
index 855912e732d9815a76cbbca28149e78f9add905f..e3be5c11b0111f34c1c101b9b45ba146cf2ea473 100644 (file)
@@ -604,7 +604,7 @@ static void fillContainerFromString(ContainerNode* paragraph, const DeprecatedSt
         return;
     }
 
-    assert(string.find('\n') == -1);
+    ASSERT(string.find('\n') == -1);
 
     DeprecatedStringList tabList = DeprecatedStringList::split('\t', string, true);
     DeprecatedString tabText = "";
index 34a1b17fcd12fdb63e7c85ecd5bedf0a2a6a92f4..6276ebe04bdd1cf1dd78815c7dc543bd9e9fb91d 100644 (file)
@@ -427,7 +427,7 @@ VisiblePosition previousLinePosition(const VisiblePosition &visiblePosition, int
                 break;
             Position pos(n, n->caretMinOffset());
             if (pos.isCandidate()) {
-                assert(n->renderer());
+                ASSERT(n->renderer());
                 box = n->renderer()->inlineBox(n->caretMaxOffset());
                 if (box) {
                     // previous root line box found
@@ -497,7 +497,7 @@ VisiblePosition nextLinePosition(const VisiblePosition &visiblePosition, int x)
                 break;
             Position pos(n, n->caretMinOffset());
             if (pos.isCandidate()) {
-                assert(n->renderer());
+                ASSERT(n->renderer());
                 box = n->renderer()->inlineBox(n->caretMinOffset());
                 if (box) {
                     // next root line box found
index 04687c9e4888f71812df5f18fbe0984898db01bf..c8036f5748abf97d64dc300a789e2de4c47c4832 100644 (file)
@@ -46,7 +46,7 @@ HTMLAppletElement::~HTMLAppletElement()
 {
 #if USE(JAVASCRIPTCORE_BINDINGS)
     // m_instance should have been cleaned up in detach().
-    assert(!m_instance);
+    ASSERT(!m_instance);
 #endif
 }
 
index 644026e15333e0ad63fe3b8c2de7143272ceb120..53fb13666bc1701a6d8b57380ef6882209436590 100644 (file)
@@ -96,7 +96,7 @@ void HTMLCollection::resetCollectionInfo() const
 
 Node *HTMLCollection::traverseNextItem(Node *current) const
 {
-    assert(current);
+    ASSERT(current);
 
     if (type == NodeChildren && m_base.get() != current)
         current = current->nextSibling();
index 3debc1667641e5d8e0d80631174be27587c06db8..dd351892af9fbdcda3378c507466b8e9a5350b88 100644 (file)
@@ -162,7 +162,7 @@ static void removeItemFromMap(HTMLDocument::NameCountMap& map, const String& nam
         return;
 
     int oldVal = it->second;
-    assert(oldVal != 0);
+    ASSERT(oldVal != 0);
     int newVal = oldVal - 1;
     if (newVal == 0)
         map.remove(it);
@@ -411,7 +411,7 @@ void HTMLDocument::determineParseMode(const String& str)
                     hMode = Html4;
                     break;
                  default:
-                    assert(false);
+                    ASSERT(false);
             }
         }   
     }
index 645a6f906eee2f2cbe6aba3d2771bc96aa04bee2..fb310eb4381e598584b44fa929069ee6da34f2a5 100644 (file)
@@ -261,15 +261,15 @@ PassRefPtr<DocumentFragment> HTMLElement::createContextualFragment(const String
             for (RefPtr<Node> child = firstChild; child; child = nextChild) {
                 nextChild = child->nextSibling();
                 node->removeChild(child.get(), ignoredExceptionCode);
-                assert(!ignoredExceptionCode);
+                ASSERT(!ignoredExceptionCode);
                 fragment->insertBefore(child, node.get(), ignoredExceptionCode);
-                assert(!ignoredExceptionCode);
+                ASSERT(!ignoredExceptionCode);
             }
             fragment->removeChild(node.get(), ignoredExceptionCode);
-            assert(!ignoredExceptionCode);
+            ASSERT(!ignoredExceptionCode);
         } else if (node->hasTagName(headTag)) {
             fragment->removeChild(node.get(), ignoredExceptionCode);
-            assert(!ignoredExceptionCode);
+            ASSERT(!ignoredExceptionCode);
         }
     }
 
index 658cad83c74d3871987fc60baf940bb4559214c3..74c6a5f984a7d9404f9f33da975830304faae390 100644 (file)
@@ -51,7 +51,7 @@ HTMLEmbedElement::~HTMLEmbedElement()
 {
 #if USE(JAVASCRIPTCORE_BINDINGS)
     // m_instance should have been cleaned up in detach().
-    assert(!m_instance);
+    ASSERT(!m_instance);
 #endif
 }
 
@@ -129,7 +129,7 @@ bool HTMLEmbedElement::rendererIsNeeded(RenderStyle *style)
 
     Node *p = parentNode();
     if (p && p->hasTagName(objectTag)) {
-        assert(p->renderer());
+        ASSERT(p->renderer());
         return false;
     }
 
index e9d0c10d970a1a13aa61f05c8ca3139d26342703..f4854602c9aac436093d0d44a7ce3a0e8d40a943 100644 (file)
@@ -81,7 +81,7 @@ void HTMLGenericFormElement::parseMappedAttribute(MappedAttribute *attr)
 
 void HTMLGenericFormElement::attach()
 {
-    assert(!attached());
+    ASSERT(!attached());
 
     HTMLElement::attach();
 
index c48a0d86822b63abc192d91f973517c774b8e724..01d49b6a779e79040aba56f23cf38138d8e1397a 100644 (file)
@@ -701,7 +701,7 @@ bool HTMLInputElement::rendererIsNeeded(RenderStyle *style)
         case HIDDEN:
             return false;
     }
-    assert(false);
+    ASSERT(false);
     return false;
 }
 
@@ -729,7 +729,7 @@ RenderObject *HTMLInputElement::createRenderer(RenderArena *arena, RenderStyle *
         case TEXT:
             return new (arena) RenderTextControl(this, false);             
     }
-    assert(false);
+    ASSERT(false);
     return 0;
 }
 
index 18e115ee71be1b6b5cfa46c4594dfa4be13d9f4a..980fbb997d732d42ef7eb555176ad77e9d83ce91 100644 (file)
@@ -77,7 +77,7 @@ void HTMLLIElement::parseMappedAttribute(MappedAttribute* attr)
 
 void HTMLLIElement::attach()
 {
-    assert(!attached());
+    ASSERT(!attached());
 
     HTMLElement::attach();
 
index fecd8865400a199b0640ed7f107797243da0c339..1ba9691dbb305f8b761a868eb004c7ab0632f5cc 100644 (file)
@@ -63,7 +63,7 @@ HTMLObjectElement::~HTMLObjectElement()
 {
 #if USE(JAVASCRIPTCORE_BINDINGS)
     // m_instance should have been cleaned up in detach().
-    assert(!m_instance);
+    ASSERT(!m_instance);
 #endif
     
     delete m_imageLoader;
index de0a7bdfa6eebc1f9c2a60e9273e0cacad5ce1a7..7e07b52a1081dbfb2720d0ce4a242e51d67fec60 100644 (file)
@@ -102,7 +102,7 @@ void HTMLScriptElement::insertedIntoDocument()
 {
     HTMLElement::insertedIntoDocument();
 
-    assert(!m_cachedScript);
+    ASSERT(!m_cachedScript);
 
     if (m_createdByParser)
         return;
@@ -142,7 +142,7 @@ void HTMLScriptElement::notifyFinished(CachedResource* o)
 {
     CachedScript *cs = static_cast<CachedScript *>(o);
 
-    assert(cs == m_cachedScript);
+    ASSERT(cs == m_cachedScript);
 
     // Evaluating the script could lead to a garbage collection which
     // can delete the script element so we need to protect it.
index c84c9fc10f2d59428270ef857494be0d5cf11264..e0d4bf501189b9db5c18ff2afe4edb4d2cfdd31b 100644 (file)
@@ -295,7 +295,7 @@ ContainerNode* HTMLTableElement::addChild(PassRefPtr<Node> child)
 
     // The creation of <tbody> elements relies on the "childAllowed" check,
     // so we need to do it even for XML documents.
-    assert(child->nodeType() != DOCUMENT_FRAGMENT_NODE);
+    ASSERT(child->nodeType() != DOCUMENT_FRAGMENT_NODE);
     if (!document()->isHTMLDocument() && !childAllowed(child.get()))
         return 0;
 
@@ -607,7 +607,7 @@ CSSMutableStyleDeclaration* HTMLTableElement::getSharedGroupDecl(bool rows)
 
 void HTMLTableElement::attach()
 {
-    assert(!m_attached);
+    ASSERT(!m_attached);
     HTMLElement::attach();
     if (renderer() && renderer()->isTable())
         static_cast<RenderTable*>(renderer())->setCellPadding(m_padding);
index aee5c811284db212e362e7ec285d0668c7294419..c74027c23baf89427f00261d862332e22db30f4c 100644 (file)
@@ -75,7 +75,7 @@ void CachedXBLDocument::data(Vector<char>& data, bool )
     if (!allDataReceived)
         return;
     
-    assert(!m_document);
+    ASSERT(!m_document);
     
     m_document = new XBL::XBLDocument();
     m_document->ref();
index c1adf4a7f2ec0f6365f8cbde0a62646b94f8ede2..3853451e717f1c504f687c3a3658788872f2e761 100644 (file)
@@ -660,7 +660,7 @@ void Frame::paintCaret(GraphicsContext* p, const IntRect& rect) const
 void Frame::paintDragCaret(GraphicsContext* p, const IntRect& rect) const
 {
     SelectionController* dragCaretController = d->m_page->dragCaretController();
-    assert(dragCaretController->selection().isCaret());
+    ASSERT(dragCaretController->selection().isCaret());
     if (dragCaretController->selection().start().node()->document()->frame() == this)
         dragCaretController->paintCaret(p, rect);
 }
@@ -864,7 +864,7 @@ Frame::TriState Frame::selectionHasStyle(CSSStyleDeclaration *style) const
         if (nodeToRemove) {
             ExceptionCode ec = 0;
             nodeToRemove->remove(ec);
-            assert(ec == 0);
+            ASSERT(ec == 0);
         }
     } else {
         for (Node* node = selectionController()->start().node(); node; node = node->traverseNextNode()) {
@@ -893,7 +893,7 @@ String Frame::selectionStartStylePropertyValue(int stylePropertyID) const
     if (nodeToRemove) {
         ExceptionCode ec = 0;
         nodeToRemove->remove(ec);
-        assert(ec == 0);
+        ASSERT(ec == 0);
     }
 
     return value;
@@ -941,7 +941,7 @@ CSSComputedStyleDeclaration *Frame::selectionComputedStyle(Node *&nodeToRemove)
                 parent->appendChild(styleElement, ec);
             }
         }
-        assert(ec == 0);
+        ASSERT(ec == 0);
 
         nodeToRemove = styleElement.get();
     }
index 5ae79474befc3220e3def9115158b66493547bd3..5b4d5cb366884e8ceadfe3325d8bc8f4e092d085 100644 (file)
@@ -50,6 +50,7 @@
 #include <assert.h>
 #include <stdlib.h>
 #include <string.h>
+#include <wtf/Assertions.h>
 #include <wtf/FastMalloc.h>
 
 using namespace std;
@@ -128,7 +129,7 @@ void* ArenaAllocate(ArenaPool *pool, unsigned int nb)
     Arena *a;   
     char *rp;     /* returned pointer */
 
-    assert((nb & pool->mask) == 0);
+    ASSERT((nb & pool->mask) == 0);
     
     nb = (uword)ARENA_ALIGN(pool, nb); /* force alignment */
 
@@ -225,7 +226,7 @@ static void FreeArenaList(ArenaPool *pool, Arena *head, bool reallyFree)
 
 #ifdef DEBUG
     do {
-        assert(a->base <= a->avail && a->avail <= a->limit);
+        ASSERT(a->base <= a->avail && a->avail <= a->limit);
         a->avail = a->base;
         CLEAR_UNUSED(a);
     } while ((a = a->next) != 0);
index 05b6ef7d713a6f93d581018f12081315ffede904..062b6358874f6ef064df1fa8a82f6183530d71e5 100644 (file)
@@ -98,7 +98,7 @@ void* ArenaAllocate(ArenaPool *pool, unsigned int nb);
 
 #ifdef DEBUG
 #define FREE_PATTERN 0xDA
-#define CLEAR_UNUSED(a) (assert((a)->avail <= (a)->limit), \
+#define CLEAR_UNUSED(a) (ASSERT((a)->avail <= (a)->limit), \
                            memset((void*)(a)->avail, FREE_PATTERN, \
                             (a)->limit - (a)->avail))
 #define CLEAR_ARENA(a)  memset((void*)(a), FREE_PATTERN, \
index 8875a6179ef79cd0e866fef4324ce5baed693ce6..616cece76412e652a9f093ea8f7c4f4113ce57e5 100644 (file)
@@ -448,19 +448,19 @@ const GlyphData& Font::glyphDataForCharacter(UChar32 c, const UChar* cluster, un
 
 const FontData* Font::primaryFont() const
 {
-    assert(m_fontList);
+    ASSERT(m_fontList);
     return m_fontList->primaryFont(this);
 }
 
 const FontData* Font::fontDataAt(unsigned index) const
 {
-    assert(m_fontList);
+    ASSERT(m_fontList);
     return m_fontList->fontDataAt(this, index);
 }
 
 const FontData* Font::fontDataForCharacters(const UChar* characters, int length) const
 {
-    assert(m_fontList);
+    ASSERT(m_fontList);
     return m_fontList->fontDataForCharacters(this, characters, length);
 }
 
@@ -510,7 +510,7 @@ float Font::xHeight() const
 
 bool Font::isFixedPitch() const
 {
-    assert(m_fontList);
+    ASSERT(m_fontList);
     return m_fontList->isFixedPitch(this);
 }
 
index e5931bcd9fdb876ab0d5a0f08bca429a89aaa861..f18be413b189c84d9896badae5777628d6a7e53c 100644 (file)
@@ -57,7 +57,7 @@ const FontData* FontFallbackList::fontDataAt(const Font* font, unsigned realized
         return m_fontList[realizedFontIndex]; // This fallback font is already in our list.
 
     // Make sure we're not passing in some crazy value here.
-    assert(realizedFontIndex == m_fontList.size());
+    ASSERT(realizedFontIndex == m_fontList.size());
 
     if (m_familyIndex == cAllFamiliesScanned)
         return 0;
index 740e9c709094fd506e6a7503b5b1bcac703852e5..4f26e0eeea7e10b80b1fef63aff8139c6742807f 100644 (file)
@@ -1478,7 +1478,7 @@ bool KURL::isHierarchical() const
 {
     if (!m_isValid)
         return false;
-    assert(urlString[schemeEndPos] == ':');
+    ASSERT(urlString[schemeEndPos] == ':');
     return urlString[schemeEndPos + 1] == '/';
 }
 
index 3f5d4d00967c9f913ef77dcf2d8efaa2bd5ee1d0..b7e0a70dc345c56becc1f704645e235ebf87a5e2 100644 (file)
@@ -95,7 +95,7 @@ void SegmentedString::append(const SegmentedSubstring &s)
 
 void SegmentedString::prepend(const SegmentedSubstring &s)
 {
-    assert(!escaped());
+    ASSERT(!escaped());
     if (s.m_length) {
         if (!m_currentString.m_length)
             m_currentString = s;
@@ -110,7 +110,7 @@ void SegmentedString::prepend(const SegmentedSubstring &s)
 
 void SegmentedString::append(const SegmentedString &s)
 {
-    assert(!s.escaped());
+    ASSERT(!s.escaped());
     append(s.m_currentString);
     if (s.m_composite) {
         DeprecatedValueListConstIterator<SegmentedSubstring> i = s.m_substrings.begin();
@@ -123,8 +123,8 @@ void SegmentedString::append(const SegmentedString &s)
 
 void SegmentedString::prepend(const SegmentedString &s)
 {
-    assert(!escaped());
-    assert(!s.escaped());
+    ASSERT(!escaped());
+    ASSERT(!s.escaped());
     if (s.m_composite) {
         DeprecatedValueListConstIterator<SegmentedSubstring> i = s.m_substrings.fromLast();
         DeprecatedValueListConstIterator<SegmentedSubstring> e = s.m_substrings.end();
index d2b939e26dffa40a36073b54c77c8f0ecf81d480..f756b69015cf5f25bb77df32f84db539bd132524 100644 (file)
@@ -85,7 +85,7 @@ public:
             m_pushedChar1 = c;
             m_currentChar = m_pushedChar1 ? &m_pushedChar1 : m_currentString.m_current;
         } else {
-            assert(!m_pushedChar2);
+            ASSERT(!m_pushedChar2);
             m_pushedChar2 = c;
         }
     }
index b8a47ad1247dadbd31a56f52d9476e6d36e9e55c..d92b76fadb37e896c0bfb533bc38e364c79688c4 100644 (file)
@@ -39,7 +39,7 @@ public:
 
     DataRef(const DataRef<T>& d)
     {
-        assert(d.m_data);
+        ASSERT(d.m_data);
         m_data = d.m_data;
         m_data->ref();
     }
@@ -67,14 +67,14 @@ public:
 
     void init()
     {
-        assert(!m_data);
+        ASSERT(!m_data);
         m_data = new T;
         m_data->ref();
     }
 
     DataRef<T>& operator=(const DataRef<T>& d)
     {
-        assert(d.m_data);
+        ASSERT(d.m_data);
         d.m_data->ref();
         if (m_data)
             m_data->deref();
@@ -84,15 +84,15 @@ public:
 
     bool operator==(const DataRef<T>& o) const
     {
-        assert(m_data);
-        assert(o.m_data);
+        ASSERT(m_data);
+        ASSERT(o.m_data);
         return m_data == o.m_data || *m_data == *o.m_data;
     }
     
     bool operator!=(const DataRef<T>& o) const
     {
-        assert(m_data);
-        assert(o.m_data);
+        ASSERT(m_data);
+        ASSERT(o.m_data);
         return m_data != o.m_data && *m_data != *o.m_data;
     }
 
index d249583eae9461a66011be2edea59b22f687c978..ac17abe2c926280a7993ce2e121b31f6c51249f0 100644 (file)
@@ -62,7 +62,7 @@ void* InlineBox::operator new(size_t sz, RenderArena* renderArena) throw()
 
 void InlineBox::operator delete(void* ptr, size_t sz)
 {
-    assert(inInlineBoxDetach);
+    ASSERT(inInlineBoxDetach);
 
     // Stash size where destroy can find it.
     *(size_t *)ptr = sz;
index 65d9aa8e24a7fd4a5fcb2e0c087f87ec1f0535a7..bf84f4a88736280607d0feb031ca78ed5f23a0a3 100644 (file)
@@ -63,7 +63,7 @@ void* InlineTextBox::operator new(size_t sz, RenderArena* renderArena) throw()
 
 void InlineTextBox::operator delete(void* ptr, size_t sz)
 {
-    assert(inInlineTextBoxDetach);
+    ASSERT(inInlineTextBoxDetach);
     
     // Stash size where destroy can find it.
     *static_cast<size_t*>(ptr) = sz;
index 083fa8adb13eb2fe1ee17d02a29b9d308bbad6f4..1ea84de27a243cee05825d22be78bd1a45477430 100644 (file)
@@ -38,6 +38,7 @@
 #include <assert.h>
 #include <stdlib.h>
 #include <string.h>
+#include <wtf/Assertions.h>
 
 #define ROUNDUP(x,y) ((((x)+((y)-1))/(y))*(y))
 
@@ -75,7 +76,7 @@ void* RenderArena::allocate(size_t size)
 {
 #ifndef NDEBUG
     // Use standard malloc so that memory debugging tools work.
-    assert(this);
+    ASSERT(this);
     void* block = ::malloc(sizeof(RenderArenaDebugHeader) + size);
     RenderArenaDebugHeader* header = (RenderArenaDebugHeader*)block;
     header->arena = this;
@@ -114,9 +115,9 @@ void RenderArena::free(size_t size, void* ptr)
 #ifndef NDEBUG
     // Use standard free so that memory debugging tools work.
     RenderArenaDebugHeader* header = (RenderArenaDebugHeader*)ptr - 1;
-    assert(header->signature == signature);
-    assert(header->size == size);
-    assert(header->arena == this);
+    ASSERT(header->signature == signature);
+    ASSERT(header->size == size);
+    ASSERT(header->arena == this);
     header->signature = signatureDead;
     ::free(header);
 #else
index a8a473a20edfeb1fbe401cebdd8c185407caf392..d1de4bfa590a9b52c06d11fbe375462f2104a2a7 100644 (file)
@@ -372,7 +372,7 @@ bool RenderImage::isWidthSpecified() const
         default:
             return false;
     }
-    assert(false);
+    ASSERT(false);
     return false;
 }
 
@@ -385,7 +385,7 @@ bool RenderImage::isHeightSpecified() const
         default:
             return false;
     }
-    assert(false);
+    ASSERT(false);
     return false;
 }
 
index 647068762ee0a3ac82c7b04c12245b82276a4579..505e148f0180c42bbd5eff16c30e354a796740f6 100644 (file)
@@ -169,7 +169,7 @@ RenderLayer::~RenderLayer()
     delete m_marquee;
     
     // Make sure we have no lingering clip rects.
-    assert(!m_clipRects);
+    ASSERT(!m_clipRects);
 }
 
 void RenderLayer::checkForRepaintOnResize()
@@ -482,7 +482,7 @@ void* RenderLayer::operator new(size_t sz, RenderArena* renderArena) throw()
 
 void RenderLayer::operator delete(void* ptr, size_t sz)
 {
-    assert(inRenderLayerDestroy);
+    ASSERT(inRenderLayerDestroy);
     
     // Stash size where destroy can find it.
     *(size_t *)ptr = sz;
index f2f29d1e9b34dd30f15950ecb66eb5fefa7b8014..a88becb1235fd714b719bf08d5803c24f0e224d6 100644 (file)
@@ -233,7 +233,6 @@ IntRect RenderView::selectionRect() const
     while (os && os != stop) {
         if ((os->canBeSelectionLeaf() || os == m_selectionStart || os == m_selectionEnd) && os->selectionState() != SelectionNone) {
             // Blocks are responsible for painting line gaps and margin gaps. They must be examined as well.
-//          assert(!selectedObjects.get(os));
             selectedObjects.set(os, new SelectionInfo(os));
             RenderBlock* cb = os->containingBlock();
             while (cb && !cb->isRenderView()) {
index 444d85560d9ea8b13f4049d91e2f1a8e8c9eeb97..ee8d416935e1f33e82ec6ffddc34d548017cc76d 100644 (file)
@@ -258,7 +258,7 @@ void BidiRun::operator delete(void* ptr, size_t sz)
 #ifndef NDEBUG
     --BidiRunCounter::count;
 #endif
-    assert(inBidiRunDestroy);
+    ASSERT(inBidiRunDestroy);
 
     // Stash size where destroy() can find it.
     *(size_t*)ptr = sz;
@@ -530,7 +530,7 @@ static void reverseRuns(int start, int end)
     if (start >= end)
         return;
 
-    assert(start >= 0 && end < sBidiRunCount);
+    ASSERT(start >= 0 && end < sBidiRunCount);
     
     // Get the item before the start of the runs to reverse and put it in
     // |beforeStart|.  |curr| should point to the first run to reverse.
@@ -722,9 +722,9 @@ static void embed(Direction d, BidiState& bidi)
         BidiContext *c = bidi.context->parent;
         if (c) {
             if (!emptyRun && bidi.eor != bidi.last) {
-                assert(bidi.status.eor != OtherNeutral);
+                ASSERT(bidi.status.eor != OtherNeutral);
                 // bidi.sor ... bidi.eor ... bidi.last eor; need to append the bidi.sor-bidi.eor run or extend it through bidi.last
-                assert(bidi.status.last == EuropeanNumberSeparator
+                ASSERT(bidi.status.last == EuropeanNumberSeparator
                     || bidi.status.last == EuropeanNumberTerminator
                     || bidi.status.last == CommonNumberSeparator
                     || bidi.status.last == BoundaryNeutral
@@ -782,9 +782,9 @@ static void embed(Direction d, BidiState& bidi)
 
         if (level < 61) {
             if (!emptyRun && bidi.eor != bidi.last) {
-                assert(bidi.status.eor != OtherNeutral);
+                ASSERT(bidi.status.eor != OtherNeutral);
                 // bidi.sor ... bidi.eor ... bidi.last eor; need to append the bidi.sor-bidi.eor run or extend it through bidi.last
-                assert(bidi.status.last == EuropeanNumberSeparator
+                ASSERT(bidi.status.last == EuropeanNumberSeparator
                     || bidi.status.last == EuropeanNumberTerminator
                     || bidi.status.last == CommonNumberSeparator
                     || bidi.status.last == BoundaryNeutral
@@ -910,7 +910,7 @@ RootInlineBox* RenderBlock::constructLine(const BidiIterator& start, const BidiI
 
     // We should have a root inline box.  It should be unconstructed and
     // be the last continuation of our line list.
-    assert(lastLineBox() && !lastLineBox()->isConstructed());
+    ASSERT(lastLineBox() && !lastLineBox()->isConstructed());
 
     // Set bits on our inline flow boxes that indicate which sides should
     // paint borders/margins/padding.  This knowledge will ultimately be used when
@@ -1102,7 +1102,7 @@ void RenderBlock::bidiReorderLine(const BidiIterator& start, const BidiIterator&
     sLastBidiRun = 0;
     sBidiRunCount = 0;
 
-    assert(bidi.dir == OtherNeutral);
+    ASSERT(bidi.dir == OtherNeutral);
 
     emptyRun = true;
 
@@ -1143,7 +1143,7 @@ void RenderBlock::bidiReorderLine(const BidiIterator& start, const BidiIterator&
                 dirCurrent = bidi.status.last;
         }
 
-        assert(bidi.status.eor != OtherNeutral);
+        ASSERT(bidi.status.eor != OtherNeutral);
         switch (dirCurrent) {
 
         // embedding and overrides (X1-X9 in the Bidi specs)
@@ -1395,7 +1395,7 @@ void RenderBlock::bidiReorderLine(const BidiIterator& start, const BidiIterator&
                             bidi.dir = bidi.status.lastStrong == LeftToRight ? LeftToRight : EuropeanNumber;
                             break;
                         default:
-                            assert(false);
+                            ASSERT(false);
                     }
                     appendRun(bidi);
                 }
@@ -2196,7 +2196,7 @@ BidiIterator RenderBlock::findNextLineBreak(BidiIterator &start, BidiState &bidi
             }
         } else if (o->isInlineFlow()) {
             // Only empty inlines matter.  We treat those similarly to replaced elements.
-            assert(!o->firstChild());
+            ASSERT(!o->firstChild());
             tmpW += o->marginLeft() + o->borderLeft() + o->paddingLeft() +
                     o->marginRight() + o->borderRight() + o->paddingRight();
         } else if (o->isReplaced()) {