Bug 165114

Summary: Add a new project for recording and playing back editing commands in editable web content
Product: WebKit Reporter: Wenson Hsieh <wenson_hsieh>
Component: Tools / TestsAssignee: Wenson Hsieh <wenson_hsieh>
Status: RESOLVED FIXED    
Severity: Normal CC: ap, bdakin, commit-queue, lforschler, mitz, sam, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Added highlight-able nodes in the playback harness.
none
Patch
none
Patch
none
Patch for landing none

Description Wenson Hsieh 2016-11-28 14:15:31 PST
Add a new project for recording and playing back editing commands in editable web content
Comment 1 Wenson Hsieh 2016-11-28 14:18:11 PST
<rdar://problem/29408135>
Comment 2 Wenson Hsieh 2016-11-28 14:57:03 PST
Created attachment 295537 [details]
Patch
Comment 3 Wenson Hsieh 2016-11-28 15:08:12 PST
Comment on attachment 295537 [details]
Patch

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

> Tools/ChangeLog:19
> +        2. Extract the JSON-serialized editing history data and load the playback harness with this data.

Whoops, there are two 2s here :| I'll fix this.
Comment 4 Alexey Proskuryakov 2016-11-28 15:56:34 PST
How is the playback part going to be used? WebKitTestRunner has JS functions to simulate text input functions, will this be different?
Comment 5 Wenson Hsieh 2016-11-28 19:15:59 PST
(In reply to comment #4)
> How is the playback part going to be used? WebKitTestRunner has JS functions
> to simulate text input functions, will this be different?

The purpose of this project is to provide a means for clients to remember DOM changes that occurred due to editing, and later replay these DOM changes in a separate page that allows the user to step through each of the editing actions and DOM changes that occurred, one by one. Thus, the EditingHistory project is unrelated to our testing infrastructure (e.g. TestWebKitAPI and WebKitTestRunner).

The playback part happens when the user opens the playback harness and loads a .json file containing editing history. This allows the user to 'seek' through the editing that occurred, an input at a time.
Comment 6 Wenson Hsieh 2016-11-29 16:07:49 PST
Created attachment 295662 [details]
Added highlight-able nodes in the playback harness.
Comment 7 Wenson Hsieh 2016-12-02 17:04:06 PST
Created attachment 296025 [details]
Patch
Comment 8 Wenson Hsieh 2016-12-02 17:13:22 PST
Comment on attachment 296025 [details]
Patch

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

> Tools/EditingHistory/DumpEditingHistory.js:1
> +(() => {

Whoops, this file isn't supposed to be here. I'll remove it.
Comment 9 Wenson Hsieh 2016-12-02 17:22:45 PST
Created attachment 296028 [details]
Patch
Comment 10 Beth Dakin 2016-12-07 11:33:49 PST
Comment on attachment 296028 [details]
Patch

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

One nit!

> Source/WebCore/ChangeLog:10
> +        scripts to the internal system directory when ionstalling. See the Tools ChangeLog and individual comments below

ionstalling -> installing :-P
Comment 11 Wenson Hsieh 2016-12-07 11:41:38 PST
Created attachment 296404 [details]
Patch for landing
Comment 12 WebKit Commit Bot 2016-12-07 12:41:11 PST
Comment on attachment 296404 [details]
Patch for landing

Clearing flags on attachment: 296404

Committed r209470: <http://trac.webkit.org/changeset/209470>
Comment 13 WebKit Commit Bot 2016-12-07 12:41:16 PST
All reviewed patches have been landed.  Closing bug.