[MRG+1] Trees: set correct impurity values in BestFirstBuilder

In master, impurity values were not set correctly when building trees in best-first. 

~~The bug is minor and only affects the resulting `impurity` vector as well as variable importances. Trees were still built in the correct way, even with this bug.~~

CC: @pprett @arjoly