| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Fields were getting duplicated for scrollbars. Mark the entire
inner viewport as blocked on scroll if we have a wheel listener
we don't need to iterate the tree.
This was reverted in https://codereview.chromium.org/1816723003 because
the sheriff thought it caused a build failure; but the actual failure was
from merge of https://chromium.googlesource.com/chromium/src/+/58cef5be289f5004932682c17694aaa7724bae70
and subsequent fix https://chromium.googlesource.com/chromium/src/+/98aba017dee34d0c08df009095e1fe7bbd6acb49
BUG=595591
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
TBR=aelias@chromium.org
Review URL: https://codereview.chromium.org/1816083002
Cr-Commit-Position: refs/heads/master@{#382295}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
of https://codereview.chromium.org/1810363002/ )
Reason for revert:
Failing on continuous builders.
https://chromegw.corp.google.com/i/official.desktop.continuous/builders/win%20stable/builds/464/steps/compile/logs/stdio
BUG=596215
Original issue's description:
> Fix display of wheel event listeners on a page.
>
> Fields were getting duplicated for scrollbars. Mark the entire
> inner viewport as blocked on scroll if we have a wheel listener
> we don't need to iterate the tree.
>
> BUG=595591
> CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
>
> Committed: https://crrev.com/8f35ce554d594f2069d7f4515602e3fb6a5fc5a9
> Cr-Commit-Position: refs/heads/master@{#382053}
TBR=aelias@chromium.org,dtapuska@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=595591
Review URL: https://codereview.chromium.org/1816723003
Cr-Commit-Position: refs/heads/master@{#382131}
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Fields were getting duplicated for scrollbars. Mark the entire
inner viewport as blocked on scroll if we have a wheel listener
we don't need to iterate the tree.
BUG=595591
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1810363002
Cr-Commit-Position: refs/heads/master@{#382053}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Update.
This fixes the referenced bug, as well as cleaning up the interaction between
PictureLayer and DisplayListRecordingSource a little bit. This way both the
invalidation and picture come from DisplayListRecordingSource, rather than one
coming from PictureLayer and the other from DisplayListRecordingSource, with the
latter modifying the invalidation during Update.
Also renamed LayerImpl::GetInvalidationRegion() to
LayerImpl::GetInvalidationRegionForDebugging() to make clear it's only used for
that purpose.
BUG=591561
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1812733003
Cr-Commit-Position: refs/heads/master@{#381977}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This patch moves SkipImageCanvas into a separate file in the same
directory as ImageHijackCanvas and also moves the creation of
SkipImageCanvas to raster source playback (the same place as
ImageHijackCanvas is created).
BUG=594679
R=enne, ericrk
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1799883002
Cr-Commit-Position: refs/heads/master@{#381101}
|
|
|
|
|
|
|
|
|
|
|
| |
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}
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This patch eliminates the conversion to a raster source raw pointer
when we're passing this down to a raster task, since the raster task
needs to have partial ownership of it.
R=danakj
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1777383002
Cr-Commit-Position: refs/heads/master@{#380853}
|
|
|
|
|
|
|
|
|
| |
BUG=434699
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1774533002
Cr-Commit-Position: refs/heads/master@{#379582}
|
|
|
|
|
|
|
|
|
| |
BUG=591708
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1754333003
Cr-Commit-Position: refs/heads/master@{#379034}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Summary. This CL:
- Makes TraceEvent ownership a move-only scoped_ptr.
- Makes ConvertableToTraceFormat (CTTF) itself move-only scoped_ptr.
- Updates all the codebase that uses CTTF in TRACE_EVENT macros
to use move-only semantics.
Background:
Historically ConvertableToTraceFormat (CTTF) was RefCounted. The main
reason seems to be supporting monitoring mode (now deprecated) where
tracing needed to copy TraceEvents without flushing the TraceLog.
Not what monitoring mode is gone, there is no reason why TraceEvent(s)
should not be move-only.
Unfortunately CTTF being RefCounted exposed that implementation detail
to its public interface. Fortunately, most of the codebase doesn't care
about the fact that CTTF is RefCounted.
The only exceptions are:
1. Memory-infra heap profiler {StackFrame,TypeInfo}Deduplicator
2. cc::Layer DebugInfo
1) Is addressed creating a proxy class which delegates the CTTF methods
to the duplicators inside MDSessionState. Essentially it makes
the CTTF metadata events shared co-owners of the MDSessionState.
2) After an offline chat with danakj@, it seems OK to make DebugInfo(s)
moved scoped_ptr (as opposite as copied), moving the ownership to the
active layer and keeping a raw ptr into the pending layer.
BUG=559117
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
TBR=thakis,jochen,tbarzic,mnaganov,skyostil
Review URL: https://codereview.chromium.org/1717283003
Cr-Commit-Position: refs/heads/master@{#378263}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This patch adds out of line copy constructors for classes that our
clang-plugin considers heavy. This is an effort to enable copy
constructor checks by default.
BUG=436357
R=enne@chromium.org, dcheng@chromium.org, thakis@chromium.org
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1732573002
Cr-Commit-Position: refs/heads/master@{#377462}
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
GraphicsLayer::paint may not be a no-op if there happened to be dirty layout or
paint since the last time a synchronized paint occurred.
BUG=585694
TBR=ajuma@chromium.org
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1690763003
Cr-Commit-Position: refs/heads/master@{#374864}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
cc does nothing with these new fields being set other than plumbing them
through the pipeline to be later queried on the layer_impl.
Add an additional field whether touch event listeners exist on the layer
(which unfortunately is effectively implement as an entire document field
in the blink EventListenerRegistry; this can be improved). There still
exists TouchRegionRects that indicate the areas of blocking event
listeners; but whether a passive event listener exists will be indicated
in this field. It is intended that this model will be replicated
with property trees when it is ready.
BUG=489802
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1577263004
Cr-Commit-Position: refs/heads/master@{#373915}
|
|
|
|
|
|
|
|
|
|
|
|
| |
As similarily done with wheel event handlers move the scroll event
handlers to the layer tree view.
BUG=489802
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1646603004
Cr-Commit-Position: refs/heads/master@{#373112}
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Since having wheel event handlers was based on the document in general
there was no need for the hit testing code for determining the layer in
cc. So it makes more sense to have these fields on the layer tree view.
BUG=489802
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1639363002
Cr-Commit-Position: refs/heads/master@{#372346}
|
|
|
|
|
|
|
|
|
|
|
|
| |
This removes all uses of base::hash_*, BASE_HASH_NAMESPACE, and
base::ScopedPtrHashMap in favor of the C++11 versions.
BUG=576864, 579229
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1587283002
Cr-Commit-Position: refs/heads/master@{#371937}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This is a reland of https://codereview.chromium.org/1502373009 with some fixes
for components/metrics/leak_detector allocator type mismatches.
Original issue's description:
> Allow std::unordered_*.
>
> base::hash_* is, as a transition step, implemented in terms of
> std::unordered_*. Later commits will convert existing uses.
>
> Also fix a host of IWYU problems that arose from this CL.
>
> (NOPRESUBMIT because the wstring presubmit check is overzealous
> and complains about the reference to wstring in the comment.)
>
> Committed: https://crrev.com/3f37f7f1459e7b5a452c0e433493e0a6e9649ca7
> Cr-Commit-Position: refs/heads/master@{#370553}
BUG=576864
TBR=derat@chromium.org,danakj@chromium.org,dalecurtis@chromium.org,jbauman@chromium.org,blundell@chromium.org
NOPRESUBMIT=true
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1615713003
Cr-Commit-Position: refs/heads/master@{#370867}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
https://codereview.chromium.org/1502373009/ )
Reason for revert:
MSan build failure.
https://build.chromium.org/p/chromium.memory.fyi/builders/Chromium%20Linux%20ChromeOS%20MSan%20Builder/builds/12498
Original issue's description:
> Allow std::unordered_*.
>
> base::hash_* is, as a transition step, implemented in terms of
> std::unordered_*. Later commits will convert existing uses.
>
> Also fix a host of IWYU problems that arose from this CL.
>
> (NOPRESUBMIT because the wstring presubmit check is overzealous
> and complains about the reference to wstring in the comment.)
>
> NOPRESUBMIT=true
> BUG=576864
> TBR=derat@chromium.org,blundell@chromium.org,jbauman@chromium.org,dalecurtis@chromium.org
> CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
>
> Committed: https://crrev.com/3f37f7f1459e7b5a452c0e433493e0a6e9649ca7
> Cr-Commit-Position: refs/heads/master@{#370553}
TBR=danakj@chromium.org,thakis@chromium.org,derat@chromium.org,blundell@chromium.org,dalecurtis@chromium.org,jbauman@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=576864
Review URL: https://codereview.chromium.org/1610023003
Cr-Commit-Position: refs/heads/master@{#370559}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
base::hash_* is, as a transition step, implemented in terms of
std::unordered_*. Later commits will convert existing uses.
Also fix a host of IWYU problems that arose from this CL.
(NOPRESUBMIT because the wstring presubmit check is overzealous
and complains about the reference to wstring in the comment.)
NOPRESUBMIT=true
BUG=576864
TBR=derat@chromium.org,blundell@chromium.org,jbauman@chromium.org,dalecurtis@chromium.org
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1502373009
Cr-Commit-Position: refs/heads/master@{#370553}
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Instead of creating separate analysis task in raster thread for
solid color detection, we do it while scheduling and rasterization
in cc thread itself. This would save us the thread overhead etc.
BUG=553612
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1531013004
Cr-Commit-Position: refs/heads/master@{#369263}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
With synchronized painting, because painting is already done when
testing record_time, the normal record_time result doesn't contain
anything for painting.
Add subsequence_caching_disabled to force painting with subsequence
caching disabled, to measure the performance of display item caching.
The new mode should generates the same record_time before
synchronized painting is enabled.
The default record_time is still useful to measure the cost other
than paint.
BUG=536999
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
CQ_EXTRA_TRYBOTS=tryserver.chromium.perf:linux_perf_bisect;tryserver.chromium.perf:mac_10_10_perf_bisect;tryserver.chromium.perf:win_perf_bisect
Review URL: https://codereview.chromium.org/1552693002
Cr-Commit-Position: refs/heads/master@{#367465}
|
|
|
|
|
|
|
|
|
|
| |
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 fixes rasterzie_and_record benchmark crash after
crrev.com/68f1b60eb2b1a534f317852a48928bf5e56c8aef.
Blink expects paint on drawsContent layers only.
BUG=570445
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1530763007
Cr-Commit-Position: refs/heads/master@{#365677}
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
pixels_recorded.
Now that the benchmark records the entire PaintableRegion, rather than just the
visual rect, use that as the space for pixels_recorded.
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1528603002
Cr-Commit-Position: refs/heads/master@{#365560}
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This eliminates the need for the temporary variable in ProxyMain and
makes the initialization set up for the singlethreaded and threade case
identical.
BUG=567930
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1520623003
Cr-Commit-Position: refs/heads/master@{#365460}
|
|
|
|
|
|
|
|
|
| |
Follow on to https://crrev.com/1501303002
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1502343005
Cr-Commit-Position: refs/heads/master@{#364154}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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}
|
|
|
|
|
|
|
|
|
|
|
|
| |
This patch updates the benchmark_instrumentation owners.
NOTRY=True
R=enne
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1504223003
Cr-Commit-Position: refs/heads/master@{#363793}
|
|
|
|
|
|
|
|
|
|
| |
The API now specifies that ContentLayerClient decides its own PaintableRegion().
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1489713004
Cr-Commit-Position: refs/heads/master@{#363005}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
As part of serializing cc::LayerTreeHost, we also need to
serialize the cc::LayerTreeSettings and the
cc::LayerTreeDebugState.
This CL focuses only on the debug state.
BUG=561210
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1469393004
Cr-Commit-Position: refs/heads/master@{#362534}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
When property trees are enabled, draw properties are only
computed for drawn layers. This means that when a non-drawn
layer's screen space transform is needed, it must be computed
on-demand using the transform tree rather than being read
from the layer's draw properties.
BUG=560758
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1492463002
Cr-Commit-Position: refs/heads/master@{#362498}
|
|
|
|
|
|
|
|
|
|
|
| |
It's somewhat confusing to have to find it in cc/trees/layer_tree_settings.h.
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
TBR=dtrainor@chromium.org,piman@chromium.org
Review URL: https://codereview.chromium.org/1462613002
Cr-Commit-Position: refs/heads/master@{#360441}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This hits the following:
cc/animation/
cc/base/
cc/blink/
cc/debug/
cc/input/
R=enne, vmpstr
BUG=557422
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1455803003
Cr-Commit-Position: refs/heads/master@{#360401}
|
|
|
|
|
|
|
|
|
|
|
|
| |
This patch removes ScopedPtrVector and cc::remove_if. It depends on
https://codereview.chromium.org/1441613002 for TakeBack.
R=danakj
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1437413002
Cr-Commit-Position: refs/heads/master@{#360318}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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}
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The impl side of the compositor uses Proxy to only access the
task runners and make debug assertions. Move those parts to
TaskRunnerProvider to seperate these dependencies.
BUG=527200
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1411663002
Cr-Commit-Position: refs/heads/master@{#355370}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
We only have one raster source now, we can remove the base class.
Also, the helper is no longer needed because we only have one
implementation now.
Change-Id: I97c63f37a039ea31fa8eaa7b0c3e4f0e8bb6f8bf
BUG=536104
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1405883002
Cr-Commit-Position: refs/heads/master@{#354915}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Remove unused methods:
- RenderingStatsInstrumentation::StartRecording
- RenderingStatsInstrumentation::EndRecording
Initializing ThreadTicks in the constructor of
RenderingStatsInstrumentation caused a regression of page load time.
Since StartRecording and EndRecording are not used, this CL removes
these methods + the initialization of ThreadTicks in the constructor.
BUG=543660
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1410713003
Cr-Commit-Position: refs/heads/master@{#354879}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Use QueryThreadCycleTime() to get the number of CPU clock cycles used
by the current thread. Convert it to microseconds using a measured
TSC frequency.
The value returned by QueryThreadCycleTime() is based on the rdtsc
instruction. For several years, Intel has been shipping CPUs with a
constant-rate counter, which means that the QueryThreadCycleTime()
results are directly proportional to wall-clock time on most systems
(see crbug.com/280743#c15). ThreadTicks::IsSupported() will return
false if the CPU doesn't have a constant rate TSC.
BUG=280743
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1390743002
Cr-Commit-Position: refs/heads/master@{#354213}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
With red color on black background, the graph lines do not stand out
distinctly. So changed few colors to make HUD status more visible.
Also separated the titles for each display to make them visible and
added memory graph to easily understand the memory status.
BUG=541121
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1377823002
Cr-Commit-Position: refs/heads/master@{#353727}
|
|
|
|
|
|
|
|
|
|
|
| |
This base class is no longer needed, removing
BUG=536104
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1383073002
Cr-Commit-Position: refs/heads/master@{#352676}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Adds UMA and Telemetry support for counting checkerboarded area in
pixels, and separate checkerboarded content area that has recording and
checkerboarded content area that does not have recording.
BUG=535732
R=enne
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1377583004
Cr-Commit-Position: refs/heads/master@{#352407}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This patch removes the cc::Picture class, since the display list
approach is the only one supported now.
Depends on https://codereview.chromium.org/1362663002/
BUG=533151
R=danakj, enne
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1365673004
Cr-Commit-Position: refs/heads/master@{#350661}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This patch removes the picture pile approach for recording and raster
sources. Most of the patch is to switch over the tests that were using
picture piles to use display list raster and recording sources.
BUG=533151
R=pdr, danakj, enne
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1362663002
Cr-Commit-Position: refs/heads/master@{#350414}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Slimming paint has shipped so the non-sp codepaths are rapidly rotting.
This patch removes the slimming paint codepaths in chrome while leaving
the removal of PicturePile and ContentLayerClient::PaintContents to a
followup.
BUG=524314
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1343493003
Cr-Commit-Position: refs/heads/master@{#349168}
|
|
|
|
|
|
|
|
|
| |
BUG=523040
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1296673004
Cr-Commit-Position: refs/heads/master@{#348763}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
As blink now only creates discardable images (not discardable
bitmaps/pixelrefs, we need to track the images.
(based on reed's http://crrev.com/1318323003)
BUG=527246
R=reed@google.com,vmpstr@chromium.org
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1314353005
Cr-Commit-Position: refs/heads/master@{#347985}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
We currently do this math in CoversRect(), GatherPixelRefs() and each
time a tile is rastered to compute damage. We also do this multiple
times in the process of creating a tile.
We can do this once at the time we create the tile and pass it around
and store it on the Tile instead. Less math!
R=vmpstr
BUG=342848
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1318733006
Cr-Commit-Position: refs/heads/master@{#346491}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Non-ideal tilings are not going to be rasterized, so it's a waste to
allocate more tiles for it. Instead, only intersect the interest rect
instead of moving/growing it. This save some allocations on pinch zoom
cases.
BUG=517749
R=danakj
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1284763003
Cr-Commit-Position: refs/heads/master@{#343087}
|