[MD Bookmarks] Handle bookmark moves.

[MD Bookmarks] Handle bookmark moves.

This CL makes MD Bookmarks responsive to bookmarks being moved.

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

Review-Url: https://codereview.chromium.org/2750463010
Cr-Commit-Position: refs/heads/master@{#457700}
Committed: https://chromium.googlesource.com/chromium/src/+/c867ce71a1428628b23236115c50695be40b707a

[calamity, 2017-03-16 04:59:13]

Description was changed from

==========
[MD Bookmarks] Handle bookmark moves.

This CL makes MD Bookmarks responsive to bookmarks being moved.

BUG=692843
==========

to

==========
[MD Bookmarks] Handle bookmark moves.

This CL makes MD Bookmarks responsive to bookmarks being moved.

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

[calamity, 2017-03-16 06:22:25]

Patchset #1 (id:1) has been deleted

[calamity, 2017-03-16 06:35:48]

Patchset #1 (id:20001) has been deleted

[calamity, 2017-03-16 06:53:29]

Patchset #1 (id:40001) has been deleted

[calamity, 2017-03-16 06:54:48]

Patchset #1 (id:60001) has been deleted

[calamity, 2017-03-16 06:55:07]

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

[calamity, 2017-03-16 06:55:07]

https://codereview.chromium.org/2750463010/diff/80001/chrome/browser/resource...
File chrome/browser/resources/md_bookmarks/api_listener.js (right):

https://codereview.chromium.org/2750463010/diff/80001/chrome/browser/resource...
chrome/browser/resources/md_bookmarks/api_listener.js:61:
chrome.bookmarks.onMoved.addListener(onBookmarkMoved);
Sorry =(

[tsergeant, 2017-03-17 02:55:20]

lgtm

https://codereview.chromium.org/2750463010/diff/80001/chrome/browser/resource...
File chrome/browser/resources/md_bookmarks/api_listener.js (right):

https://codereview.chromium.org/2750463010/diff/80001/chrome/browser/resource...
chrome/browser/resources/md_bookmarks/api_listener.js:61:
chrome.bookmarks.onMoved.addListener(onBookmarkMoved);
On 2017/03/16 06:55:07, calamity wrote:
> Sorry =(

aww, onCreated was even maintaining the trend.

https://codereview.chromium.org/2750463010/diff/80001/chrome/browser/resource...
File chrome/browser/resources/md_bookmarks/reducers.js (right):

https://codereview.chromium.org/2750463010/diff/80001/chrome/browser/resource...
chrome/browser/resources/md_bookmarks/reducers.js:171:
console.log(action.oldIndex + '=>' + action.index);
console.log('probably should remove this')

[calamity, 2017-03-17 05:30:17]

The CQ bit was checked by calamity@chromium.org

[calamity, 2017-03-17 05:30:18]

The patchset sent to the CQ was uploaded after l-g-t-m from
tsergeant@chromium.org
Link to the patchset: https://codereview.chromium.org/2750463010/#ps100001
(title: "remove log")

[commit-bot: I haz the power, 2017-03-17 05:30:33]

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-03-17 06:33:40]

CQ is committing da patch.

Bot data: {"patchset_id": 100001, "attempt_start_ts": 1489728617800070,
"parent_rev": "16bd4cbca0322b2d75f7c1d49377fb374ffcf804", "commit_rev":
"c867ce71a1428628b23236115c50695be40b707a"}

[commit-bot: I haz the power, 2017-03-17 06:34:37]

Description was changed from

==========
[MD Bookmarks] Handle bookmark moves.

This CL makes MD Bookmarks responsive to bookmarks being moved.

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

to

==========
[MD Bookmarks] Handle bookmark moves.

This CL makes MD Bookmarks responsive to bookmarks being moved.

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

Review-Url: https://codereview.chromium.org/2750463010
Cr-Commit-Position: refs/heads/master@{#457700}
Committed:
https://chromium.googlesource.com/chromium/src/+/c867ce71a1428628b23236115c50...
==========

[commit-bot: I haz the power, 2017-03-17 06:34:38]

Committed patchset #2 (id:100001) as
https://chromium.googlesource.com/chromium/src/+/c867ce71a1428628b23236115c50...