Fixes deleting history when there is only one item

Fixes removing of single child top level history items.

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

Committed: https://crrev.com/3af071ebc6a49ab6e7af9561d92fa5d729407832
Cr-Commit-Position: refs/heads/master@{#413668}

[ahmetemirercin, 2016-08-21 18:42:36]

Description was changed from

==========
Fixes deleting history when there is only one item

BUG=
==========

to

==========
Fixes deleting history when there is only one item

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

[ahmetemirercin, 2016-08-21 18:49:01]

Description was changed from

==========
Fixes deleting history when there is only one item

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

to

==========
Fixes removing of single child top level history items.

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

[ahmetemirercin, 2016-08-21 18:49:01]

ahmetemiremir@gmail.com changed reviewers:
+ calamity@chromium.org, nyerramilli@chromium.org, ranjitkan@chromium.org

[ahmetemirercin, 2016-08-21 18:52:34]

https://codereview.chromium.org/2262993002/diff/1/chrome/browser/resources/md...
File chrome/browser/resources/md_history/history_list_behavior.js (right):

https://codereview.chromium.org/2262993002/diff/1/chrome/browser/resources/md...
chrome/browser/resources/md_history/history_list_behavior.js:178: if
(array.length == 0 && node.currentPath.indexOf('.') != -1)
Returns true if this item is single child and is not top level node.

[calamity, 2016-08-22 02:00:20]

Can you please add a test case to history_list_test.js for this?

Thanks.

[calamity, 2016-08-23 03:27:54]

lgtm

[calamity, 2016-08-23 03:28:01]

calamity@chromium.org changed reviewers:
- nyerramilli@chromium.org, ranjitkan@chromium.org

[calamity, 2016-08-23 04:07:40]

The CQ bit was checked by calamity@chromium.org

[commit-bot: I haz the power, 2016-08-23 04:07:58]

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, 2016-08-23 05:10:28]

Description was changed from

==========
Fixes removing of single child top level history items.

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

to

==========
Fixes removing of single child top level history items.

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

[commit-bot: I haz the power, 2016-08-23 05:10:29]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-08-23 05:12:37]

Description was changed from

==========
Fixes removing of single child top level history items.

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

to

==========
Fixes removing of single child top level history items.

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

Committed: https://crrev.com/3af071ebc6a49ab6e7af9561d92fa5d729407832
Cr-Commit-Position: refs/heads/master@{#413668}
==========

[commit-bot: I haz the power, 2016-08-23 05:12:38]

Patchset 2 (id:??) landed as
https://crrev.com/3af071ebc6a49ab6e7af9561d92fa5d729407832
Cr-Commit-Position: refs/heads/master@{#413668}

[ahmetemirercin, 2016-08-23 17:44:32]

On 2016/08/23 03:27:54, calamity wrote:
> lgtm

Thanks calamity for your review.