Bug 184881 - [CoordGraphics] Remove unused trajectory cruft in CoordinatedLayerTreeHost, CoordinatedGraphicsLayer
Summary: [CoordGraphics] Remove unused trajectory cruft in CoordinatedLayerTreeHost, C...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Zan Dobersek
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2018-04-23 05:25 PDT by Zan Dobersek
Modified: 2018-04-23 07:18 PDT (History)
2 users (show)

See Also:


Attachments
Patch (10.45 KB, patch)
2018-04-23 05:31 PDT, Zan Dobersek
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Zan Dobersek 2018-04-23 05:25:26 PDT
[CoordGraphics] Remove unused trajectory cruft in CoordinatedLayerTreeHost, CoordinatedGraphicsLayer
Comment 1 Zan Dobersek 2018-04-23 05:31:49 PDT
Created attachment 338578 [details]
Patch
Comment 2 Michael Catanzaro 2018-04-23 05:43:09 PDT
Comment on attachment 338578 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=338578&action=review

> Source/WebKit/WebProcess/WebPage/CoordinatedGraphics/CompositingCoordinator.cpp:-286
> -void CompositingCoordinator::setVisibleContentsRect(const FloatRect& rect, const FloatPoint& trajectoryVector)

FWIW this function shows up in lots of crash reports. Bug #169918 showed up with a quick Bugzilla search.
Comment 3 Zan Dobersek 2018-04-23 07:16:03 PDT
Comment on attachment 338578 [details]
Patch

Clearing flags on attachment: 338578

Committed r230913: <https://trac.webkit.org/changeset/230913>
Comment 4 Zan Dobersek 2018-04-23 07:16:07 PDT
All reviewed patches have been landed.  Closing bug.
Comment 5 Radar WebKit Bug Importer 2018-04-23 07:18:21 PDT
<rdar://problem/39645249>