| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Disclaimer: This CL is only about moving files and sections of code.
We:
- Remove Web prefixes on files/classes and add Compositor prefix where necessary.
- Move exported/WebTransformKeyframe.cpp and exported/WebFilterKeyframe.cpp
to animation/ to group them with their headers.
- Move CompositorFilterOperations not to animation/ but to graphics/ folder,
because it's a property of WebLayer.
- Introduce CompositorFactory singleton (works as Platform) to create CC objects
in blink.
All animation methods are moved from CompositorSupport to CompositorFactory.
Other methods are also expected to be moved there.
- Decouple WebLayer and WebCompositorAnimation interfaces
by using cc::Animation class.
It allows us to make WebCompositorAnimation non-public.
WebLayer animation dependencies will go away when we erase the old animation system
(it's already disabled). http://crbug.com/575041
- Move static asserts from content/child/assert_matching_enums.cc to
Source/platform/WebCompositorAnimation.cpp (so the enum becomes non-public).
What's next (separate CLs):
- Remove virtual functions (when gmock-based tests erased)
- Remove redundant enum declarations (using alias).
- Erase the animation delegate adapter.
BUG=577016
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1616653002
Cr-Commit-Position: refs/heads/master@{#374792}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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
Review URL: https://codereview.chromium.org/1599673002
Cr-Commit-Position: refs/heads/master@{#371006}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This change introduces a wrapper around compositer-owned state that may
be mutated by a compositor worker.
This depends on crrev.com/1405993008
BUG=430155
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1447893002
Cr-Commit-Position: refs/heads/master@{#368273}
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This CLs unblocks submitting crrev.com/1534813004 where running the tests
under fast/scroll-behavior/smooth-scroll would crash the renderer. Turns out
we weren't setting the delegate to null correctly.
BUG=552556
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1547463003
Cr-Commit-Position: refs/heads/master@{#366651}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This CL fixes
WebLayerImplFixedBoundsTest.CompareToWebLayerImplComplex and
WebLayerImplFixedBoundsTest.CompareToWebLayerImplSimple tests.
BUG=440441
R=enne, danakj
Review URL: https://codereview.chromium.org/1130123005
Cr-Commit-Position: refs/heads/master@{#330167}
|
|
|
|
|
|
|
|
|
|
|
|
| |
For tools/sort_sources.py to be useful, existing GYP and GN files
should be sorted.
BUG=456014
TEST=everything should build as before
Review URL: https://codereview.chromium.org/931053002
Cr-Commit-Position: refs/heads/master@{#316866}
|
|
These files provide the implementations of the public Blink API that
bind to the cc implementation. This moves them out of content/ into a
subdirectory of cc/ so that we can reuse these files in mojo which
can depend on cc/ and on blink but not on content/. To make sure we
don't introduce dependencies from cc/ on blink proper, cc/DEPS forbids
including cc/blink/ and cc/blink/DEPS allows using the blink headers.
cc_blink is built as a separate component to avoid linking anything from
blink into targets that don't care about the blink bindings (such as
the browser compositor).
R=darin@chromium.org
Review URL: https://codereview.chromium.org/470983004
Cr-Commit-Position: refs/heads/master@{#290464}
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@290464 0039d316-1c4b-4281-b951-d872f2087c98
|