Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1027)

Unified Diff: Source/core/frame/LocalFrame.h

Issue 517043003: Move Frame to the Oilpan heap. (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Rebase past r181245 conflict Created 6 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: Source/core/frame/LocalFrame.h
diff --git a/Source/core/frame/LocalFrame.h b/Source/core/frame/LocalFrame.h
index 5a3a8c01177d3e84b4feed8cacd6427352220e48..64157173b30f1225c557bc59a73d2cbf66f45407 100644
--- a/Source/core/frame/LocalFrame.h
+++ b/Source/core/frame/LocalFrame.h
@@ -64,9 +64,9 @@ namespace blink {
class TreeScope;
class VisiblePosition;
- class LocalFrame : public Frame, public WillBePersistentHeapSupplementable<LocalFrame> {
+ class LocalFrame : public Frame, public WillBeHeapSupplementable<LocalFrame> {
public:
- static PassRefPtr<LocalFrame> create(FrameLoaderClient*, FrameHost*, FrameOwner*);
+ static PassRefPtrWillBeRawPtr<LocalFrame> create(FrameLoaderClient*, FrameHost*, FrameOwner*);
virtual bool isLocalFrame() const OVERRIDE { return true; }
@@ -77,6 +77,7 @@ namespace blink {
ScrollbarMode = ScrollbarAuto, bool verticalLock = false);
virtual ~LocalFrame();
+ virtual void trace(Visitor*) OVERRIDE;
virtual void detach() OVERRIDE;
@@ -155,6 +156,8 @@ namespace blink {
bool shouldReuseDefaultView(const KURL&) const;
void removeSpellingMarkersUnderWords(const Vector<String>& words);
+ virtual void dispose() OVERRIDE;
+
// ========
private:
@@ -168,15 +171,15 @@ namespace blink {
RefPtr<FrameView> m_view;
// Usually 0. Non-null if this is the top frame of PagePopup.
- RefPtrWillBePersistent<Element> m_pagePopupOwner;
+ RefPtrWillBeMember<Element> m_pagePopupOwner;
OwnPtr<ScriptController> m_script;
- const OwnPtrWillBePersistent<Editor> m_editor;
- const OwnPtr<SpellChecker> m_spellChecker;
- const OwnPtrWillBePersistent<FrameSelection> m_selection;
- const OwnPtrWillBePersistent<EventHandler> m_eventHandler;
- const OwnPtrWillBePersistent<FrameConsole> m_console;
- OwnPtr<InputMethodController> m_inputMethodController;
+ const OwnPtrWillBeMember<Editor> m_editor;
+ const OwnPtrWillBeMember<SpellChecker> m_spellChecker;
+ const OwnPtrWillBeMember<FrameSelection> m_selection;
+ const OwnPtrWillBeMember<EventHandler> m_eventHandler;
+ const OwnPtrWillBeMember<FrameConsole> m_console;
+ OwnPtrWillBeMember<InputMethodController> m_inputMethodController;
float m_pageZoomFactor;
float m_textZoomFactor;

Powered by Google App Engine
This is Rietveld 408576698