refactor(RoutingManager): remove dead code

[samouss]

Summary

This PR removes dead code inside the RoutingManager. Here is the changes:

• L66-72: It was previously used inside the URLSync implementation to avoid useless search + render. Inside the previous implementation, it was working fine because both values were objects (getState with a filter function returns a plain object). With the routing system it's not the case. We compare a plain object with an instance of SearchParameters, even with the same parameters the two values are different. It means that with the new implementation the condition never worked. I removed the condition because we never had an issue related to performance in that part of the code since the release of the feature.

• L167-172: It was previously used inside the URLSync implementation to provide the full state to the callback. It was required because the URLSync was not able to restore a full state from the URL. With the routing system it's not the case. We compute the SearchParameters directly from the current helper.state which means that it already contains the "configuration" part (which is disjunctiveFacets, etc... crafted at getConfiguration).

• L26-37: We got rid of the usage of this.originalConfig inside the manager at the two previous steps which means that we can drop it completely. The call to algoliasearchHelper is replaced with SearchParameters.make because we only use the state of the helper. You can take a look at the constructor implementation, the replacement call is equivalent.

[algobot]

Deploy preview for instantsearchjs ready!

Built with commit 9f4a459

https://deploy-preview-3631--instantsearchjs.netlify.com