Source/core/frame/PlatformEventDispatcher.h - Issue 1301543002: [Oilpan] Remove a raw pointer to PlatformEventController class on Oilpan build

Keyboard Shortcuts

	File
u :	up to issue
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line

	Issue
u :	up to list of issues
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue

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

Issue 1301543002: [Oilpan] Remove a raw pointer to PlatformEventController class on Oilpan build (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master

Patch Set: Created 5 years, 4 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/PlatformEventDispatcher.h

diff --git a/Source/core/frame/PlatformEventDispatcher.h b/Source/core/frame/PlatformEventDispatcher.h

index 18f728cf205113a2617eb3eff23ec8db9848b2fd..8535a03aae071e9bec70c3045a310ec8f48613c0 100644

--- a/Source/core/frame/PlatformEventDispatcher.h

+++ b/Source/core/frame/PlatformEventDispatcher.h

@@ -30,13 +30,7 @@ protected:

private:

void purgeControllers();

-#if ENABLE(OILPAN)

- void clearWeakMembers(Visitor*);

-#endif

- WillBeHeapVector<PlatformEventController*> m_controllers;

- bool m_needsPurge;

- bool m_isDispatching;

haraken 2015/08/19 11:12:51 We cannot simply remove the purge logic. See my co

peria 2015/08/20 09:35:50 Acknowledged.

+ WillBeHeapHashSet<RawPtrWillBeWeakMember<PlatformEventController>> m_controllers;

};

} // namespace blink

« no previous file with comments | « no previous file | Source/core/frame/PlatformEventDispatcher.cpp » ('j') | Source/core/frame/PlatformEventDispatcher.cpp » ('J')