Bug 125967 - WebKit2 View Gestures: Move WebProcess-side geometry collection into its own class
Summary: WebKit2 View Gestures: Move WebProcess-side geometry collection into its own ...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit2 (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Tim Horton
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-12-18 18:10 PST by Tim Horton
Modified: 2013-12-18 18:38 PST (History)
5 users (show)

See Also:


Attachments
patch (27.60 KB, patch)
2013-12-18 18:18 PST, Tim Horton
andersca: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Tim Horton 2013-12-18 18:10:25 PST
When smart magnification is implemented, we're going to need a bit more code in the WebProcess dealing with retrieving geometry of elements and such.

Move the message from UIProcess->WebProcess to grab the visible content rect out of the DrawingArea and into ViewGestureGeometryCollector (new names HAPPILY accepted). This class will grow in the future, but for now it's just the one simple message.
Comment 1 Tim Horton 2013-12-18 18:18:31 PST
Created attachment 219599 [details]
patch
Comment 2 WebKit Commit Bot 2013-12-18 18:20:06 PST
Attachment 219599 [details] did not pass style-queue:

Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Source/WebKit2/ChangeLog', u'Source/WebKit2/DerivedSources.make', u'Source/WebKit2/UIProcess/DrawingAreaProxy.h', u'Source/WebKit2/UIProcess/mac/TiledCoreAnimationDrawingAreaProxy.h', u'Source/WebKit2/UIProcess/mac/TiledCoreAnimationDrawingAreaProxy.mm', u'Source/WebKit2/UIProcess/mac/ViewGestureController.cpp', u'Source/WebKit2/UIProcess/mac/ViewGestureController.h', u'Source/WebKit2/UIProcess/mac/ViewGestureController.messages.in', u'Source/WebKit2/WebKit2.xcodeproj/project.pbxproj', u'Source/WebKit2/WebProcess/WebPage/DrawingArea.h', u'Source/WebKit2/WebProcess/WebPage/DrawingArea.messages.in', u'Source/WebKit2/WebProcess/WebPage/ViewGestureGeometryCollector.cpp', u'Source/WebKit2/WebProcess/WebPage/ViewGestureGeometryCollector.h', u'Source/WebKit2/WebProcess/WebPage/ViewGestureGeometryCollector.messages.in', u'Source/WebKit2/WebProcess/WebPage/WebPage.cpp', u'Source/WebKit2/WebProcess/WebPage/WebPage.h', u'Source/WebKit2/WebProcess/WebPage/mac/TiledCoreAnimationDrawingArea.h', u'Source/WebKit2/WebProcess/WebPage/mac/TiledCoreAnimationDrawingArea.mm', '--commit-queue']" exit_code: 1
ERROR: Source/WebKit2/WebProcess/WebPage/ViewGestureGeometryCollector.cpp:29:  Alphabetical sorting problem.  [build/include_order] [4]
Total errors found: 1 in 11 files


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 3 Anders Carlsson 2013-12-18 18:22:33 PST
Comment on attachment 219599 [details]
patch

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

> Source/WebKit2/ChangeLog:45
> +        it can be safely used from other member's constructors.

members' ?

> Source/WebKit2/WebProcess/WebPage/ViewGestureGeometryCollector.h:50
> +

Extra newline.
Comment 4 Tim Horton 2013-12-18 18:38:42 PST
http://trac.webkit.org/changeset/160818