Commit aeafd48e authored by Eriko Kurimoto's avatar Eriko Kurimoto Committed by Commit Bot

Revert "Revert "Remove unneeded list from event_stream_logger.js""

This reverts commit 695d7853.

This commit is just removing arrays already stored in EventType.
Also, it is called only when chromevox developer option flag is enabled.
This option is WIP and the flag is disabled in default.
This is irrelevant to webview benchmark.

TBR=dtseng@chromium.org,nednguyen@google.com,yawano@chromium.org,sarakato@chromium.org

Bug: 878965
Change-Id: Id2ab67061bd39597a2ebac0fedb2d7f00e6494ee
Reviewed-on: https://chromium-review.googlesource.com/1198649
Commit-Queue: Eriko Kurimoto <elkurin@google.com>
Reviewed-by: default avatarYuki Awano <yawano@chromium.org>
Cr-Commit-Position: refs/heads/master@{#587962}
parent bd593373
......@@ -104,63 +104,9 @@ EventStreamLogger.prototype = {
* @param {boolean} checked
*/
notifyEventStreamFilterChangedAll: function(checked) {
var EventTypeList = [
EventType.ACTIVEDESCENDANTCHANGED,
EventType.ALERT,
EventType.ARIA_ATTRIBUTE_CHANGED,
EventType.AUTOCORRECTION_OCCURED,
EventType.BLUR,
EventType.CHECKED_STATE_CHANGED,
EventType.CHILDREN_CHANGED,
EventType.CLICKED,
EventType.DOCUMENT_SELECTION_CHANGED,
EventType.DOCUMENT_TITLE_CHANGED,
EventType.EXPANDED_CHANGED,
EventType.FOCUS,
EventType.FOCUS_CONTEXT,
EventType.IMAGE_FRAME_UPDATED,
EventType.HIDE,
EventType.HIT_TEST_RESULT,
EventType.HOVER,
EventType.INVALID_STATUS_CHANGED,
EventType.LAYOUT_COMPLETE,
EventType.LIVE_REGION_CREATED,
EventType.LIVE_REGION_CHANGED,
EventType.LOAD_COMPLETE,
EventType.LOCATION_CHANGED,
EventType.MEDIA_STARTED_PLAYING,
EventType.MEDIA_STOPPED_PLAYING,
EventType.MENU_END,
EventType.MENU_LIST_ITEM_SELECTED,
EventType.MENU_LIST_VALUE_CHANGED,
EventType.MENU_POPUP_END,
EventType.MENU_POPUP_START,
EventType.MENU_START,
EventType.MOUSE_CANCELED,
EventType.MOUSE_DRAGGED,
EventType.MOUSE_MOVED,
EventType.MOUSE_PRESSED,
EventType.MOUSE_RELEASED,
EventType.ROW_COLLAPSED,
EventType.ROW_COUNT_CHANGED,
EventType.ROW_EXPANDED,
EventType.SCROLL_POSITION_CHANGED,
EventType.SCROLLED_TO_ANCHOR,
EventType.SELECTED_CHILDREN_CHANGED,
EventType.SELECTION,
EventType.SELECTION_ADD,
EventType.SELECTION_REMOVE,
EventType.SHOW,
EventType.STATE_CHANGED,
EventType.TEXT_CHANGED,
EventType.TEXT_SELECTION_CHANGED,
EventType.TREE_CHANGED,
EventType.VALUE_CHANGED
];
for (var evtType of EventTypeList) {
if (localStorage[evtType] == 'true')
this.notifyEventStreamFilterChanged(evtType, checked);
for (var type in EventType) {
if (localStorage[EventType[type]] == 'true')
this.notifyEventStreamFilterChanged(EventType[type], checked);
}
},
};
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment