Bug 100711

Summary: ENABLE(UNDO_MANAGER) is disabled everywhere and is not under active development
Product: WebKit Reporter: Adam Barth <abarth>
Component: New BugsAssignee: Adam Barth <abarth>
Status: RESOLVED FIXED    
Severity: Normal CC: abecsi, arv, cmarcelo, dbates, enrica, eric, gyuyoung.kim, haraken, japhet, jim, macpherson, menard, mifenton, ojan, rakuco, rniwa, vestbo, webkit.review.bot, zan
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 77562, 87189, 89722, 94868, 99041    
Attachments:
Description Flags
Patch eric: review+

Adam Barth
Reported 2012-10-29 16:48:56 PDT
ENABLE(UNDO_MANAGER) is disabled everywhere and is not under active development
Attachments
Patch (128.42 KB, patch)
2012-10-29 16:51 PDT, Adam Barth 		eric: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Adam Barth
Comment 1 2012-10-29 16:51:17 PDT
Created attachment 171337 [details] Patch
Eric Seidel (no email)
Comment 2 2012-11-01 12:51:59 PDT
Comment on attachment 171337 [details] Patch I've not heard objections. I've blocked all the open UndoManager bugs on this one.
Adam Barth
Comment 3 2012-11-02 12:01:14 PDT
Committed r133326: <http://trac.webkit.org/changeset/133326>
Jim Nelson
Comment 4 2012-11-05 12:20:09 PST
I believe this also blocks #88595.
Zan Dobersek
Comment 5 2012-12-04 02:55:15 PST
Layout tests for the undo manager functionality are still located in the tree. Should they be removed as well? http://trac.webkit.org/browser/trunk/LayoutTests/editing/undomanager
Adam Barth
Comment 6 2012-12-04 08:11:10 PST
Yes.
Zan Dobersek
Comment 7 2012-12-06 04:17:44 PST
(In reply to comment #6) > Yes. Handled in bug #104246.
Note You need to log in before you can comment on or make changes to this bug.