summaryrefslogtreecommitdiffstats
path: root/cc
Commit message (Collapse)AuthorAgeFilesLines
* Directly use GrGLInterface in cc tests rather than SkNullGLContext.bsalomon2016-03-251-4/+5
| | | | | | | | | | This unblocks removal of SkGLContext and its subclasses from the Skia lib. Using the null GrGLInterface here used to be problematic because its implementation stored its state globally, assuming there was only one instance. This is no longer the case. CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel Review URL: https://codereview.chromium.org/1829363003 Cr-Commit-Position: refs/heads/master@{#383375}
* Add SurfaceId::id_namespace()enne2016-03-252-0/+17
| | | | | | | | | | | | This ties SurfaceIdAllocator to SurfaceId, such that it is now the only valid allocator for SurfaceIds. R=jbauman@chromium.org CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel Review URL: https://codereview.chromium.org/1834583004 Cr-Commit-Position: refs/heads/master@{#383323}
* cc : Fix layer transform changed tracking bugjaydasika2016-03-252-2/+30
| | | | | | | | | | | | 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}
* cc: Force a redraw on initializing a new output surface for remote compositor.khushalsagar2016-03-255-8/+40
| | | | | | | | | | | | | | | 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}
* Add more out of line copy ctors for complex classes.vmpstr2016-03-2412-0/+24
| | | | | | | | | | | | | | | 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}
* Handle out of order tile priorities when building graphericrk2016-03-241-6/+14
| | | | | | | | | | | | | | | | | | | | | | Currently, InsertNodesForRasterTask in tile_manager.cc assumes that all high priority tasks will be added before low priority ones. This is generally true, but two cases violate this: - low resolution tiles in the NOW bin may have higher priority than other tiles that are required for draw/activation. - some SOON tiles not required for activation may come before SOON tiles which are required for activation. (bug files) To address the first issue, the calculation of high_priority now takes the NOW bucket into account. To fix the second issue, InsertNodesForRasterTask now handles out-of-order priorities and a bug has been filed to fix this. BUG=592160 CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel Review URL: https://codereview.chromium.org/1796423002 Cr-Commit-Position: refs/heads/master@{#383126}
* cc: Remove more code that was only used by CDPajuma2016-03-248-197/+1
| | | | | | | | | | | | 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}
* cc : Determine if a layer is root by using the value in LayerTreeImpljaydasika2016-03-247-47/+74
| | | | | | | | | | | | 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}
* Remove some dead code in LayerImplvollick2016-03-232-14/+0
| | | | | | | | | BUG=None CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel Review URL: https://codereview.chromium.org/1834453002 Cr-Commit-Position: refs/heads/master@{#382971}
* cc: Fix LayerTree deserialization for commit.khushalsagar2016-03-238-115/+211
| | | | | | | | | | | | | | | | | | | | | | | | | | | | 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}
* Fixes cc_blink_unittests in a better way.yukishiino2016-03-231-2/+2
| | | | | | | | | | | | | | This is a better version of the fix of http://crrev.com/1737873002 We should add "blink_for_unittests" to "cc_blink_unittests" rather than "cc_blink". BUG=589949 CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel Review URL: https://codereview.chromium.org/1820903005 Cr-Commit-Position: refs/heads/master@{#382800}
* cc : Update render surfaces using LayerListIterator instead of treewalkjaydasika2016-03-238-166/+230
| | | | | | | | | | 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}
* Enable LayerTreeHostTestGpuRasterDeviceSizeChanged test on winjaydasika2016-03-221-9/+5
| | | | | | | | | | | | 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}
* Remove unused LayerImpl::GetPicture()fmalita2016-03-224-14/+0
| | | | | | | | | | | No clients AFAICT. R=enne@chromium.org,danakj@chromium.org CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel Review URL: https://codereview.chromium.org/1827523002 Cr-Commit-Position: refs/heads/master@{#382692}
* Update secure output flags on display changes.dcastagna2016-03-2218-12/+45
| | | | | | | | | | | | | | | | | | | | | 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}
* Disable LayerTreeHostTestGpuRasterDeviceSizeChanged test on win.dgozman2016-03-221-0/+3
| | | | | | | | | | | 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}
* Transfer LayerImpl ownership to LayerTreeImplvollick2016-03-2251-585/+577
| | | | | | | | | | | | | | | | | | | 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}
* Use sk_sp-based picture recording APIsfmalita2016-03-2223-72/+57
| | | | | | | | | | | | | | | | 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}
* Switch SkImageFilter::filterBounds() calls over to non-deprecated API.senorblanco2016-03-222-7/+4
| | | | | | | | | | | Also remove the protective define for the legacy API. BUG=skia:5094 CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel Review URL: https://codereview.chromium.org/1820023003 Cr-Commit-Position: refs/heads/master@{#382540}
* Make a command buffer workaround for glReadPixels on IOSurface backed textures.erikchen2016-03-221-50/+0
| | | | | | | | | | | | The workaround previously existed in GLRenderer. This CL moves the workaround into the command buffer. BUG=581777 CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel Review URL: https://codereview.chromium.org/1806753003 Cr-Commit-Position: refs/heads/master@{#382466}
* Make more of content pass "gn check"brettw2016-03-211-1/+3
| | | | | | | | | | Adds dependencies and makes some deps public so more of content passes include checking. This adds 25 targets. CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel Review URL: https://codereview.chromium.org/1817903002 Cr-Commit-Position: refs/heads/master@{#382430}
* cc: Remove a default constructor argument.vmpstr2016-03-212-1/+5
| | | | | | | | | | | | This patch changes the default constructor argument in favor of two constructors. R=ericrk, danakj CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel Review URL: https://codereview.chromium.org/1819773005 Cr-Commit-Position: refs/heads/master@{#382408}
* Bind GrContext to GLES2Interface rather than C GLES interfacebsalomon2016-03-211-33/+1
| | | | | | | | | | CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel BUG=442811 Review URL: https://codereview.chromium.org/1671283002 Cr-Commit-Position: refs/heads/master@{#382393}
* cc : Make tree synchronization independent of layer tree hierarchy (2)jaydasika2016-03-2118-784/+470
| | | | | | | | | | | | | | | | | 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}
* 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}
* CC Animation: Use unordered_map instead of vector.loyso2016-03-213-41/+37
| | | | | | | | | | | | A performance optimization to align the implementation with AnimationTimeline::id_to_player_map_ BUG=595584 CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel Review URL: https://codereview.chromium.org/1814103002 Cr-Commit-Position: refs/heads/master@{#382241}
* 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}
* Add CONTEXT_TYPE_OPENGLES2_PEPPERericrk2016-03-184-2/+53
| | | | | | | | | | | | | | | 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}
* Remove lots of usage of WebCString.dtapuska2016-03-181-1/+0
| | | | | | | | | | BUG=585665 CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel TBR=dalecurtis@chromium.org,jochen@chromium.org Review URL: https://codereview.chromium.org/1807193004 Cr-Commit-Position: refs/heads/master@{#382064}
* 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}
* Allow null begin frame sources in cc::Schedulerenne2016-03-184-10/+68
| | | | | | | | | | | | | | | | | | | This adds support to make it possible to set the begin frame source in Scheduler back to null before setting it to something else again. This is primarily to support the use case of losing an output surface (which will provide the begin frame source) before asynchronously later initializing another output surface (with potentially another source). In this case, there will be some period of time where there is no valid begin frame source. The scheduler owner could provide a fake one, but I think that's more confusing than helpful. Depends on https://codereview.chromium.org/1774323003 CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel Review URL: https://codereview.chromium.org/1781663004 Cr-Commit-Position: refs/heads/master@{#382016}
* cc: Removed unused layer of redirection on functionweiliangc2016-03-181-28/+19
| | | | | | | | | | | | | | 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}
* Store recording invalidations in DisplayListRecordingSource, save them via ↵chrishtr2016-03-1813-30/+127
| | | | | | | | | | | | | | | | | | | | | 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}
* Use preferred format to allocate memory for decoded imagesbashi2016-03-183-7/+42
| | | | | | | | | | | | | | | | | | | | | | | | | | | 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}
* CC Animation: Use bitset instead of unordered_set in LayerAnimationControllerloyso2016-03-183-19/+33
| | | | | | | | | | | A performance optimization. This avoids any heap allocations. BUG=595584 CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel Review URL: https://codereview.chromium.org/1806223002 Cr-Commit-Position: refs/heads/master@{#381850}
* Trim some headers from base/memory/scoped_ptr.hdcheng2016-03-171-0/+1
| | | | | | | | | | BUG=554298 CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel TBR=brettw@chromium.org Review URL: https://codereview.chromium.org/1810893003 Cr-Commit-Position: refs/heads/master@{#381793}
* Allow cc::Scheduler begin frame source to be changedenne2016-03-174-15/+197
| | | | | | | | | | | Also, add back throttling tests to check changing the source. Depends on https://codereview.chromium.org/1765723002 CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel Review URL: https://codereview.chromium.org/1774323003 Cr-Commit-Position: refs/heads/master@{#381779}
* cc: Add test for the fix in https://codereview.chromium.org/1808043002/vmpstr2016-03-171-0/+67
| | | | | | | | | | | | This patch adds a unittest that fails without the patch, and passes with it. R=enne CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel Review URL: https://codereview.chromium.org/1814643002 Cr-Commit-Position: refs/heads/master@{#381748}
* Monitor VideoResourceUpdater reusing destructed resource in Debug mode.xjz2016-03-172-3/+84
| | | | | | | | | | | | | | | | | VideoResourceUpdater software planes use VideoFrame raw pointer and video timestamp as the unique ID for each VideoFrame. However, if the VideoFrame is destructed and the pointer is reused by a new VideoFrame, and if the timestamp is not correctly set, the resource maybe incorrectly reused. This CL marks the resource as destructed when the original VideoFrame is destructed, and monitors if the destructed resource is re-used. The CL only takes effect in Debug mode. BUG=581480 CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel Review URL: https://codereview.chromium.org/1688033005 Cr-Commit-Position: refs/heads/master@{#381739}
* cc: Impl thread scroll on ScrollNode instead of LayerImplsunxd2016-03-1711-75/+153
| | | | | | | | | BUG=568830 CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel Review URL: https://codereview.chromium.org/1805343006 Cr-Commit-Position: refs/heads/master@{#381614}
* cc: Ensure to process active low res pending high res tilings at rastervmpstr2016-03-171-5/+5
| | | | | | | | | | | | | | | This patch ensures that we process active low res pending high res tilings, since they can have required for activation tiles. Previously, this only applied to non ideal tiles, but clearly it's possible to hit the rare case of low res on active while high res on pending. R=enne, chrishtr BUG=595209 CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel Review URL: https://codereview.chromium.org/1808043002 Cr-Commit-Position: refs/heads/master@{#381613}
* Fold BeginFrameSource::SetClientReady into AddObserverenne2016-03-176-19/+0
| | | | | | | | | | | | | | | | | | As a part of https://codereview.chromium.org/1774323003, it became clear that it needed to be possible to call SetClientReady repeatedly, which some sources didn't support. Instead of just making that part of the contract, instead fold the two places where SetClientReady mattered into AddObserver (the first time where a begin frame could actually be delivered) and remove SetClientReady from the API. R=skyostil@chromium.org,sunnyps@chomium.org CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel Review URL: https://codereview.chromium.org/1778853003 Cr-Commit-Position: refs/heads/master@{#381612}
* Re-enable IOSurface Canvas2D.erikchen2016-03-161-2/+4
| | | | | | | | | | | | | | | Previously, WebExternalTextureLayerImpl inferred the size of the texture from the bounds of the WebLayer, but the two are not necessarily the same. This CL passes the texture size through WebExternalTextureMailbox. This CL also adds a pixel test. BUG=595063 CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel Review URL: https://codereview.chromium.org/1804243002 Cr-Commit-Position: refs/heads/master@{#381570}
* Hoist begin frame sources out of schedulerenne2016-03-1613-749/+131
| | | | | | | | | | | | | | | | | | | | | | | | | | 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}
* cc: Remove some unnecessary const scoped_refptr&.vmpstr2016-03-1619-93/+90
| | | | | | | | | | | | | | | This patch makes the API a bit clearer by taking a copy of scoped_refptr in some situations (where ownership is shared) and using a raw pointer in situations where ownership is not needed. The only references that remain are the ones captured in lambdas or Callbacks. R=danakj BUG=589044 CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel Review URL: https://codereview.chromium.org/1803863003 Cr-Commit-Position: refs/heads/master@{#381524}
* cc: Make sure to set clip rect to empty on layers when neededweiliangc2016-03-162-49/+88
| | | | | | | | | | | | | 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}
* Clean scroll logic in LayerTreeImpl:sunxd2016-03-1618-116/+187
| | | | | | | | | | | | * 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}
* cc: Skip image predecode tasks for LOW_RESOLUTION tiles.vmpstr2016-03-151-5/+14
| | | | | | | | | | | | | | This patch ensures that if we have a low resolution tile, we skip image predecode tasks. The reason for this is that we skip images altogether for LOW_RESOLUTION tiles, so having image decode tasks would defeat the purpose (ie we'd still do work). R=enne, ericrk CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel Review URL: https://codereview.chromium.org/1802793002 Cr-Commit-Position: refs/heads/master@{#381270}
* CC Animation: Erase LayerTreeSettings::accelerated_animation_enabled flag.loyso2016-03-156-191/+79
| | | | | | | | | | | | | | | | | 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}
* CC Animation: Redirect ScrollAnimationAbort via LayerTreeImplloyso2016-03-153-3/+9
| | | | | | | | | | | | 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}
generated by cgit v1.1 at 2024-05-04 20:57:00 +0000