Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(12)

Issue 16084008: Restore Views dialog dragging and titlebar system menus. (Closed)

Created:
7 years, 6 months ago by msw
Modified:
7 years, 6 months ago
CC:
chromium-reviews, tfarina, alicet1, msw+watch_chromium.org
Visibility:
Public.

Description

Restore Views dialog dragging and titlebar system menus. Allow non-resizable dialog dragging (and system menu). (revise NonClientHitTest to return HTCAPTION as needed) This was a regression from my http://crrev.com/202000 Allow dragging dialogs from the frame view side/botttom. This is new behavior that I think is good :) Expand BubbleFrameViewTest.NonClientHitTest and support. Pass through BubbleBorderDelegate's CanResize logic. (really just needed temporarily for the added unit testing) BUG=244676, 166075 TEST=all non-constrained dialogs can be dragged, and their Windows system menus are shown on title-bar right click. R=sky@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=203354

Patch Set 1 #

Patch Set 2 : Expand resize and move areas; refactor. #

Patch Set 3 : Expand unit test; fix BubbleBorderDelegate pass-through; etc. #

Patch Set 4 : Restore the test anchor view, its parent widget is needed on Aura. #

Patch Set 5 : Limit the HTCAPTION return value to dialogs. #

Patch Set 6 : Add a TestBubbleDelegateView hack for hit testing. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+63 lines, -26 lines) Patch
M ui/views/bubble/bubble_delegate.cc View 1 2 1 chunk +1 line, -0 lines 0 comments Download
M ui/views/bubble/bubble_frame_view.cc View 1 2 3 4 5 1 chunk +8 lines, -6 lines 0 comments Download
M ui/views/bubble/bubble_frame_view_unittest.cc View 1 2 3 4 5 2 chunks +54 lines, -20 lines 0 comments Download

Messages

Total messages: 12 (0 generated)
msw
Hey Scott, please take a look; thanks!
7 years, 6 months ago (2013-05-29 19:38:35 UTC) #1
sky
LGTM
7 years, 6 months ago (2013-05-29 20:22:04 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/msw@chromium.org/16084008/12001
7 years, 6 months ago (2013-05-29 20:38:14 UTC) #3
commit-bot: I haz the power
Retried try job too often on linux_chromeos for step(s) interactive_ui_tests http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_chromeos&number=119681
7 years, 6 months ago (2013-05-29 22:06:49 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/msw@chromium.org/16084008/30001
7 years, 6 months ago (2013-05-29 23:38:32 UTC) #5
commit-bot: I haz the power
Failed to trigger a try job on win7_aura HTTP Error 400: Bad Request
7 years, 6 months ago (2013-05-30 00:58:44 UTC) #6
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/msw@chromium.org/16084008/28002
7 years, 6 months ago (2013-05-30 00:59:06 UTC) #7
commit-bot: I haz the power
Retried try job too often on win7_aura for step(s) views_unittests http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win7_aura&number=44811
7 years, 6 months ago (2013-05-30 04:43:52 UTC) #8
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/msw@chromium.org/16084008/64001
7 years, 6 months ago (2013-05-30 19:31:05 UTC) #9
commit-bot: I haz the power
Sorry for I got bad news for ya. Compile failed with a clobber build on ...
7 years, 6 months ago (2013-05-30 19:51:45 UTC) #10
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/msw@chromium.org/16084008/34002
7 years, 6 months ago (2013-05-30 20:05:35 UTC) #11
commit-bot: I haz the power
7 years, 6 months ago (2013-05-31 07:18:29 UTC) #12
Message was sent while issue was closed.
Change committed as 203354

Powered by Google App Engine
This is Rietveld 408576698