| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
|
| |
Also, erase LayerTreeSettings::use_compositor_animation_timelines flag.
BUG=575041
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1783613004
Cr-Commit-Position: refs/heads/master@{#380914}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
- Added RemoteChannelMain and RemoteChannelImpl to serialize and transmit
inter-proxy messages between the main and impl components of the
compositor using protobufs.
- Added RemoteChannelHost to be used by the cc embedder as the API for
interacting with the impl components of the remote compositor.
- Added tests to verify the initialization and shutdown sequence for the
remote compositor.
BUG=550687
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1513643010
Cr-Commit-Position: refs/heads/master@{#372236}
|
|
|
|
|
|
|
|
|
|
|
|
| |
This fixes the data race in ThreadProxyTestCommitWaitsForActivation.
Reported by TSAN.
BUG=437454
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1486253002
Cr-Commit-Position: refs/heads/master@{#369940}
|
|
|
|
|
|
|
|
|
|
| |
BUG=138542
TBR=danakj@chromium.org
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1539203002
Cr-Commit-Position: refs/heads/master@{#366333}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This is the final patch that splits ThreadProxy into ProxyMain and
ProxyImpl routing all inter-proxy communication using ChannelMain
and ChannelImpl.
ThreadProxy currently implements the logic for glueing together the
Scheduler, LayerTreeHostImpl and the LayerTreeHost and separating these
components across the main and impl thread boundary. This patch isolates
the logic for this glue code in ThreadProxy exclusive to each thread to
ProxyMain and ProxyImpl. This will allow us to abstract the medium the
2 sides use to communicate with each other so these components can be
run across a thread/process/network boundary.
ThreadedChannel implements the in-process threaded case.
BUG=527200
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1417053005
Cr-Commit-Position: refs/heads/master@{#364034}
|
|
|
|
|
|
|
|
|
|
|
|
| |
Update LayerTreeTest::RunTest to use CompositorMode to configure the
compositor mode in tests.
BUG=550687
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1476873004
Cr-Commit-Position: refs/heads/master@{#363937}
|
|
|
|
|
|
|
|
|
|
|
|
| |
To do this, introduces a new render process shared setting for synchronized
painting.
BUG=552545
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1452353002
Cr-Commit-Position: refs/heads/master@{#361282}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The LayerTreeHost uses the abstract part of Proxy to access the impl
side of the compositor and the TaskRunnerProvider to access the main
task runner and assertion checks.
Decoupling these 2 will allow the LayerTreeHost used only for
deserialization and commits on the client to be built without a proxy.
This is a follow-up patch to: https://codereview.chromium.org/1418953002/
BUG=527200
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1419283002
Cr-Commit-Position: refs/heads/master@{#359196}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
ProxyImpl uses the LayerTreeHost for 2 blocking operations:
initialization and commit. Pass this through the channel to make
the main state seen by the impl side for an operation explicit.
Also route post tasks for initialization and shutdown through the channel.
This is a follow-up patch to: https://codereview.chromium.org/1377063003/
BUG= 527200
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1418953002
Cr-Commit-Position: refs/heads/master@{#357759}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
If no-one has requested a layer update, either by calling
Proxy::SetNeedsUpdateLayers() or Proxy::SetNeedsCommit(), skip computing
the layer update in BeginMainFrame. This reduces the static overhead of
a BeginMainFrame/Commit operation when the layer tree doesn't change.
This change improves efficiency, for example, when a page has a
requestAnimationFrame callback which does not modify the DOM.
BUG=490465
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1255863004
Cr-Commit-Position: refs/heads/master@{#343004}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
While doing so I audited a bunch of MULTI_THREAD only tests and turned
them into SINGLE_AND_MULTI_THREAD or documented why not to. Some of the
NOIMPL tests really are NOIMPL and I deleted them. The others I turned
on with impl-side instead.
NOTE TO PERF SHERIFFS: This changes the names of some cc_perftests and
makes some of them use impl-side painting now. If this has perf side-
effects these should be considered new (more accurate) baselines.
R=enne@chromium.org, enne, vmpstr
BUG=413479
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1180613010.
Cr-Commit-Position: refs/heads/master@{#334286}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The Google C++ style guide states:
Explicitly annotate overrides of virtual functions or virtual
destructors with an override or (less frequently) final specifier.
Older (pre-C++11) code will use the virtual keyword as an inferior
alternative annotation. For clarity, use exactly one of override,
final, or virtual when declaring an override.
To better conform to these guidelines, the following constructs have
been rewritten:
- if a base class has a virtual destructor, then:
virtual ~Foo(); -> ~Foo() override;
- virtual void Foo() override; -> void Foo() override;
- virtual void Foo() override final; -> void Foo() final;
This patch was automatically generated. The clang plugin can generate
fixit hints, which are suggested edits when it is 100% sure it knows how
to fix a problem. The hints from the clang plugin were applied to the
source tree using the tool in https://codereview.chromium.org/598073004.
Several formatting edits by clang-format were manually reverted, due to
mangling of some of the more complicate IPC macros.
BUG=417463
Review URL: https://codereview.chromium.org/818033004
Cr-Commit-Position: refs/heads/master@{#309520}
|
|
|
|
|
|
|
|
|
| |
BUG=417463
TBR=enne@chromium.org
Review URL: https://codereview.chromium.org/645853008
Cr-Commit-Position: refs/heads/master@{#300439}
|
|
|
|
|
|
|
|
| |
BUG=417463
Review URL: https://codereview.chromium.org/628443002
Cr-Commit-Position: refs/heads/master@{#298271}
|
|
This is initial patch for proxy unittests.
New testing stubs are added to LayerTreeTest.
R=brianderson@chromium.org, danakj@chomium.org
BUG=356832
TEST=*ProxyTest*
Review URL: https://codereview.chromium.org/242783003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@272051 0039d316-1c4b-4281-b951-d872f2087c98
|