DescriptionPage info not closing on non-Mac desktop when close button icon clicked.
This fixes the page info close icon so when it is clicked, it closes the dialog
properly. This was a regression introduced by
https://codereview.chromium.org/473643002/ where the widget's Close() method was
accidentally moved inside of a conditional in the button handler when it really
should have been executed on all button presses. This moves the Close() method
outside the conditional so it properly closes the dialog.
BUG=404963
TBR=pkasting@chromium.org
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=290659
Patch Set 1 #
Messages
Total messages: 6 (0 generated)
|