[MRG] 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