Oilpan: Remove raw pointer to LocalFrame from MediaValuesDynamic

Source/core/css/MediaValuesDynamic.h

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef MediaValuesDynamic_h
 #define MediaValuesDynamic_h
 
 #include "core/css/MediaValues.h"
 
 namespace blink {
@@ skipping 25 matching lines @@
     const String mediaType() const override;
     WebDisplayMode displayMode() const override;
     Document* document() const override;
     bool hasValues() const override;
 
 protected:
     MediaValuesDynamic(LocalFrame*);
 
     // This raw ptr is safe, as MediaValues would not outlive MediaQueryEvaluator, and
     // MediaQueryEvaluator is reset on |Document::detach|.
-    // FIXME: Oilpan: This raw ptr should be changed to a Member when LocalFrame is migrated to the heap.
-    LocalFrame* m_frame;
+    RawPtrWillBePersistent<LocalFrame> m_frame;

[haraken, 2015/07/30 06:08:58]

If the RawPtrWillBePersistent doesn't cause a cycle, I'd change the RawPtr to
RefPtr on non-oilpan.

That said, a better fix would be to move MediaValues to oilpan's heap. If it's
not hard, I'd prefer moving MediaValues to the heap and use Member here.

[keishi, 2015/08/06 06:48:49]

Done.

 };
 
 } // namespace
 
 #endif // MediaValuesDynamic_h