Showing a BView does not trigger appropriate update session when the resulting dirty region is already part of an update.

[jscipione]

This ticket used to be about a "bug" in DeskCalc, and a work around has been commited. The real issue has never been fixed and is a problem in app_server that can also be reproduced with ShowImage when removing similar work arounds there.

Since hrev40864 the following steps no longer work. Reverting the patch should make the problem reproducable again, but it has nothing to do with DeskCalc:

1. Launch Deskcalc
2. Right click and hold on the text input box, select 'Show Keypad' which will hide the keypad (since it is on by default).
3. Right click and hold again on the text input box and select 'Show Keypad' again. This will show the keypad.
4. Observe that the keypad is not fully drawn.

If you resize the window the keypad is then drawn, but you shouldn't have to resize the window to make the keypad draw.

[jscipione]

I tried adding Invalidate() the end of _ShowKeypad() in CalcView.cpp and that did not fix the bug so this is going to take a bit more work to nail down.

[leavengood]

I just noticed and can confirm this myself. It may be a bug in the Interface Kit or app_server but I would keep digging in DeskCalc before looking in those places.

[stpere]

Fixed in hrev40863.

Even tho the view had a FULL_REDRAW_ON_RESIZE flag, the view itself wasn't following the window, so wasn't resized, so not redrawn on resize.

[jscipione]

Screenshot showing the the bug still exists in 40863

[jscipione]

Tested this on 40863 and the bug still appears to exist. I looked at the commit and you just added an Invlidate() to the ShowKeypad() method. I wrote in a comment above (comment #1) that I already tried that and it didn't fix the bug. Am I missing something?

[stpere]

hmm, that's weird. I swear that I saw it working. O_o.. I suppose I clicked in the view by accident. I'll look further.

[stpere]

Fixed in hrev40864. I doubled-checked :)

[jscipione]

Tested on 40864. Bug is squashed. Thanks stpere!

[jscipione]

Removed Invalidate() from CalcView::FrameResized() and restored B_FULL_UPDATE_ON_RESIZE to CalcView ctor on hrev51054 to see if Axel's changes had any effect. No change, DeskCalc still does not invalidate the keypad until the window is resized.