|
compositor-worker: Remove code from cc_blink
In this CL, the compositor worker code in cc_blink has
been removed, mostly by moving it into
Source/platform/graphics, a more natural home for
compositor-related stuff.
An additional benefit is that much of the code in
public/platform/ is unnecessary now. It was there only
so that it could be implemented by classes in cc_blink.
There was a wrinkle when moving the test: the cc test
machinery expected a base::MessageLoop to have been
constructed to work. This violated the current DEPS
rules for platform/graphics.
I did try to remove this dependency, but the code is
quite coupled and the refactor was not only large, it
resulted in more complex code in cc.
Since platform is in general allowed to talk to base,
it seemed more reasonable to update the DEPS in
platform/graphics/ to allow the use of message_loop.h
in tests.
BUG= 430155
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Committed: https://crrev.com/fd6e4f4a7a3f166de9d2fbc89770b4f2eb90272f
Cr-Commit-Position: refs/heads/master@{#371006}
Total comments: 18
Total comments: 7
Total comments: 4
|
Unified diffs |
Side-by-side diffs |
Delta from patch set |
Stats (+524 lines, -624 lines) |
Patch |
|
D |
cc/animation/layer_tree_mutation.h
|
View
|
|
1 chunk |
+0 lines, -63 lines |
0 comments
|
Download
|
|
M |
cc/animation/mutable_properties.h
|
View
|
1
2
3
4
5
6
|
1 chunk |
+10 lines, -6 lines |
0 comments
|
Download
|
|
M |
cc/blink/BUILD.gn
|
View
|
|
2 chunks |
+0 lines, -5 lines |
0 comments
|
Download
|
|
M |
cc/blink/cc_blink.gyp
|
View
|
|
1 chunk |
+0 lines, -4 lines |
0 comments
|
Download
|
|
M |
cc/blink/cc_blink_tests.gyp
|
View
|
|
1 chunk |
+0 lines, -1 line |
0 comments
|
Download
|
|
D |
cc/blink/web_compositor_mutable_state_impl.h
|
View
|
|
1 chunk |
+0 lines, -46 lines |
0 comments
|
Download
|
|
D |
cc/blink/web_compositor_mutable_state_impl.cc
|
View
|
|
1 chunk |
+0 lines, -72 lines |
0 comments
|
Download
|
|
D |
cc/blink/web_compositor_mutable_state_impl_unittest.cc
|
View
|
|
1 chunk |
+0 lines, -172 lines |
0 comments
|
Download
|
|
D |
cc/blink/web_compositor_mutable_state_provider_impl.h
|
View
|
|
1 chunk |
+0 lines, -44 lines |
0 comments
|
Download
|
|
D |
cc/blink/web_compositor_mutable_state_provider_impl.cc
|
View
|
|
1 chunk |
+0 lines, -34 lines |
0 comments
|
Download
|
|
M |
cc/blink/web_layer_impl.cc
|
View
|
1
2
3
4
5
|
3 chunks |
+0 lines, -31 lines |
0 comments
|
Download
|
|
M |
cc/layers/layer.cc
|
View
|
1
2
3
4
5
|
1 chunk |
+1 line, -1 line |
0 comments
|
Download
|
|
M |
cc/layers/layer_impl.cc
|
View
|
1
2
3
4
5
|
2 chunks |
+2 lines, -2 lines |
0 comments
|
Download
|
|
M |
cc/layers/layer_impl_unittest.cc
|
View
|
1
2
3
4
5
|
2 chunks |
+2 lines, -2 lines |
0 comments
|
Download
|
|
M |
cc/layers/layer_unittest.cc
|
View
|
1
2
3
4
5
|
2 chunks |
+4 lines, -4 lines |
0 comments
|
Download
|
|
M |
cc/trees/layer_tree_impl.cc
|
View
|
1
2
3
4
5
|
1 chunk |
+2 lines, -2 lines |
0 comments
|
Download
|
|
M |
third_party/WebKit/Source/core/css/resolver/StyleAdjuster.cpp
|
View
|
1
2
3
4
5
|
1 chunk |
+0 lines, -1 line |
0 comments
|
Download
|
|
M |
third_party/WebKit/Source/core/dom/CompositorProxiedPropertySet.h
|
View
|
1
2
3
4
5
6
|
2 chunks |
+2 lines, -2 lines |
0 comments
|
Download
|
|
M |
third_party/WebKit/Source/core/dom/CompositorProxiedPropertySet.cpp
|
View
|
1
2
3
4
5
|
3 chunks |
+4 lines, -4 lines |
0 comments
|
Download
|
|
M |
third_party/WebKit/Source/core/dom/CompositorProxy.cpp
|
View
|
1
2
3
4
5
|
8 chunks |
+22 lines, -22 lines |
0 comments
|
Download
|
|
M |
third_party/WebKit/Source/core/dom/Element.cpp
|
View
|
1
2
3
4
5
|
2 chunks |
+3 lines, -2 lines |
0 comments
|
Download
|
|
M |
third_party/WebKit/Source/core/layout/compositing/CompositedLayerMapping.cpp
|
View
|
1
2
3
4
5
|
3 chunks |
+5 lines, -5 lines |
0 comments
|
Download
|
|
M |
third_party/WebKit/Source/core/layout/compositing/PaintLayerCompositor.cpp
|
View
|
1
2
3
4
5
|
2 chunks |
+3 lines, -3 lines |
0 comments
|
Download
|
|
M |
third_party/WebKit/Source/core/paint/PaintLayerScrollableArea.cpp
|
View
|
1
2
3
4
5
|
2 chunks |
+2 lines, -2 lines |
0 comments
|
Download
|
|
M |
third_party/WebKit/Source/platform/BUILD.gn
|
View
|
1
2
3
4
5
|
2 chunks |
+4 lines, -0 lines |
0 comments
|
Download
|
|
M |
third_party/WebKit/Source/platform/blink_platform.gypi
|
View
|
1
2
3
4
5
|
3 chunks |
+6 lines, -1 line |
0 comments
|
Download
|
|
M |
third_party/WebKit/Source/platform/blink_platform_tests.gyp
|
View
|
1
|
1 chunk |
+4 lines, -0 lines |
0 comments
|
Download
|
|
A |
third_party/WebKit/Source/platform/graphics/CompositorMutableProperties.h
|
View
|
1
2
3
4
5
6
|
1 chunk |
+16 lines, -0 lines |
0 comments
|
Download
|
|
A |
third_party/WebKit/Source/platform/graphics/CompositorMutableState.h
|
View
|
1
2
|
1 chunk |
+46 lines, -0 lines |
0 comments
|
Download
|
|
A |
third_party/WebKit/Source/platform/graphics/CompositorMutableState.cpp
|
View
|
1
2
3
|
1 chunk |
+78 lines, -0 lines |
0 comments
|
Download
|
|
A |
third_party/WebKit/Source/platform/graphics/CompositorMutableStateProvider.h
|
View
|
|
1 chunk |
+37 lines, -0 lines |
0 comments
|
Download
|
|
A |
third_party/WebKit/Source/platform/graphics/CompositorMutableStateProvider.cpp
|
View
|
1
2
3
4
|
1 chunk |
+43 lines, -0 lines |
0 comments
|
Download
|
|
A |
third_party/WebKit/Source/platform/graphics/CompositorMutableStateTest.cpp
|
View
|
1
2
3
4
5
6
7
|
1 chunk |
+159 lines, -0 lines |
0 comments
|
Download
|
|
A |
third_party/WebKit/Source/platform/graphics/CompositorMutation.h
|
View
|
1
2
3
4
5
|
1 chunk |
+61 lines, -0 lines |
0 comments
|
Download
|
|
M |
third_party/WebKit/Source/platform/graphics/DEPS
|
View
|
1
2
3
4
5
|
1 chunk |
+1 line, -0 lines |
0 comments
|
Download
|
|
M |
third_party/WebKit/Source/web/tests/CompositorWorkerTest.cpp
|
View
|
1
2
3
4
5
|
2 chunks |
+7 lines, -7 lines |
0 comments
|
Download
|
|
D |
third_party/WebKit/public/platform/WebCompositorMutableProperties.h
|
View
|
|
1 chunk |
+0 lines, -24 lines |
0 comments
|
Download
|
|
D |
third_party/WebKit/public/platform/WebCompositorMutableState.h
|
View
|
|
1 chunk |
+0 lines, -34 lines |
0 comments
|
Download
|
|
D |
third_party/WebKit/public/platform/WebCompositorMutableStateProvider.h
|
View
|
|
1 chunk |
+0 lines, -28 lines |
0 comments
|
Download
|
Total messages: 30 (14 generated)
|