Update command state on "load" of a folder

Update command state on "load" of a folder

We used to only update the command when selection changes. However, when
a folder is loaded the selection does not change, it goes from [] to []
so no event is fired. However, we do have callbacks after the folder is
populated. Use that to update the state of all the commands.

BUG=313592
R=dbeam@chromium.org

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

[arv (Not doing code reviews), 2013-11-04 20:50:48]

I have another fix coming that takes care of the timing issues during startup.

[Dan Beam, 2013-11-04 22:19:21]

lgtm

https://codereview.chromium.org/57883004/diff/40001/chrome/browser/resources/...
File chrome/browser/resources/bookmark_manager/js/main.js (right):

https://codereview.chromium.org/57883004/diff/40001/chrome/browser/resources/...
chrome/browser/resources/bookmark_manager/js/main.js:376: return
list.dataModel.slice();
nit: if this is to make a copy, can you comment that?  or alternately freeze
this and not make a copy?

[arv (Not doing code reviews), 2013-11-04 22:29:22]

https://codereview.chromium.org/57883004/diff/40001/chrome/browser/resources/...
File chrome/browser/resources/bookmark_manager/js/main.js (right):

https://codereview.chromium.org/57883004/diff/40001/chrome/browser/resources/...
chrome/browser/resources/bookmark_manager/js/main.js:376: return
list.dataModel.slice();
On 2013/11/04 22:19:21, Dan Beam wrote:
> nit: if this is to make a copy, can you comment that?  or alternately freeze
> this and not make a copy?

Freezing is not an option. I'll add a comment.

[commit-bot: I haz the power, 2013-11-04 22:50:15]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/arv@chromium.org/57883004/90001

[commit-bot: I haz the power, 2013-11-05 02:35:41]

Retried try job too often on linux_rel for step(s) browser_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_rel&...

[commit-bot: I haz the power, 2013-11-05 14:33:34]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/arv@chromium.org/57883004/90001

[commit-bot: I haz the power, 2013-11-05 18:42:55]

Retried try job too often on linux_aura for step(s) browser_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_aura...

[commit-bot: I haz the power, 2013-11-05 19:45:18]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/arv@chromium.org/57883004/90001

[arv (Not doing code reviews), 2013-11-05 20:42:02]

Committed patchset #2 manually as r233078 (presubmit successful).