I don't think it is useful to add another variable name for component (unless component is reassigned to a different value in the code below). It just made me question if component and root_xblock are the same value.