Removing flag OperationInReleasingDataSource from DataGridView.cs

[LeafShi1]

Fixes #13304

Proposed changes

• This is a supplemental fix to PR Add _isReleasingDataSource to prevent unnecessary operations on CurrentCell when changing or releasing DataSource #13320

• Add judgment IsHandleCreated before invoking EndEdit to avoid the exception

• Microsoft Reviewers: Open in CodeFlow

[Copilot]

Pull Request Overview

This PR refactors the handling of the OperationInReleasingDataSource flag and the subsequent cleanup of the DataGridView’s state when a bound data source is disposed. Key changes include removing the inline try–finally block that set the flag and reset CurrentCell from DataGridView.cs and moving that logic into a new dedicated method, OnDataSourceDisposed, in DataGridView.Methods.cs.

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File	Description
src/System.Windows.Forms/System/Windows/Forms/Controls/DataGridView/DataGridView.cs	Removed the inline try–finally disposal block so that the flag and CurrentCell reset are handled by OnDataSourceDisposed.
src/System.Windows.Forms/System/Windows/Forms/Controls/DataGridView/DataGridView.Methods.cs	Added a new OnDataSourceDisposed method that wraps setting DataSource and resetting CurrentCell within a try–finally block.

Comments suppressed due to low confidence (1)

src/System.Windows.Forms/System/Windows/Forms/Controls/DataGridView/DataGridView.cs:1922

• Verify that removing the inline try–finally block which sets _dataGridViewOper and resets CurrentCell does not affect any existing side effects in the property setter; ensure that all necessary state cleanup is now correctly handled by OnDataSourceDisposed.

-                _dataGridViewOper[OperationInReleasingDataSource] = true;

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 95.69342%. Comparing base (e966bca) to head (f20795a).
Report is 12 commits behind head on main.

Additional details and impacted files

@@                 Coverage Diff                  @@
##                main      #13362          +/-   ##
====================================================
+ Coverage   62.59209%   95.69342%   +33.10133%     
====================================================
  Files           1560         542        -1018     
  Lines         159763       63461       -96302     
  Branches       14915        1451       -13464     
====================================================
- Hits           99999       60728       -39271     
+ Misses         58988        2400       -56588     
+ Partials         776         333         -443     

Flag	Coverage Δ
Debug	95.69342% <ø> (+33.10133%)	⬆️
integration	?
production	?
test	95.69342% <ø> (ø)
unit	?

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.

[Tanya-Solyanik]

PLease update the servicing PRs