|
Reland "Streamline frame detach"
Share a bit more code between LocalFrame and RemoteFrame, including moving the detach-time client notification down to FrameClient.
Move the max frame counter to FrameHost instead of Page.
Move detach logic from FrameLoader to LocalFrame, unless it manipulates FrameLoader members.
BUG=
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=183773
Total comments: 2
Total comments: 2
|
Unified diffs |
Side-by-side diffs |
Delta from patch set |
Stats (+80 lines, -108 lines) |
Patch |
 |
M |
Source/core/dom/NodeRareData.cpp
|
View
|
|
2 chunks |
+2 lines, -2 lines |
0 comments
|
Download
|
 |
M |
Source/core/frame/Frame.h
|
View
|
1
|
2 chunks |
+1 line, -7 lines |
0 comments
|
Download
|
 |
M |
Source/core/frame/Frame.cpp
|
View
|
1
2
3
4
|
3 chunks |
+15 lines, -3 lines |
0 comments
|
Download
|
 |
M |
Source/core/frame/FrameClient.h
|
View
|
|
1 chunk |
+2 lines, -0 lines |
0 comments
|
Download
|
 |
M |
Source/core/frame/FrameHost.h
|
View
|
1
2
3
4
5
|
2 chunks |
+10 lines, -0 lines |
0 comments
|
Download
|
 |
M |
Source/core/frame/FrameHost.cpp
|
View
|
1
2
3
4
5
|
2 chunks |
+22 lines, -0 lines |
0 comments
|
Download
|
 |
M |
Source/core/frame/LocalFrame.h
|
View
|
1
2
3
4
|
1 chunk |
+0 lines, -1 line |
0 comments
|
Download
|
 |
M |
Source/core/frame/LocalFrame.cpp
|
View
|
1
2
3
4
5
|
4 chunks |
+11 lines, -15 lines |
0 comments
|
Download
|
 |
M |
Source/core/frame/RemoteFrame.cpp
|
View
|
1
2
3
4
|
1 chunk |
+1 line, -1 line |
0 comments
|
Download
|
 |
M |
Source/core/loader/EmptyClients.h
|
View
|
1
2
3
4
|
1 chunk |
+1 line, -1 line |
0 comments
|
Download
|
 |
M |
Source/core/loader/FrameLoader.h
|
View
|
1
2
3
4
|
2 chunks |
+1 line, -2 lines |
0 comments
|
Download
|
 |
M |
Source/core/loader/FrameLoader.cpp
|
View
|
1
2
3
4
|
1 chunk |
+2 lines, -41 lines |
0 comments
|
Download
|
 |
M |
Source/core/loader/FrameLoaderClient.h
|
View
|
1
2
3
4
|
1 chunk |
+0 lines, -2 lines |
0 comments
|
Download
|
 |
M |
Source/core/page/Page.h
|
View
|
1
2
3
4
5
|
4 chunks |
+0 lines, -17 lines |
0 comments
|
Download
|
 |
M |
Source/core/page/Page.cpp
|
View
|
1
2
3
4
5
|
2 chunks |
+0 lines, -14 lines |
0 comments
|
Download
|
 |
M |
Source/web/FrameLoaderClientImpl.h
|
View
|
1
2
3
4
5
|
1 chunk |
+1 line, -1 line |
0 comments
|
Download
|
 |
M |
Source/web/FrameLoaderClientImpl.cpp
|
View
|
1
2
3
4
5
|
1 chunk |
+1 line, -1 line |
0 comments
|
Download
|
 |
M |
Source/web/RemoteFrameClientImpl.h
|
View
|
|
1 chunk |
+2 lines, -0 lines |
0 comments
|
Download
|
 |
M |
Source/web/RemoteFrameClientImpl.cpp
|
View
|
|
1 chunk |
+5 lines, -0 lines |
0 comments
|
Download
|
 |
M |
Source/web/WebFrame.cpp
|
View
|
1
2
3
4
|
3 chunks |
+3 lines, -0 lines |
0 comments
|
Download
|
Total messages: 14 (2 generated)
|