Enable drag and drop on breadcrumbs

Enable drag and drop on breadcrumbs

BUG=137983
TEST=Manual test.


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=149442

[Oleg Eterevsky, 2012-08-01 14:56:52]

lgtm

https://chromiumcodereview.appspot.com/10823119/diff/1/chrome/browser/resourc...
File chrome/browser/resources/file_manager/js/breadcrumbs_controller.js (right):

https://chromiumcodereview.appspot.com/10823119/diff/1/chrome/browser/resourc...
chrome/browser/resources/file_manager/js/breadcrumbs_controller.js:214: for (var
i = 1; items[i - 1] != event.target; i++) {
You can use PathUtil.join() instead of this loop.

[SeRya, 2012-08-01 15:23:43]

https://chromiumcodereview.appspot.com/10823119/diff/1/chrome/browser/resourc...
File chrome/browser/resources/file_manager/js/breadcrumbs_controller.js (right):

https://chromiumcodereview.appspot.com/10823119/diff/1/chrome/browser/resourc...
chrome/browser/resources/file_manager/js/breadcrumbs_controller.js:214: for (var
i = 1; items[i - 1] != event.target; i++) {
On 2012/08/01 14:56:52, Oleg wrote:
> You can use PathUtil.join() instead of this loop.

Join is not enough, textContent also need to be resolved. So benefit is not
obvious.

[commit-bot: I haz the power, 2012-08-01 15:23:49]

No LGTM from a valid reviewer yet. Only full committers are accepted.
Even if an LGTM may have been provided, it was from a non-committer or
a lowly provisional committer, _not_ a full super star committer.
See http://www.chromium.org/getting-involved/become-a-committer
Note that this has nothing to do with OWNERS files.

[SeRya, 2012-08-01 15:39:49]

Vlad, I need valid owner's review.

[Vladislav Kaznacheev, 2012-08-01 16:01:57]

lgtm

[commit-bot: I haz the power, 2012-08-01 16:09:25]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/serya@chromium.org/10823119/1

[commit-bot: I haz the power, 2012-08-01 17:50:58]

Change committed as 149442