Modified: trunk/Source/_javascript_Core/ChangeLog (102281 => 102282)
--- trunk/Source/_javascript_Core/ChangeLog 2011-12-08 00:25:17 UTC (rev 102281)
+++ trunk/Source/_javascript_Core/ChangeLog 2011-12-08 00:28:55 UTC (rev 102282)
@@ -1,3 +1,14 @@
+2011-12-07 Sheriff Bot <webkit.review....@gmail.com>
+
+ Unreviewed, rolling out r102267.
+ http://trac.webkit.org/changeset/102267
+ https://bugs.webkit.org/show_bug.cgi?id=74032
+
+ Breaks build on Chromium Mac Debug (Requested by aklein on
+ #webkit).
+
+ * wtf/HashTraits.h:
+
2011-12-07 Adam Klein <ad...@chromium.org>
Use HashMap<Node*, OwnPtr<...>> in ChildListMutationScope
Modified: trunk/Source/_javascript_Core/wtf/HashTraits.h (102281 => 102282)
--- trunk/Source/_javascript_Core/wtf/HashTraits.h 2011-12-08 00:25:17 UTC (rev 102281)
+++ trunk/Source/_javascript_Core/wtf/HashTraits.h 2011-12-08 00:28:55 UTC (rev 102282)
@@ -117,7 +117,6 @@
typedef PassOwnPtr<P> PassOutType;
static PassOwnPtr<P> passOut(OwnPtr<P>& value) { return value.release(); }
- static PassOwnPtr<P> passOut(std::nullptr_t) { return nullptr; }
typedef typename OwnPtr<P>::PtrType PeekType;
static PeekType peek(const OwnPtr<P>& value) { return value.get(); }
Modified: trunk/Source/WebCore/ChangeLog (102281 => 102282)
--- trunk/Source/WebCore/ChangeLog 2011-12-08 00:25:17 UTC (rev 102281)
+++ trunk/Source/WebCore/ChangeLog 2011-12-08 00:28:55 UTC (rev 102282)
@@ -1,3 +1,18 @@
+2011-12-07 Sheriff Bot <webkit.review....@gmail.com>
+
+ Unreviewed, rolling out r102267.
+ http://trac.webkit.org/changeset/102267
+ https://bugs.webkit.org/show_bug.cgi?id=74032
+
+ Breaks build on Chromium Mac Debug (Requested by aklein on
+ #webkit).
+
+ * dom/ChildListMutationScope.cpp:
+ (WebCore::MutationAccumulationRouter::MutationAccumulationRouter::childAdded):
+ (WebCore::MutationAccumulationRouter::MutationAccumulationRouter::willRemoveChild):
+ (WebCore::MutationAccumulationRouter::MutationAccumulationRouter::incrementScopingLevel):
+ (WebCore::MutationAccumulationRouter::MutationAccumulationRouter::decrementScopingLevel):
+
2011-12-07 Kentaro Hara <hara...@chromium.org>
REGRESSION (r95249): Right side can be truncated when printing
Modified: trunk/Source/WebCore/dom/ChildListMutationScope.cpp (102281 => 102282)
--- trunk/Source/WebCore/dom/ChildListMutationScope.cpp 2011-12-08 00:25:17 UTC (rev 102281)
+++ trunk/Source/WebCore/dom/ChildListMutationScope.cpp 2011-12-08 00:28:55 UTC (rev 102282)
@@ -40,7 +40,7 @@
#include "Node.h"
#include "StaticNodeList.h"
#include <wtf/HashMap.h>
-#include <wtf/OwnPtr.h>
+#include <wtf/RefCounted.h>
namespace WebCore {
@@ -50,7 +50,7 @@
// and precede any additions. If this is violated (i.e. because of code changes elsewhere
// in WebCore) it will likely result in both (a) ASSERTions failing, and (b) mutation records
// being enqueued for delivery before the outer-most scope closes.
-class ChildListMutationAccumulator {
+class ChildListMutationAccumulator : public RefCounted<ChildListMutationAccumulator> {
WTF_MAKE_NONCOPYABLE(ChildListMutationAccumulator);
public:
ChildListMutationAccumulator(PassRefPtr<Node>, PassOwnPtr<MutationObserverInterestGroup> observers);
@@ -95,7 +95,7 @@
typedef HashMap<Node*, unsigned> ScopingLevelMap;
ScopingLevelMap m_scopingLevels;
- HashMap<Node*, OwnPtr<ChildListMutationAccumulator> > m_accumulations;
+ HashMap<Node*, RefPtr<ChildListMutationAccumulator> > m_accumulations;
static MutationAccumulationRouter* s_instance;
};
@@ -216,7 +216,7 @@
void MutationAccumulationRouter::childAdded(Node* target, Node* child)
{
- HashMap<Node*, OwnPtr<ChildListMutationAccumulator> >::iterator iter = m_accumulations.find(target);
+ HashMap<Node*, RefPtr<ChildListMutationAccumulator> >::iterator iter = m_accumulations.find(target);
ASSERT(iter != m_accumulations.end());
if (iter->second)
@@ -225,7 +225,7 @@
void MutationAccumulationRouter::willRemoveChild(Node* target, Node* child)
{
- HashMap<Node*, OwnPtr<ChildListMutationAccumulator> >::iterator iter = m_accumulations.find(target);
+ HashMap<Node*, RefPtr<ChildListMutationAccumulator> >::iterator iter = m_accumulations.find(target);
ASSERT(iter != m_accumulations.end());
if (iter->second)
@@ -242,9 +242,9 @@
OwnPtr<MutationObserverInterestGroup> observers = MutationObserverInterestGroup::createForChildListMutation(target);
if (observers->isEmpty())
- m_accumulations.set(target, nullptr);
+ m_accumulations.set(target, 0);
else
- m_accumulations.set(target, adoptPtr(new ChildListMutationAccumulator(target, observers.release())));
+ m_accumulations.set(target, adoptRef(new ChildListMutationAccumulator(target, observers.release())));
}
void MutationAccumulationRouter::decrementScopingLevel(Node* target)
@@ -258,7 +258,7 @@
m_scopingLevels.remove(iter);
- OwnPtr<ChildListMutationAccumulator> record = m_accumulations.take(target);
+ RefPtr<ChildListMutationAccumulator> record = m_accumulations.take(target);
if (record)
record->enqueueMutationRecord();
}
