MD Bookmarks: Update TestStore API

MD Bookmarks: Update TestStore API

This changes TestStore to inherit from Store, allowing tests to use
more production code paths. It also adds a new flag to TestStore which
allows page actions to be handled normally (rather than preventing them
from executing), which is useful for integration testing.

BUG=697706
Review-Url: https://codereview.chromium.org/2839393002
Cr-Commit-Position: refs/heads/master@{#467607}
Committed: https://chromium.googlesource.com/chromium/src/+/5cdc120ce13ae37a6019159936d54920556f9b54

[tsergeant, 2017-04-27 01:45:17]

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

[tsergeant, 2017-04-27 01:45:23]

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

[commit-bot: I haz the power, 2017-04-27 01:46:25]

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-27 03:52:50]

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

[commit-bot: I haz the power, 2017-04-27 03:52:51]

Dry run: This issue passed the CQ dry run.

[calamity, 2017-04-27 04:23:53]

lgtm

https://codereview.chromium.org/2839393002/diff/1/chrome/test/data/webui/md_b...
File chrome/test/data/webui/md_bookmarks/test_store.js (right):

https://codereview.chromium.org/2839393002/diff/1/chrome/test/data/webui/md_b...
chrome/test/data/webui/md_bookmarks/test_store.js:23: }
nit: no curlies.

[tsergeant, 2017-04-27 04:48:16]

https://codereview.chromium.org/2839393002/diff/1/chrome/test/data/webui/md_b...
File chrome/test/data/webui/md_bookmarks/test_store.js (right):

https://codereview.chromium.org/2839393002/diff/1/chrome/test/data/webui/md_b...
chrome/test/data/webui/md_bookmarks/test_store.js:23: }
On 2017/04/27 04:23:53, calamity wrote:
> nit: no curlies.

Done.

[tsergeant, 2017-04-27 04:48:20]

The CQ bit was checked by tsergeant@chromium.org

[tsergeant, 2017-04-27 04:48:20]

The patchset sent to the CQ was uploaded after l-g-t-m from
calamity@chromium.org
Link to the patchset: https://codereview.chromium.org/2839393002/#ps20001
(title: "Bye, curlies")

[commit-bot: I haz the power, 2017-04-27 04:48:47]

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-27 06:40:37]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1493268500215560,
"parent_rev": "2b61f1cbbd44c6ada5f7022ff3a55493f54836f5", "commit_rev":
"5cdc120ce13ae37a6019159936d54920556f9b54"}

[commit-bot: I haz the power, 2017-04-27 06:40:49]

Description was changed from

==========
MD Bookmarks: Update TestStore API

This changes TestStore to inherit from Store, allowing tests to use
more production code paths. It also adds a new flag to TestStore which
allows page actions to be handled normally (rather than preventing them
from executing), which is useful for integration testing.

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

to

==========
MD Bookmarks: Update TestStore API

This changes TestStore to inherit from Store, allowing tests to use
more production code paths. It also adds a new flag to TestStore which
allows page actions to be handled normally (rather than preventing them
from executing), which is useful for integration testing.

BUG=697706

Review-Url: https://codereview.chromium.org/2839393002
Cr-Commit-Position: refs/heads/master@{#467607}
Committed:
https://chromium.googlesource.com/chromium/src/+/5cdc120ce13ae37a6019159936d5...
==========

[commit-bot: I haz the power, 2017-04-27 06:40:49]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/5cdc120ce13ae37a6019159936d5...