perf(injector): edited fromParent constructor module config copying

Updated fromParent constructor to pass providers list to module to update instead of having module constructing a temporary map. 10-15% faster on create child benchmark.

Module providers are no longer cached which could mean worse performance for deep module trees, but since Angular never has child modules more than one deep this should not be a problem. A cache can be added later if necessary. 
I think forEach was slower than a regular for loop...