MD Bookmarks: Allow 'complex' actions to access the page state directly

MD Bookmarks: Allow 'complex' actions to access the page state directly

This adds a new file, complex_actions.js, which contains action creators
that perform complex logic based off the page state. Splitting the
actions in this way improves testability and works well with Closure.

BUG=697706
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

[tsergeant, 2017-04-05 03:35:44]

Description was changed from

==========
MD Bookmarks: Allow 'complex' actions to access the page state directly

This adds a new file, complex_actions.js, which contains action creators
that perform complex logic based off the page state. Splitting the
actions in this way improves testability and works well with Closure.

BUG=697706
==========

to

==========
MD Bookmarks: Allow 'complex' actions to access the page state directly

This adds a new file, complex_actions.js, which contains action creators
that perform complex logic based off the page state. Splitting the
actions in this way improves testability and works well with Closure.

BUG=697706
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[tsergeant, 2017-04-06 03:36:03]

tsergeant@chromium.org changed reviewers:
+ calamity@chromium.org

[tsergeant, 2017-04-06 03:36:04]

PTAL

https://codereview.chromium.org/2804503002/diff/20001/chrome/test/data/webui/...
File chrome/test/data/webui/md_bookmarks/list_test.js (left):

https://codereview.chromium.org/2804503002/diff/20001/chrome/test/data/webui/...
chrome/test/data/webui/md_bookmarks/list_test.js:35: test('selects individual
items', function() {
I've removed some of these tests since they're now more directly tested by
ComplexActionsTest

[tsergeant, 2017-04-06 03:40:27]

The CQ bit was checked by tsergeant@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-04-06 03:40:57]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-04-06 04:43:44]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-04-06 04:43:44]

Dry run: This issue passed the CQ dry run.

[tsergeant, 2017-04-10 02:59:26]

FWIW, I'm putting this CL on ice while I decide whether we actually need it.