The CellParsing event of the datagridview can't be invoked when changing the value of the cell firstly #1098

[M-Lipin]

Fixes #1098

Proposed changes

• Reworking DataGridViewCell.InitializeEditingControl() method to delay EditingControlAccessibleObject initialization until DataGridView EditingControl is initialized and its handle is created.
• Delaying EditingControlAccessibleObject initialization is done by moving dgv.EditingControlAccessibleObject.get() call to dgv.EditingControl.HandleCreated event handler, so we have EditingControlAccessibleObject accessed only after the control's handle creation.

Customer Impact

• None
• There were no major releases which include the bug (introduced and fixed within the dev process in 3.0)

Regression?

• Yes

Risk

• None, product is not being changed

Screenshots

Before

• For the first time control's handle is created when accessing EditingControlAccessibleObject.

After

• For the first time control's handle is created and the EditingControlAccessibleObject is accessed.

Test methodology

• Manual testing, debugging, Narrator announcement - passed.
• Need QA approval.
• Need to test near cases and search for regressions.

Accessibility testing

Test environment(s)

.NET Core SDK (reflecting any global.json):
Version: 3.0.100-preview8-012929
Commit: 795f8aeaa8

Runtime Environment:
OS Name: Windows
OS Version: 10.0.18362
OS Platform: Windows
RID: win10-x64
Base Path: C:\Program Files\dotnet\sdk\3.0.100-preview8-012929\

Host (useful for support):
Version: 3.0.0-preview8-27907-09
Commit: fc924dc319

.NET Core SDKs installed:
2.1.700-preview-009601 [C:\Program Files\dotnet\sdk]
2.1.800-preview-009696 [C:\Program Files\dotnet\sdk]
3.0.100-preview8-012929 [C:\Program Files\dotnet\sdk]

.NET Core runtimes installed:
Microsoft.AspNetCore.All 2.1.9 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.All]
Microsoft.AspNetCore.All 2.1.11 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.All]
Microsoft.AspNetCore.App 2.1.9 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 2.1.11 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 3.0.0-preview8.19351.9 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.NETCore.App 2.1.9 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 2.1.11 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 3.0.0-preview8-27907-09 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.WindowsDesktop.App 3.0.0-preview8-27907-09 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]

To install additional .NET Core runtimes or SDKs:
https://aka.ms/dotnet-download

Microsoft Reviewers: Open in CodeFlow

[codecov]

Codecov Report

Merging #1524 into master will decrease coverage by 0.07766%.
The diff coverage is 0%.

@@                 Coverage Diff                 @@
##              master       #1524         +/-   ##
===================================================
- Coverage   25.74308%   25.66541%   -0.07767%     
===================================================
  Files            786         786                 
  Lines         268243      268256         +13     
  Branches       37977       37978          +1     
===================================================
- Hits           69054       68849        -205     
- Misses        194295      194512        +217     
- Partials        4894        4895          +1

Flag	Coverage Δ
#Debug	25.66541% <0%> (-0.07767%)	⬇️
#production	25.66541% <0%> (-0.07767%)	⬇️
#test	100% <ø> (?)

[RussKie]

LGTM

[RussKie]

As we are now in the "tell-mode" please update the description to include the new compliance sections
https://github.com/dotnet/winforms/blob/master/.github/PULL_REQUEST_TEMPLATE.md#L12-L27

[ghost]

Hello @RussKie!

Because this pull request has the AutoMerge label, I will be glad to assist with helping to merge this pull request once all check-in policies pass.

p.s. you can customize the way I help with merging this pull request, such as holding this pull request until a specific person approves. Simply @mention me (@msftbot) and give me an instruction to get started! Learn more here.

[merriemcgaw]

Sweet! Thanks @M-Lipin!!