| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
|
|
| |
One of the functions makes changes on a local copy of property trees
instead of the property trees stored on LayerTreeImpl. This fixes that.
BUG=597143
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1834883002
Cr-Commit-Position: refs/heads/master@{#383219}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
When the output surface is lost, the LTHI drops all resources and tiles, and
waits for the next commit for the next draw. This commit might early out with
no updates in which case we will never perform raster new tiles and redraw.
Schedule a draw when a new output is surface is initialized to avoid this.
BUG=597710
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1831083002
Cr-Commit-Position: refs/heads/master@{#383204}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This patch adds the remaining copy constructors for complex classes.
After this patch, it should be possible to enable the heavy class copy
constructor checks by default.
R=thakis@chromium.org, dcheng@chromium.org
TBR=jam@chromium.org
BUG=436357
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1825273002
Cr-Commit-Position: refs/heads/master@{#383131}
|
|
|
|
|
|
|
|
|
|
|
|
| |
This removes a draw property and some setters/getters on Layer
and LayerImpl that were only used by CDP.
BUG=497821
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1829053002
Cr-Commit-Position: refs/heads/master@{#383091}
|
|
|
|
|
|
|
|
|
|
|
|
| |
This CL also makes ValidateRenderSurfaces in DrawPropertyUtils use
Layer Iterator.
BUG=594024,568794
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1832663002
Cr-Commit-Position: refs/heads/master@{#383005}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
When we receive a commit protobuf from the engine, we collect all the
existing Layers in a map and proceed with deserializing the new LayerTree
in the commit protobuf, re-using any existing Layers if possible.
When deserializing a Layer, Layer::FromLayerNodeProto creates a copy of
the existing children, in order to clear the parent pointers for these
layers, if they have now been added to the children of another layer.
However, since we only traverse the layers from the updated LayerTree
during deserialization, if a Layer has been destroyed on the engine, it will
not be traversed during the derserialization step, and will still hold a
reference to all its children. When this Layer is being destroyed on the
client, it will call Layer::RemoveFromParent for all its children, which
may now have been added to the children list of a different Layer.
This patch clears all the properties that are sent with a Layer Hierarchy
update in Layer::ClearLayerHierarchyPropertiesForDeserializationAndAddToMap,
which is run for the existing LayerTree before parsing the new LayerTree from
the commit protobuf.
BUG=590102
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1804703004
Cr-Commit-Position: refs/heads/master@{#382953}
|
|
|
|
|
|
|
|
|
|
| |
This is the update render surfaces treewalk in draw_property_utils.cc
BUG=594024
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1823833002
Cr-Commit-Position: refs/heads/master@{#382767}
|
|
|
|
|
|
|
|
|
|
|
|
| |
And make it run only for multi-threaded case as it relies on pending
tree.
BUG=596880
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1827643002
Cr-Commit-Position: refs/heads/master@{#382736}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This patch adds a WindowTreeHostManager::Observer to exo that
sets the appropriate output secure flag to the compositors
when the display configuration changes.
The first iteration for the logic that determines if the output
is secure is simply a check if the connected display is internal and
mirroring is disabled on all the devices.
Additionally this patch moves the 'is_secure_' flag from the OutputSurface
to LayerTreeImpl, since the lifetime of the latter better matches the
display configuration changes.
BUG=b/27173841,b/27174223
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1821553002
Cr-Commit-Position: refs/heads/master@{#382683}
|
|
|
|
|
|
|
|
|
|
|
| |
BUG=596880
TBR=jaydasika@chromium.org
NOTRY=true
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1821633006
Cr-Commit-Position: refs/heads/master@{#382677}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
I have taken a very simple strategy. I have
retained all LayerImpl functions related to
transfer of ownership (AddChild, etc). These now
update raw pointers on the LayerImpl as well as
updating true ownership by talking to the LTI.
This allows most unit test code to remain as is.
LayerImpl's may be constructed and ownership may
be transferred just as it had been.
BUG=594026
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1801853002
Cr-Commit-Position: refs/heads/master@{#382619}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
1) use SkPictureRecorder::finishRecordingAsPicture() over
endRecordingAsPicture()
2) convert to sk_sp<SkPicture> fields/params where feasible
BUG=skia:5077
R=reed@google.com,danakj@chromium.org,enne@chromium.org
TBR=pdr@chromium.org,alekseys@chromium.org
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1819683002
Cr-Commit-Position: refs/heads/master@{#382563}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This CL :
* Stores layers that need to push properties in LayerTreeHost(layer_set)
* Deletes bools needs_push_properties and dependants_needs_push_properties
from Layer.
* Iterate the layer_set tp push properties during commit.
* Changes what layers are serialized (Both dirty layers and their ancestors
are serialized currently. With this CL, only dirty layers serialize).
BUG=568874
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1808373002
Cr-Commit-Position: refs/heads/master@{#382379}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Adds a new context type for Pepper GL contexts. This allows us to
fine tune driver bug workarounds for pepper vs. non-pepper contexts.
Currently used to disable the Intel MSAA blacklist on pepper contexts,
as it should be up to the app whether to use MSAA.
BUG=527565
CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel
Review URL: https://codereview.chromium.org/1781093002
Cr-Commit-Position: refs/heads/master@{#382097}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
For CalculateRenderTarget and CalculateRenderSurfaceLayerList, no longer
need to redirect call through to Internal version of function, they can
both be called directly.
R=enne
BUG=
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1806343004
Cr-Commit-Position: refs/heads/master@{#381983}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
We use fixed color format (kN32_SkColorType) to decode images. Since the default
preferred format is 16-bit color on low-end devices, we can reduce memory
footprint by storing decoded image in 16-bit color format when we use the
default settings for the preferred format. This CL makes
SoftwareImageDecodeController allocate memory based on the preferred format.
I ran memory.blink_memory_mobile 20 times. The results can be found at [1].
I removed SimulateMemoryPressureNotification() so that I can observe memory usage in
common patterns (still, memory pressure notifications would be sent if memory
usage is high). According to the results, we could reduce ~40% discardable
memory on average.
Note that Blink still uses fixed 32-bit color for decoding. This means that
skia/cc will convert decoded images when we store decoded image in 16-bit color.
We might want to make Blink's decoding format configurable in the future.
[1] https://drive.google.com/file/d/0B6NYyLPujP4TaWItc3lsZXR6TTg/view?usp=sharing
BUG=519146
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1808633002
Cr-Commit-Position: refs/heads/master@{#381859}
|
|
|
|
|
|
|
|
|
| |
BUG=568830
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1805343006
Cr-Commit-Position: refs/heads/master@{#381614}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
To avoid cc::Scheduler having to sometimes own / sometimes create /
switch between sources / manage parameters on sources, hoist all this
logic up to the owners of cc::Scheduler.
This refactoring makes it possible to eventually allow an OutputSurface
to set a BeginFrameSource on its client and then forward that to the
scheduler to use. That future is made easier by the scheduler having a
single begin frame source managed by its owner.
This patch adds a bit of duplicated logic in three places (single thread
proxy / threaded proxy / tests) for the creation of the begin frame
source. However, the single thread and threaded versions were already
doing different things (re: authoritative vsync signals). The hope
is that the vsync information and begin frame source creation for the
synthetic sources can get further hoisted into OutputSurface itself.
Depends on https://codereview.chromium.org/1762823002
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1765723002
Cr-Commit-Position: refs/heads/master@{#381558}
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Accidentally didn't set clip rect to empty on previous clean up patch.
Fix that and also add a unittest.
R=ajuma
BUG=
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1805703003
Cr-Commit-Position: refs/heads/master@{#381477}
|
|
|
|
|
|
|
|
|
|
|
|
| |
* Move DidUpdateScrollOffset to LayerTreeImpl
* Make SyncedScrollOffset transparent to LayerImpl
BUG=568830
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1803643002
Cr-Commit-Position: refs/heads/master@{#381467}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
We use disable-threaded-animation command line switch for
testing purposes.
Now blink knows ahead of time, whether we support threaded animations.
See
RendererBlinkPlatformImpl::isThreadedAnimationEnabled()
BUG=575041
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1798503004
Cr-Commit-Position: refs/heads/master@{#381192}
|
|
|
|
|
|
|
|
|
|
|
|
| |
To align it with other animation interactions so LayerImpl doesn't depend on
AnimationHost interface.
BUG=575041
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1793313003
Cr-Commit-Position: refs/heads/master@{#381181}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This feature was added to compensate for fixed-position layers getting
positioned at integer positions but counterscrolled by a full,
fractional amount. It wound up getting unused because it caused a
performance regression due to defeating the "aborted commit"
optimization during scrolling (every scroll increment with a fractional
change would cause a commit).
There are two ways we can move forward with main-thread fractional
scroll support from here, either 1) have CC automatically compute the
fractional part on the impl thread, or 2) change Blink so that
fixed-position elements don't include the scroll offset in their layout
position at all. Either way, this scroll compensation feature is dead
code, so we can delete it now.
BUG=414283
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1385123003
Cr-Commit-Position: refs/heads/master@{#381180}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
When a clip is created by a render surface, it uses only parent clip.
When this render surface doesn't apply clip, this clip node should
inherit parent clip's |clip_in_target_space| adjusted to current target
space.
R=ajuma
BUG=594675
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1802753002
Cr-Commit-Position: refs/heads/master@{#381080}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
One of the LayerTreeImpl unittest did not set up clip children correct.
To set up clip children, clip child layer need to set its clip parent,
and also clip parent layer need to update its clip children list.
R=ajuma
BUG=
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1800813002
Cr-Commit-Position: refs/heads/master@{#381072}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Scroll children's effect tree index should be from their parent, not
scroll parent. Correct this.
This went undetected because we don't really exercise layer's effect
tree index as much. Previous code still set target correctly.
R=ajuma
BUG=521717
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1796793004
Cr-Commit-Position: refs/heads/master@{#381058}
|
|
|
|
|
|
|
|
|
|
|
| |
Also, erase Layer::IsActive method (it is meaningless and not used anywhere)
BUG=575041
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1797703002
Cr-Commit-Position: refs/heads/master@{#380926}
|
|
|
|
|
|
|
|
|
|
|
| |
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}
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
For layers owning a render surface that has unclipped descendants, we
used to not respect clip even when layer has mask to bounds.
R=ajuma
BUG=
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1790563004
Cr-Commit-Position: refs/heads/master@{#380858}
|
|
|
|
|
|
|
|
|
|
|
|
| |
LayerTreeImpl will iterate the list of layers and reset change tracking
on individual layers.
BUG=568817
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1778163002
Cr-Commit-Position: refs/heads/master@{#380817}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Since mouse wheel events will use gestures. Make it clear that this
scroll input type was really referring to touchscreen.
Rename executed via:
sed -i 's/InputHandler::GESTURE/InputHandler::TOUCHSCREEN/g'
git cl format
BUG=593017
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1775013004
Cr-Commit-Position: refs/heads/master@{#380742}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Create wheel based gestures from mouse wheel event phases when
available. Pass overscroll into the input elasticity controller
to handle the elastic overflow scroll on OSX.
Transition the FrameWatcher to be a global IPC Message filter so
that it gets first crack at the messages (it is a passive
listener); because the RenderBrowserFilter was consuming the
FrameSwap messages causing the browser test to always fail on Mac.
BUG=587979
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1749343004
Cr-Commit-Position: refs/heads/master@{#380639}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Basically, specialize all the code as if:
- use_compositor_animation_timelines is always true.
- Layer::layer_animation_controller_ is always nullptr.
- LayerImpl::layer_animation_controller_ is always nullptr.
- LayerTreeHost::animation_registrar_ is always nullptr.
- LayerTreeHostImpl::animation_registrar_ is always nullptr.
Next CL: Erase cc::LayerSettings everywhere.
BUG=575041
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1782433002
Cr-Commit-Position: refs/heads/master@{#380576}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Merge tests from layer_tree_host_unittest_animation_timelines.cc
onto their respective layer_tree_host_unittest_animation.cc entries.
Erase layer_tree_host_unittest_animation_timelines.cc.
BUG=575041
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1766483002
Cr-Commit-Position: refs/heads/master@{#380343}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The getter ScrollDelta is left because it is called by
property tree builder.
LayerImpl is now not aware of the internal logic of synced
property.
BUG=568830
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1766053002
Cr-Commit-Position: refs/heads/master@{#380217}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This was added in https://codereview.chromium.org/811523002, but was
never actually hooked up to anything. So, time to remove it. Sorry
year old feature that never got finished, your time can come again.
This is just scheduler cleanup to make it possible to do further begin
frame source refactoring. The followups to this patch are to make
scheduler not own any begin frame source (always external, from its
perspective) and then make it possible to remove/change the begin frame
source the scheduler is using.
Review URL: https://codereview.chromium.org/1762823002
Cr-Commit-Position: refs/heads/master@{#380000}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
In the remote compositor we can get into a state where the LTH on the
client requests a BeginMainFrame and then releases the output surface.
The server will still send a commit but we can't push this commit till
the client initializes a new output surface.
At the same time the scheduler will not request a new output surface
till it clears the pipeline of the previous commit.
This change adds a setting to the Scheduler to allow it to request a new
output surface, while there is a commit pending. The RemoteChannelImpl
queues any protos received if the output is released and process them
when a new output surface is initialized.
BUG=586210
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1691143002
Cr-Commit-Position: refs/heads/master@{#379947}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This CL :
* Stores layers that need to push properties on LayerTreeImpl(layer_set)
* Deletes bools needs_push_properties and dependants_needs_push_properties
from LayerImpl
* Iterate the layer_set to push propeties while activating pending tree.
BUG=568874
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1770913003
Cr-Commit-Position: refs/heads/master@{#379727}
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
We only use a layer's clip_rect when layer is_clipped. Do not set
clip_rect when layer is not clipped.
BUG=581843
R=ajuma
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1763873002
Cr-Commit-Position: refs/heads/master@{#379151}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Add missing tests from layer_tree_host_unittest_animation.cc
to layer_tree_host_unittest_animation_timelines.cc
(and port them as well)
LayerTreeHostTimelinesTestPendingTreeAnimatesFirstCommit
LayerTreeHostTimelinesTestAnimatedLayerRemovedAndAdded
LayerTreeHostTimelinesTestIsAnimating
LayerTreeHostTimelinesTestNotifyAnimationFinished
Evolved from:
LayerTreeHostAnimationTestPendingTreeAnimatesFirstCommit
LayerTreeHostAnimationTestAnimatedLayerRemovedAndAdded
LayerTreeHostAnimationTestIsAnimating
LayerTreeHostAnimationTestNotifyAnimationFinished
BUG=575041
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1758583002
Cr-Commit-Position: refs/heads/master@{#379126}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Now updating scrolling information on impl side can be independent of layer impl.
There are still some left-over changes corresponding to NoteLayerPropertiesChanged.
SyncedScrollOffset of scrollable layers are now stored in property trees instead of
layer impl. The main thread property tree has one copy while pending and active share
one.
BUG=568830
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1736073002
Cr-Commit-Position: refs/heads/master@{#379116}
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Now we are always using property trees, the names saying from property
trees are redundant.
BUG=
R=enne
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1751983002
Cr-Commit-Position: refs/heads/master@{#379042}
|
|
|
|
|
|
|
|
|
|
|
| |
Dead code because CDP is gone
BUG=581832
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1764483002
Cr-Commit-Position: refs/heads/master@{#379027}
|
|
|
|
|
|
|
|
|
| |
BUG=568817
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1757303002
Cr-Commit-Position: refs/heads/master@{#379018}
|
|
|
|
|
|
|
|
|
| |
BUG=557194,591512
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1757073002
Cr-Commit-Position: refs/heads/master@{#379009}
|
|
|
|
|
|
|
|
|
|
|
|
| |
This also allows for some pretty cute ways to iterate. See some
examples in the unit test.
BUG=None
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1752263003
Cr-Commit-Position: refs/heads/master@{#378967}
|
|
|
|
|
|
|
|
|
|
| |
This is to remove its dependancy on the layer hierarchy on the impl side
BUG=568822
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1753423002
Cr-Commit-Position: refs/heads/master@{#378940}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This patch separates ImageDecodeController into the software and gpu
components, which are created when tile manager resources are set.
This allows independent development of the two paths without extra
complications involved trying to handle both cases in the same class.
R=enne, ericrk
BUG=581423
CQ_EXTRA_TRYBOTS=tryserver.blink:linux_blink_rel
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1642803003
Cr-Commit-Position: refs/heads/master@{#378895}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This fixes cases where renaming a method from thisStyle to ThisStyle
makes it collide with something, usually another method with the same
name.
For the case in PrintStream.h, the collision function is actually dead
code so remove it.
TBR=avi,pdr
R=pdr
BUG=584539
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1753023002
Cr-Commit-Position: refs/heads/master@{#378868}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This CL also :
* tracks filter changes on main thread and filter
animations on effect tree.
* tracks scroll offset changes on transform tree.
* tracks bounds delta changes on property trees.
BUG=584512
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1752993002
Cr-Commit-Position: refs/heads/master@{#378824}
|