Remove redundant property changed callback

[yourordinarycat]

Fixes

There is a redundant property change callback in the Segmented code. It is already covered by the DP registration so this just removes that line.

PR Type

What kind of change does this PR introduce?

Bugfix

What is the current behavior?

There is a redundant callback installed for the orientation prop, and it's the wrong one.

What is the new behavior?

No redundant callback.

PR Checklist

Please check if your PR fulfills the following requirements:

• Created a feature/dev branch in your fork (vs. submitting directly from a commit on main)
• Based off latest main branch of toolkit
• Tested code with current supported SDKs
• New component
  • Documentation has been added
  • Sample in sample app has been added
  • Analyzers are passing for documentation and samples
  • Icon has been created (if new sample) following the Thumbnail Style Guide and templates
• Tests for the changes have been added (if applicable)
• Header has been added to all new source files
• Contains NO breaking changes

Other information

[yourordinarycat]

@dotnet-policy-service agree

[Arlodotexe]

I believe the intent here was to wire up the existing OnOrientationChanged callback, we should do that instead of removing the callback registration.

I see, the difference is that SelectedItem is coming from ListViewBase whereas Orientation is added by Segmented. Nevermind, in that case this can be merged as-is.