Bookmark manager: Handle "copy", "cut"and "paste" events only when list or tree pane active.

Bookmark manager: Handle "copy", "cut"and "paste" events only when list or tree pane active.

This caused by a change of getSelectedBookmarkNodes() in r172230. Before r177230, getSelectedBookmarkNodes() threw error unless list or tree is active. From r177230, getSelectedBookmarkNodes() returns selected items of list when list nor tree is active.

Note: For a11y, when context menu or organize menu is open, menu takes focus. So, list/tree is no more active element when menu is opened.

This causes "cut" event handler of document object does "cut" operation on an active item of list. In bug case, an active item is editing bookmark.

This patch changes to check active element in "copy", "cut", and "paste" event handler.

BUG=174613
TEST=On bookmark manager, edit/rename bookmark/folder and hit Ctrl+X in edit field.

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=186312

[yosin_UTC9, 2013-03-01 04:30:46]

Hi Eric,
Could you review this patch?
Thanks in advance.

Note: we also need to have patch on M25 and M26 branch.

[yosin_UTC9, 2013-03-04 02:20:56]

ping

[arv (Not doing code reviews), 2013-03-05 18:59:32]

LGTM

[yosin_UTC9, 2013-03-06 01:18:02]

Committed patchset #1 manually as r186312 (presubmit successful).