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

Unified Diff: third_party/WebKit/Source/core/editing/commands/UndoStack.cpp

Issue 1686483002: Oilpan: Remove most WillBe types from the code base (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 9 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: third_party/WebKit/Source/core/editing/commands/UndoStack.cpp
diff --git a/third_party/WebKit/Source/core/editing/commands/UndoStack.cpp b/third_party/WebKit/Source/core/editing/commands/UndoStack.cpp
index bff41c2016664ebe65fd3029aafe315e305c0669..07b20f6748c2ee7cd8d65041d46b5d06f7462722 100644
--- a/third_party/WebKit/Source/core/editing/commands/UndoStack.cpp
+++ b/third_party/WebKit/Source/core/editing/commands/UndoStack.cpp
@@ -44,14 +44,12 @@ UndoStack::UndoStack()
{
}
-DEFINE_EMPTY_DESTRUCTOR_WILL_BE_REMOVED(UndoStack)
-
-PassOwnPtrWillBeRawPtr<UndoStack> UndoStack::create()
+RawPtr<UndoStack> UndoStack::create()
{
- return adoptPtrWillBeNoop(new UndoStack());
+ return new UndoStack();
}
-void UndoStack::registerUndoStep(PassRefPtrWillBeRawPtr<UndoStep> step)
+void UndoStack::registerUndoStep(RawPtr<UndoStep> step)
{
if (m_undoStack.size() == maximumUndoStackDepth)
m_undoStack.removeFirst(); // drop oldest item off the far end
@@ -60,7 +58,7 @@ void UndoStack::registerUndoStep(PassRefPtrWillBeRawPtr<UndoStep> step)
m_undoStack.append(step);
}
-void UndoStack::registerRedoStep(PassRefPtrWillBeRawPtr<UndoStep> step)
+void UndoStack::registerRedoStep(RawPtr<UndoStep> step)
{
m_redoStack.append(step);
}
@@ -98,7 +96,7 @@ void UndoStack::undo()
{
if (canUndo()) {
UndoStepStack::iterator back = --m_undoStack.end();
- RefPtrWillBeRawPtr<UndoStep> step(back->get());
+ RawPtr<UndoStep> step(back->get());
m_undoStack.remove(back);
step->unapply();
// unapply will call us back to push this command onto the redo stack.
@@ -109,7 +107,7 @@ void UndoStack::redo()
{
if (canRedo()) {
UndoStepStack::iterator back = --m_redoStack.end();
- RefPtrWillBeRawPtr<UndoStep> step(back->get());
+ RawPtr<UndoStep> step(back->get());
m_redoStack.remove(back);
ASSERT(!m_inRedo);

Powered by Google App Engine
This is Rietveld 408576698