[1.6 servicing] PackageDeploymentManager - Fix .Stop() events not being generated. Include PackageReadyOrNewerAvailableStatus in .Stop() events

[DrusTheAxe]

!cherry-pick 0562da1
!cherry-pick 3c17287

• PackageDeploymentManager - Fix .Stop() events not being generated. Include PackageReadyOrNewerAvailableStatus in .Stop() events. #5278
• PackageDeploymentManager - Add missing .Stop() event on CompletedSuccess #5284

https://task.ms/56951078

[DrusTheAxe]

/azp run

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).

[codendone]

    logTelemetry.Stop(packageUriAsString);

Are you intentionally not fixing these, like you did for 1.7?

---

Refers to: dev/PackageManager/API/M.W.M.D.PackageDeploymentManager.cpp:577 in a93477b. [](commit_id = a93477b, deletion_comment = False)

[DrusTheAxe]

/azp run

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).

[DrusTheAxe]

    logTelemetry.Stop(packageUriAsString);

Are you intentionally not fixing these, like you did for 1.7?

Refers to: dev/PackageManager/API/M.W.M.D.PackageDeploymentManager.cpp:577 in a93477b. [](commit_id = a93477b, deletion_comment = False)

Grrr. Bad merge. The 1.7 PR has a couple of commits fixing this, same needed in 1.6. Fixing now. Thanks

[DrusTheAxe]

    logTelemetry.Stop(packageUriAsString);

Are you intentionally not fixing these, like you did for 1.7?
Refers to: dev/PackageManager/API/M.W.M.D.PackageDeploymentManager.cpp:577 in a93477b. [](commit_id = a93477b, deletion_comment = False)

Grrr. Bad merge. The 1.7 PR has a couple of commits fixing this, same needed in 1.6. Fixing now. Thanks

P.S. With the latest commit PackageTelemetry.h is identical here (1.6) with 1.7 and M.W.M.D.PackageDeploymentManager.cpp is identical except 1.7 adds #include "PackageManager.Containment.h" and Containment check in the 1 if

 if (WinAppSdk::Containment::IsChangeEnabled<WINAPPSDK_CHANGEID_56439870>() && options.RegisterNewerIfAvailable())

making 1.6 not identical to 1.7 except 1.7 adds the containment check. As intended

[DrusTheAxe]

/azp run

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).