summaryrefslogtreecommitdiffstats
path: root/cc/trees
Commit message (Collapse)AuthorAgeFilesLines
* cc : Fix layer transform changed tracking bugjaydasika2016-03-251-0/+28
| | | | | | | | | | | | 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-253-8/+32
| | | | | | | | | | | | | | | 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-242-0/+6
| | | | | | | | | | | | | | | 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}
* cc: Remove more code that was only used by CDPajuma2016-03-242-123/+0
| | | | | | | | | | | | 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}
* cc: Fix LayerTree deserialization for commit.khushalsagar2016-03-232-14/+49
| | | | | | | | | | | | | | | | | | | | | | | | | | | | 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}
* cc : Update render surfaces using LayerListIterator instead of treewalkjaydasika2016-03-235-153/+203
| | | | | | | | | | 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}
* Update secure output flags on display changes.dcastagna2016-03-2213-0/+38
| | | | | | | | | | | | | | | | | | | | | 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-2227-433/+411
| | | | | | | | | | | | | | | | | | | 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-222-4/+4
| | | | | | | | | | | | | | | | 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}
* cc : Make tree synchronization independent of layer tree hierarchy (2)jaydasika2016-03-219-302/+315
| | | | | | | | | | | | | | | | | 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}
* Add CONTEXT_TYPE_OPENGLES2_PEPPERericrk2016-03-181-0/+36
| | | | | | | | | | | | | | | 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}
* 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}
* Use preferred format to allocate memory for decoded imagesbashi2016-03-181-1/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | 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: Impl thread scroll on ScrollNode instead of LayerImplsunxd2016-03-176-46/+122
| | | | | | | | | BUG=568830 CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel Review URL: https://codereview.chromium.org/1805343006 Cr-Commit-Position: refs/heads/master@{#381614}
* Hoist begin frame sources out of schedulerenne2016-03-164-10/+65
| | | | | | | | | | | | | | | | | | | | | | | | | | 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: 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-1610-42/+141
| | | | | | | | | | | | * 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 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-152-0/+7
| | | | | | | | | | | | 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}
* Delete scrollCompensationAdjustment.aelias2016-03-153-168/+6
| | | | | | | | | | | | | | | | | | | | | | | 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}
* cc: Correctly inherit clip parent's clip in target spaceweiliangc2016-03-142-6/+7
| | | | | | | | | | | | | | | 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}
* cc: Fix one unittest to set up clip children correctlyweiliangc2016-03-141-0/+3
| | | | | | | | | | | | | | | 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}
* cc: Correctly set scroll children's effect tree indexweiliangc2016-03-142-0/+4
| | | | | | | | | | | | | | | | 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}
* CC Animation: Rename HasAnimation to HasAnimationForTesting.loyso2016-03-143-8/+9
| | | | | | | | | | | 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}
* CC Animation: Erase cc::LayerSettings everywhere.loyso2016-03-1423-643/+501
| | | | | | | | | | | 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: Clips layer when render surface it owns has unclipped descendantsweiliangc2016-03-122-5/+6
| | | | | | | | | | | | | 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}
* cc : Move ResetAllChangeTracking to LayerTreeImpl from LayerImpljaydasika2016-03-126-198/+205
| | | | | | | | | | | | 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}
* Rename InputHandler::GESTURE to InputHandler::TOUCHSCREENdtapuska2016-03-113-148/+188
| | | | | | | | | | | | | | | | 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}
* Implement Wheel Gesture Scrolling on OSX.dtapuska2016-03-112-7/+50
| | | | | | | | | | | | | | | | | | 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}
* CC Animation: Erase old animation system.loyso2016-03-1110-490/+139
| | | | | | | | | | | | | | | | | | 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}
* CC Animation: Erase old animation tests.loyso2016-03-102-1531/+377
| | | | | | | | | | | | | | 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}
* Clean LayerImpl's scroll offset callers in unit tests.sunxd2016-03-0910-158/+380
| | | | | | | | | | | | | | | 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}
* Remove runtime toggling of throttling framesenne2016-03-0917-65/+0
| | | | | | | | | | | | | | | | 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}
* cc: Fix for releasing output surface during commit.khushalsagar2016-03-085-7/+89
| | | | | | | | | | | | | | | | | | | | | | 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}
* cc : Make tree synchronization independent of layer tree hierarchy (1)jaydasika2016-03-085-13/+35
| | | | | | | | | | | | | | | 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}
* cc: Do not set clip when layers are not clippedweiliangc2016-03-042-58/+78
| | | | | | | | | | | | | 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}
* CC Animation: Add missing tests from old animation system.loyso2016-03-032-1/+320
| | | | | | | | | | | | | | | | | | | | | | | | 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}
* cc: Move SyncedScrollOffset to scroll treesunxd2016-03-0311-27/+399
| | | | | | | | | | | | | | | | 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}
* Clean up function names with from property treesweiliangc2016-03-0311-387/+337
| | | | | | | | | | | | | 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}
* cc : Remove dead code from LayerTreeHostCommonjaydasika2016-03-031-491/+0
| | | | | | | | | | | 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}
* Teach LTHI about LayerListIteratorvollick2016-03-031-11/+10
| | | | | | | | | BUG=568817 CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel Review URL: https://codereview.chromium.org/1757303002 Cr-Commit-Position: refs/heads/master@{#379018}
* Use LayerListIterator when hit testing.vollick2016-03-031-61/+54
| | | | | | | | | 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}
* Add LayerTreeImpl end/begin/rend/rbeginvollick2016-03-032-0/+23
| | | | | | | | | | | | 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}
* cc : Compute safe opaque background color while building property treesjaydasika2016-03-031-3/+28
| | | | | | | | | | 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}
* cc: Separate ImageDecodeController into software and gpu parts.vmpstr2016-03-034-0/+23
| | | | | | | | | | | | | | | | | 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}
* Fix name collisions that will occur with chromium style.danakj2016-03-022-8/+8
| | | | | | | | | | | | | | | | | | 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}
* cc : Delete LayerImpl::NoteLayerPropertyChangedForSubtreejaydasika2016-03-026-28/+71
| | | | | | | | | | | | | | | | 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}