|
|
Have Document::nodeWillBeRemoved() take a reference
Have Document::nodeWillBeRemoved() take a reference instead of a pointer
as it can never be null. This makes the caller simpler as it already
have a Node reference. This also gets rid of several unnecessary null
checks.
R=adamk
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=160986
|
Unified diffs |
Side-by-side diffs |
Delta from patch set |
Stats (+56 lines, -59 lines) |
Patch |
 |
M |
Source/core/dom/ContainerNode.cpp
|
View
|
|
1 chunk |
+1 line, -1 line |
0 comments
|
Download
|
|
M |
Source/core/dom/Document.h
|
View
|
|
1 chunk |
+1 line, -1 line |
0 comments
|
Download
|
|
M |
Source/core/dom/Document.cpp
|
View
|
|
1 chunk |
+5 lines, -5 lines |
0 comments
|
Download
|
|
M |
Source/core/dom/NodeIterator.h
|
View
|
|
2 chunks |
+2 lines, -2 lines |
0 comments
|
Download
|
|
M |
Source/core/dom/NodeIterator.cpp
|
View
|
|
2 chunks |
+13 lines, -14 lines |
0 comments
|
Download
|
|
M |
Source/core/dom/Range.h
|
View
|
|
1 chunk |
+1 line, -1 line |
0 comments
|
Download
|
|
M |
Source/core/dom/Range.cpp
|
View
|
|
4 chunks |
+6 lines, -7 lines |
0 comments
|
Download
|
|
M |
Source/core/dom/RangeBoundaryPoint.h
|
View
|
|
2 chunks |
+5 lines, -6 lines |
0 comments
|
Download
|
|
M |
Source/core/editing/Caret.h
|
View
|
|
1 chunk |
+1 line, -1 line |
0 comments
|
Download
|
|
M |
Source/core/editing/Caret.cpp
|
View
|
|
3 chunks |
+6 lines, -6 lines |
0 comments
|
Download
|
|
M |
Source/core/editing/FrameSelection.h
|
View
|
|
2 chunks |
+2 lines, -2 lines |
0 comments
|
Download
|
|
M |
Source/core/editing/FrameSelection.cpp
|
View
|
|
5 chunks |
+10 lines, -10 lines |
0 comments
|
Download
|
|
M |
Source/core/page/EventHandler.h
|
View
|
|
1 chunk |
+1 line, -1 line |
0 comments
|
Download
|
|
M |
Source/core/page/EventHandler.cpp
|
View
|
|
1 chunk |
+2 lines, -2 lines |
0 comments
|
Download
|
Total messages: 5 (0 generated)
|
Chromium Code Reviews
Issue
48803004:
Have Document::nodeWillBeRemoved() take a reference (Closed)
