Fix 13247 by removing unnecessary judgment from method Serialize of DemoConsole.DesignerSerializationService

[LeafShi1]

Fixes #13247

Cause:

• This problem is due to the ToolStripItem not being serialized

  Judgment if (obj is Control) was added to the Serialize method of the DemoConsole.DesignerSerializationService, resulting in ToolStripItem not being serialized

  winforms/src/test/integration/DesignSurface/DesignSurfaceExt/DesignerSerializationServiceImpl.cs

  Lines 36 to 37 in 716bc12

  	if (obj is Control)
  	componentSerializationService.Serialize(serializationStore, obj);

  .netFramwork does not have this logic

Proposed changes

• Removing unnecessary judgment if (obj is Control) from method Serialize of DemoConsole.DesignerSerializationService

Customer Impact

• The ToolStrip can be copy and paste normally

Regression?

• No

Risk

• Minimal

Screenshots

Before

The subitem of the ToolStrip cannot be copied

After

All of the content can be copied normally

Test methodology

• Manually

Test environment(s)

• .net 10.0.100-preview.5.25265.106

Microsoft Reviewers: Open in CodeFlow

[Copilot]

Pull Request Overview

This PR removes an unnecessary if (obj is Control) check in the Serialize method of the test integration for the designer serialization service, allowing all objects in the collection to be serialized rather than only controls.

• Eliminates a type guard that prevented non-Control objects from being serialized in tests.
• Ensures tool strip items (and other objects) can be round-tripped in designer serialization tests.

Comments suppressed due to low confidence (2)

src/test/integration/DesignSurface/DesignSurfaceExt/DesignerSerializationServiceImpl.cs:35

• [nitpick] Consider adding a brief comment explaining why the Control type check was removed, so future readers understand the intentional widening of serialization behavior.

componentSerializationService.Serialize(serializationStore, obj);

src/test/integration/DesignSurface/DesignSurfaceExt/DesignerSerializationServiceImpl.cs:33

• Add a test case that includes non-Control objects in the objects collection to verify that serialization still succeeds and does not throw unexpected exceptions.

public object Serialize(ICollection objects)

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 76.59949%. Comparing base (716bc12) to head (81bafee).
Report is 27 commits behind head on main.

Additional details and impacted files

@@                 Coverage Diff                 @@
##                main      #13478         +/-   ##
===================================================
+ Coverage   76.59823%   76.59949%   +0.00126%     
===================================================
  Files           3230        3230                 
  Lines         639097      639157         +60     
  Branches       47289       47295          +6     
===================================================
+ Hits          489537      489591         +54     
+ Misses        145992      145989          -3     
- Partials        3568        3577          +9     

Flag	Coverage Δ
Debug	76.59949% <ø> (+0.00126%)	⬆️
integration	18.77994% <ø> (-0.00219%)	⬇️
production	51.00698% <ø> (+0.00816%)	⬆️
test	97.40411% <ø> (ø)
unit	48.37893% <ø> (-0.00734%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[ricardobossan]

All LGTM!

[Shyam-Gupta]

Looks good.