summaryrefslogtreecommitdiffstats
path: root/cc/debug
Commit message (Collapse)AuthorAgeFilesLines
* Transfer LayerImpl ownership to LayerTreeImplvollick2016-03-221-1/+1
| | | | | | | | | | | | | | | | | | | 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}
* Re-land fix display of wheel event listeners on a page.dtapuska2016-03-212-14/+13
| | | | | | | | | | | | | | | | | | | 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}
* Revert of Fix display of wheel event listeners on a page. (patchset #1 id:1 ↵leviw2016-03-182-13/+14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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}
* Fix display of wheel event listeners on a page.dtapuska2016-03-182-14/+13
| | | | | | | | | | | | | 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}
* Store recording invalidations in DisplayListRecordingSource, save them via ↵chrishtr2016-03-181-1/+1
| | | | | | | | | | | | | | | | | | | | | 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}
* cc: Refactor SkipImageCanvas and create it during playback.vmpstr2016-03-141-1/+2
| | | | | | | | | | | | | | | 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}
* CC Animation: Erase cc::LayerSettings everywhere.loyso2016-03-141-2/+1
| | | | | | | | | | | 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}
* cc: Don't ref ptr -> raw ptr -> ref ptr for raster_source.vmpstr2016-03-121-1/+1
| | | | | | | | | | | | | 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}
* Set debug color for borders of compressed tiles.auygun2016-03-072-0/+11
| | | | | | | | | BUG=434699 CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel Review URL: https://codereview.chromium.org/1774533002 Cr-Commit-Position: refs/heads/master@{#379582}
* Use the iterator for DebugRectHistoryvollick2016-03-031-8/+8
| | | | | | | | | BUG=591708 CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel Review URL: https://codereview.chromium.org/1754333003 Cr-Commit-Position: refs/heads/master@{#379034}
* tracing: Make ConvertableToTraceFormat move-onlyprimiano2016-02-299-33/+30
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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}
* cc: Add out-of-line copy ctors for complex classes.vmpstr2016-02-254-0/+11
| | | | | | | | | | | | | | 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}
* Never call GraphicsLayer::paint when fetching painted output from cc.chrishtr2016-02-111-1/+1
| | | | | | | | | | | | | 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}
* Communicate whether passive event listeners exist to cc.dtapuska2016-02-051-1/+7
| | | | | | | | | | | | | | | | | | | | 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}
* Move scroll event handlers from layer to layer tree view.dtapuska2016-02-031-1/+1
| | | | | | | | | | | | 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}
* Move have_wheel_event_handlers to WebLayerTreeView.dtapuska2016-01-291-1/+2
| | | | | | | | | | | | | 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}
* Switch cc to std::unordered_*.davidben2016-01-281-3/+3
| | | | | | | | | | | | 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}
* Allow std::unordered_*.davidben2016-01-221-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | 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}
* Revert of Allow std::unordered_*. (patchset #15 id:280001 of ↵davidben2016-01-211-1/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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}
* Allow std::unordered_*.davidben2016-01-211-0/+1
| | | | | | | | | | | | | | | | | | | 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}
* cc: Do solid color analysis before scheduling tiles.sohan.jyoti2016-01-131-5/+4
| | | | | | | | | | | | | 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}
* Add paint testing mode subsequence_caching_disabledwangxianzhu2016-01-051-2/+9
| | | | | | | | | | | | | | | | | | | | | | 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}
* Switch to standard integer types in cc/.avi2015-12-2129-34/+76
| | | | | | | | | | 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}
* Don't test rasterize_and_record for non-DrawsContent layerwangxianzhu2015-12-161-0/+3
| | | | | | | | | | | | | 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}
* Adjust metrics in RasterizeAndRecordBenchmark to have the correct ↵chrishtr2015-12-162-11/+2
| | | | | | | | | | | | | 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}
* cc:: Change plumbing for external_begin_frame_source to the Scheduler.khushalsagar2015-12-161-1/+1
| | | | | | | | | | | | | 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}
* PNGSerializer, BitmapSerializer: use new APIhalcanary2015-12-091-3/+4
| | | | | | | | | 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}
* cc: Split ThreadProxy into ProxyMain and ProxyImplkhushalsagar2015-12-091-0/+2
| | | | | | | | | | | | | | | | | | | | | | | 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}
* cc/debug: Change benchmark_instrumentation owner from ernstm to vmpstr.vmpstr2015-12-081-2/+2
| | | | | | | | | | | | 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}
* Remove the clip parameter from ContentLayerClient.chrishtr2015-12-031-2/+1
| | | | | | | | | | 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}
* Added support for (de)serializing cc::LayerTreeDebugStatenyquist2015-12-013-0/+112
| | | | | | | | | | | | | | | | 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}
* cc: Fix screen space transform computation for non-drawn layersajuma2015-12-011-12/+12
| | | | | | | | | | | | | | | 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}
* Move cc::LayerSettings to cc/layers/layer_settings.h.jbroman2015-11-181-0/+1
| | | | | | | | | | | 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}
* cc: Replace Pass() with std::move() in some subdirs.danakj2015-11-189-20/+21
| | | | | | | | | | | | | | | | | 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}
* cc: Remove ScopedPtrVector and cc::remove_if.vmpstr2015-11-185-53/+30
| | | | | | | | | | | | 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}
* cc: Split Proxy and TaskRunnerProvider for the LayerTreeHostkhushalsagar2015-11-121-1/+3
| | | | | | | | | | | | | | | | | 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}
* cc: Split Proxy to eliminate unnecessary dependencies on the impl sidekhushalsagar2015-10-213-28/+28
| | | | | | | | | | | | | 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}
* cc: Remove the base RasterSourcehendrikw2015-10-191-4/+4
| | | | | | | | | | | | | | | | 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 in RenderingStatsInstrumentation.fdoray2015-10-192-24/+0
| | | | | | | | | | | | | | | | | | 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}
* Implement ThreadTicks::Now on Windows.fdoray2015-10-152-0/+4
| | | | | | | | | | | | | | | | | | | | 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}
* Change HUD colors to distinguish status easily.prashant.n2015-10-132-5/+11
| | | | | | | | | | | | | | | 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}
* cc: remove recording source base classhendrikw2015-10-062-16/+17
| | | | | | | | | | | 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}
* UMA and Telemetry for separate checkerboarded area has recording or notweiliangc2015-10-054-2/+33
| | | | | | | | | | | | | | 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}
* cc: Remove Picture.vmpstr2015-09-246-258/+2
| | | | | | | | | | | | | | | 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}
* cc: Remove PicturePile and PicturePileImpl.vmpstr2015-09-241-1/+0
| | | | | | | | | | | | | | 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}
* Remove non-slimming-paint codepaths in chromepdr2015-09-162-69/+1
| | | | | | | | | | | | | | 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}
* Devtools: Remove continuous repainting featuresamli2015-09-144-96/+3
| | | | | | | | | BUG=523040 CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel Review URL: https://codereview.chromium.org/1296673004 Cr-Commit-Position: refs/heads/master@{#348763}
* Convert pixel_ref_map -> discardable_image_mapfmalita2015-09-091-2/+3
| | | | | | | | | | | | | | | 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}
* cc: Do the math for a tile's content rect in layer space once.danakj2015-08-311-3/+2
| | | | | | | | | | | | | | | | | 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}
* cc: Don't create tiles in a non-ideal tiling.vmpstr2015-08-121-0/+1
| | | | | | | | | | | | | | | 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}