RESOLVED FIXED Bug 80142
[EFL] Implement LayoutTestController::clearBackForwardList 
https://bugs.webkit.org/show_bug.cgi?id=80142
Summary [EFL] Implement LayoutTestController::clearBackForwardList
Sudarsana Nagineni (babu)
Reported 2012-03-02 03:59:51 PST
We should implement LayoutTestController::clearBackForwardList for the EFL port so that we can unskip some history feature tests.
Attachments
V1:Patch (4.11 KB, patch)
2012-03-02 04:22 PST, Sudarsana Nagineni (babu) 		no flags
DetailsFormatted DiffDiff
Implement clearBackForwardList (4.24 KB, patch)
2012-03-13 08:20 PDT, Sudarsana Nagineni (babu) 		no flags
DetailsFormatted DiffDiff
New patch fixing comment #4 (4.14 KB, patch)
2012-03-13 09:39 PDT, Sudarsana Nagineni (babu) 		no flags
DetailsFormatted DiffDiff
Show Obsolete (2) View All Add attachment proposed patch, testcase, etc.
Sudarsana Nagineni (babu)
Comment 1 2012-03-02 04:22:14 PST
Created attachment 129876 [details] V1:Patch Implemented LayoutTestController::clearBackForwardList and unskipped some of the history feature tests which are passing now.
Gyuyoung Kim
Comment 2 2012-03-06 23:41:14 PST
Comment on attachment 129876 [details] V1:Patch View in context: https://bugs.webkit.org/attachment.cgi?id=129876&action=review Add description to ChangeLog, then request review and commit queue. > Tools/ChangeLog:7 > + You should add description for this patch. > LayoutTests/ChangeLog:7 > + ditto.
Sudarsana Nagineni (babu)
Comment 3 2012-03-13 08:20:11 PDT
Created attachment 131618 [details] Implement clearBackForwardList
Raphael Kubo da Costa (:rakuco)
Comment 4 2012-03-13 09:11:10 PDT
Comment on attachment 131618 [details] Implement clearBackForwardList View in context: https://bugs.webkit.org/attachment.cgi?id=131618&action=review > Tools/DumpRenderTree/efl/LayoutTestControllerEfl.cpp:75 > + const int limit = ewk_history_limit_get(history); > + ewk_history_limit_set(history, 0); > + ewk_history_limit_set(history, limit); You can use ewk_history_clear() instead.
Sudarsana Nagineni (babu)
Comment 5 2012-03-13 09:39:32 PDT
Created attachment 131636 [details] New patch fixing comment #4
Raphael Kubo da Costa (:rakuco)
Comment 6 2012-03-13 09:41:32 PDT
Comment on attachment 131636 [details] New patch fixing comment #4 Thanks, looks fine to me.
WebKit Review Bot
Comment 7 2012-03-14 08:15:49 PDT
Comment on attachment 131636 [details] New patch fixing comment #4 Clearing flags on attachment: 131636 Committed r110700: <http://trac.webkit.org/changeset/110700>
WebKit Review Bot
Comment 8 2012-03-14 08:15:54 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.