Oilpan: Fix oilpan builds after r170720

Source/core/animation/KeyframeEffectModel.cpp

Index: Source/core/animation/KeyframeEffectModel.cpp
diff --git a/Source/core/animation/KeyframeEffectModel.cpp b/Source/core/animation/KeyframeEffectModel.cpp
index ff9e597be44a35166d5612ba87e4a9f7f6205c4d..9460170d7beedeab5ccd1269a113e84bd9f8de09 100644
--- a/Source/core/animation/KeyframeEffectModel.cpp
+++ b/Source/core/animation/KeyframeEffectModel.cpp
@@ -36,7 +36,7 @@
 
 namespace WebCore {
 
-bool Keyframe::compareOffsets(const RefPtrWillBeRawPtr<Keyframe>& a, const RefPtrWillBeRawPtr<Keyframe>& b)
+bool Keyframe::compareOffsets(const RefPtrWillBeMember<Keyframe>& a, const RefPtrWillBeMember<Keyframe>& b)

[kouhei (in TOK), 2014/04/03 02:01:05]

Why Member?

[haraken, 2014/04/03 02:02:05]

Because this is a callback function used in std::stable_sort that sorts Members.

[kouhei (in TOK), 2014/04/03 02:03:46]

Got it.

[zerny-chromium, 2014/04/03 05:17:11]

Is the use of members needed in order to satisfy the compiler? Otherwise, using
pointers and relying on the Member->RawPtr conversion would be more consistent.

[haraken, 2014/04/03 06:11:00]

Yes, it's needed for making the compiler happy.

 {
     return a->offset() < b->offset();
 }
@@ -214,7 +214,7 @@ Keyframe::PropertySpecificKeyframe::PropertySpecificKeyframe(double offset, Pass
 {
 }
 
-void KeyframeEffectModelBase::PropertySpecificKeyframeGroup::appendKeyframe(PassOwnPtr<PropertySpecificKeyframe> keyframe)
+void KeyframeEffectModelBase::PropertySpecificKeyframeGroup::appendKeyframe(PassOwnPtrWillBeRawPtr<PropertySpecificKeyframe> keyframe)
 {
     ASSERT(m_keyframes.isEmpty() || m_keyframes.last()->offset() <= keyframe->offset());
     m_keyframes.append(keyframe);