In order to apply all changes in Contacts based on NGA, we need to get rid of the hashchange navigation (due to Contacts will navigate itself). This is the first step in order to have an isolated Contacts App in the near future.

Comment on attachment 8640019 [details] [gaia] borjasalguero:dialer_navigation > mozilla-b2g:master Francisco, this is a first approach. Could you take a look? Thanks!

Comment on attachment 8640019 [details] [gaia] borjasalguero:dialer_navigation > mozilla-b2g:master Hei Borja, could you ping Gabriele for feedback here?

Comment on attachment 8640019 [details] [gaia] borjasalguero:dialer_navigation > mozilla-b2g:master As the NGA work is based on changes in location (no <iframes> or wrapper needed, just navigate as the web does), the navigation model in Dialer must change slightly in order to avoid collision with Contacts NGA. Based on that, we are going to remove the navigation based in 'hashchange' due to this affects the navigation within the Contacts iframe (<iframe> and wrapper share the history, so any change in Contacts history affects the history in the Dialer). In the near future Contacts will be out of Dialer, so we will revisit this once Dialer will use NGA proposal. Gabriele, Could you take a look? This is blocking some other bugs related with NGA. Thanks!

[Blocking Requested - why for this release]: According to comment 4, and despite that we want to remove contacts from dialer in 2.5, we will need to have a solution before contacts is able to land their multiple documents solution. As dialer wont work while we have separated documents.

Comment on attachment 8640019 [details] [gaia] borjasalguero:dialer_navigation > mozilla-b2g:master This looks quite good in general, there's only one small thing I'd like to see addressed in the Navigation object (comment in the PR) and then I think we're good to land.

Comms triage: Needed for the split between Dialer and Contacts.

Comment on attachment 8640019 [details] [gaia] borjasalguero:dialer_navigation > mozilla-b2g:master Hi Gabriele! I've update the patch with your suggestions, could you take a look? Thanks!

Comment on attachment 8640019 [details] [gaia] borjasalguero:dialer_navigation > mozilla-b2g:master Looks good to me and I couldn't find any problem while testing, time to land! http://shipitsquirrel.github.io/