Title: [212174] trunk/Source/WebCore
Revision
212174
Author
dba...@webkit.org
Date
2017-02-10 19:09:10 -0800 (Fri, 10 Feb 2017)

Log Message

Attempt to fix the build following <https://trac.webkit.org/changeset/212173>
(https://bugs.webkit.org/show_bug.cgi?id=166774)

* dom/Document.cpp:
(WebCore::Document::detachFromCachedFrame): Use ASSER_UNUSED() instead of ASSERT() since
the parameter cachedFrame is unused in non-debug build.

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (212173 => 212174)


--- trunk/Source/WebCore/ChangeLog	2017-02-11 02:53:53 UTC (rev 212173)
+++ trunk/Source/WebCore/ChangeLog	2017-02-11 03:09:10 UTC (rev 212174)
@@ -1,5 +1,14 @@
 2017-02-10  Daniel Bates  <daba...@apple.com>
 
+        Attempt to fix the build following <https://trac.webkit.org/changeset/212173>
+        (https://bugs.webkit.org/show_bug.cgi?id=166774)
+
+        * dom/Document.cpp:
+        (WebCore::Document::detachFromCachedFrame): Use ASSER_UNUSED() instead of ASSERT() since
+        the parameter cachedFrame is unused in non-debug build.
+
+2017-02-10  Daniel Bates  <daba...@apple.com>
+
         Detach frame from document when entering page cache
         https://bugs.webkit.org/show_bug.cgi?id=166774
         <rdar://problem/29904368>

Modified: trunk/Source/WebCore/dom/Document.cpp (212173 => 212174)


--- trunk/Source/WebCore/dom/Document.cpp	2017-02-11 02:53:53 UTC (rev 212173)
+++ trunk/Source/WebCore/dom/Document.cpp	2017-02-11 03:09:10 UTC (rev 212174)
@@ -2191,8 +2191,8 @@
 
 void Document::detachFromCachedFrame(CachedFrameBase& cachedFrame)
 {
+    ASSERT_UNUSED(cachedFrame, cachedFrame.view());
     ASSERT_WITH_SECURITY_IMPLICATION(cachedFrame.document() == this);
-    ASSERT(cachedFrame.view());
     ASSERT(m_frame == &cachedFrame.view()->frame());
     ASSERT(m_pageCacheState == Document::InPageCache);
     detachFromFrame();
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to