Keep active directory when same root item on navigation list is activated.

Keep active directory when same root item on navigation list is activated.

NavigationList's activeModelItem_() is called even when the actual active model is not changed.
It occurs when the list items on navigation list are permutated by mounting/unmounting devices.
This patch checks if the actual active model is changed.
Code for clearing selection on clicking root items is moved to click event handler. 

BUG=361047
TEST=manually tested
ran browser_tests gtest_filter=*FileManager*

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

[fukino, 2014-04-24 11:14:21]

yoshiki san, could you take a look?
This CL is for the issue discussed today.

[yoshiki, 2014-04-24 12:02:20]

LGTM with nit. Thanks for doing this!

nit: Please add "TEST" field in the description. If a patch is trivial, use just
"TEST=none".

[fukino, 2014-04-25 02:06:49]

On 2014/04/24 12:02:20, yoshiki wrote:
> LGTM with nit. Thanks for doing this!
> 
> nit: Please add "TEST" field in the description. If a patch is trivial, use
just
> "TEST=none".

Added TEST field. I'll use "TEST=none" when a patch is trivial. Thank you for
advice!

[yoshiki, 2014-04-25 02:15:53]

On 2014/04/25 02:06:49, fukino wrote:
> On 2014/04/24 12:02:20, yoshiki wrote:
> > LGTM with nit. Thanks for doing this!
> > 
> > nit: Please add "TEST" field in the description. If a patch is trivial, use
> just
> > "TEST=none".
> 
> Added TEST field. I'll use "TEST=none" when a patch is trivial. Thank you for
> advice!

still LGTM.

[fukino, 2014-04-28 02:34:21]

The CQ bit was checked by fukino@chromium.org

[commit-bot: I haz the power, 2014-04-28 02:36:01]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/fukino@chromium.org/251483002/20001

[commit-bot: I haz the power, 2014-04-28 11:52:48]

Change committed as 266507