perf(status): Remove MakeDefaultStatus() from Status move assignment …

[andrewro-google]

…and construction

It's a small perf hit but it does show up in profiles. In a long running query I see it taking 6% of the cpu time. I think the existing guarantee is stronger than it needs to be: generally no one should assume that a moved-from object has any invariants except it can be reinstantiated.

---

This change is 

[scotthart]

/gcbrun

[codecov[bot]]

Codecov Report

:white_check_mark: All modified and coverable lines are covered by tests. :white_check_mark: Project coverage is 93.04%. Comparing base (77ebc1e) to head (f14aac2). :warning: Report is 42 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main   #15566   +/-   ##
=======================================
  Coverage   93.04%   93.04%           
=======================================
  Files        2406     2406           
  Lines      219803   219797    -6     
=======================================
+ Hits       204518   204521    +3     
+ Misses      15285    15276    -9     

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

:rocket: New features to boost your workflow:

• :snowflake: Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[scotthart]

/gcbrun