Gallery.app: Update the ribbon control when the item is inserted.

Gallery.app: Update the ribbon control when the item is inserted.

The CL also fixes the insertion position of the new item, so that the new item
is inserted behind of the existing selected item.  Previously the image item is
inserted ahead of the selected item. It changes the index of the selected item,
and it causes the inconsistency between the selected index and the selected
item.

BUG=392055
TEST=manually

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

[hirono, 2014-07-08 10:04:41]

PTAL the CL? Thank you!

[yoshiki, 2014-07-08 10:06:41]

lgtm

[hirono, 2014-07-09 03:36:17]

On 2014/07/08 10:06:41, yoshiki wrote:
> lgtm

I realized that calling redraw() method is not enough to update the ribbon
control.
Please take another look? Thank you!

[yoshiki, 2014-07-09 03:49:01]

https://codereview.chromium.org/371333002/diff/60001/ui/file_manager/gallery/...
File ui/file_manager/gallery/js/ribbon.js (right):

https://codereview.chromium.org/371333002/diff/60001/ui/file_manager/gallery/...
ui/file_manager/gallery/js/ribbon.js:118: this.insertBefore(element,
nextElement);
if nextElement is null, it may fail.

[hirono, 2014-07-09 04:04:54]

https://codereview.chromium.org/371333002/diff/60001/ui/file_manager/gallery/...
File ui/file_manager/gallery/js/ribbon.js (right):

https://codereview.chromium.org/371333002/diff/60001/ui/file_manager/gallery/...
ui/file_manager/gallery/js/ribbon.js:118: this.insertBefore(element,
nextElement);
On 2014/07/09 03:49:01, yoshiki wrote:
> if nextElement is null, it may fail.

The reference element can be null.
https://developer.mozilla.org/en/docs/Web/API/Node.insertBefore

[yoshiki, 2014-07-09 07:32:14]

I didn't know that. Then LGTM

[hirono, 2014-07-09 09:11:13]

The CQ bit was checked by hirono@chromium.org

[hirono, 2014-07-09 09:11:19]

On 2014/07/09 07:32:14, yoshiki wrote:
> I didn't know that. Then LGTM

Thanks!

[commit-bot: I haz the power, 2014-07-09 09:11:45]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/hirono@chromium.org/371333002/60001

[commit-bot: I haz the power, 2014-07-09 15:46:17]

Change committed as 282044