Name the graph property the same as in engine

In order to keep consistency between the engine
properties and the analyzer/decider properties
we should try to keep the naming the same (in
this case they both return the execution_graph).

Change-Id: I981d13c68ab597a91dadea2925183674d027ce0b
Avoid not_done naming conflict

In order to make understanding the
usage of the 'not_done' variable easier
allow the schedule to take in the list
it will be appending to instead of relying
on a function local variable (which is
overwritten in the contained while loop)

Change-Id: I60c0c970d8fb444983e0d8b567ebe82c47e13d96
Merge "Name the graph property the same as in engine"
Merge "Avoid not_done naming conflict"