Use old.All() to be consistent with Update

This changes the inputs to Check/Diff to match what we also
pass to the Update function later on (old.All() vs. old.Outputs).

pkg/resource/deploy/plan_apply.go

@@ -284,7 +284,7 @@ func (iter *PlanIterator) makeRegisterResouceSteps(e RegisterResourceEvent) ([]S
 	var oldouts resource.PropertyMap
 	if hasold {
 		olds = old.Inputs
-		oldouts = old.Outputs
+		oldouts = old.All()
 	}
 
 	// Fetch the provider for this resource type, assuming it isn't just a logical one.

pkg/resource/resource_state.go

@@ -39,7 +39,7 @@ func NewState(t tokens.Type, urn URN, custom bool, del bool, id ID,
 	}
 }
 
-// All returns all resource state, including the inputs, defaults, and outputs, overlaid in that order.
+// All returns all resource state, including the inputs and outputs, overlaid in that order.
 func (s *State) All() PropertyMap {
 	return s.Inputs.Merge(s.Outputs)
 }