From c7db5c436c593f6dc78f17cacd8bd8c3a6afb1bf Mon Sep 17 00:00:00 2001 From: "sfraser%netscape.com" Date: Mon, 20 Mar 2000 23:20:12 +0000 Subject: [PATCH] Fix a missing break in the menu bar click code. No bug, r=pinkerton git-svn-id: svn://10.0.0.236/trunk@63489 18797224-902f-48f8-a5cc-f745e15eee43 --- mozilla/widget/src/mac/nsMacMessagePump.cpp | 33 +++++++++++---------- 1 file changed, 18 insertions(+), 15 deletions(-) diff --git a/mozilla/widget/src/mac/nsMacMessagePump.cpp b/mozilla/widget/src/mac/nsMacMessagePump.cpp index 8e78cf74ec3..1f883e192df 100644 --- a/mozilla/widget/src/mac/nsMacMessagePump.cpp +++ b/mozilla/widget/src/mac/nsMacMessagePump.cpp @@ -397,20 +397,23 @@ void nsMacMessagePump::DoMouseDown(EventRecord &anEvent) case inMenuBar: { - // If a xul popup is displayed, roll it up and don't allow the click - // through to the menu code. This is how MacOS context menus work, so - // I think this is a valid solution. - if ( gRollupListener && gRollupWidget ) - gRollupListener->Rollup(); - else { - long menuResult = ::MenuSelect(anEvent.where); - if (HiWord(menuResult) != 0) - { - menuResult = ConvertOSMenuResultToPPMenuResult(menuResult); - DoMenu(anEvent, menuResult); - } - break; - } + // If a xul popup is displayed, roll it up and don't allow the click + // through to the menu code. This is how MacOS context menus work, so + // I think this is a valid solution. + if ( gRollupListener && gRollupWidget ) + { + gRollupListener->Rollup(); + } + else + { + long menuResult = ::MenuSelect(anEvent.where); + if (HiWord(menuResult) != 0) + { + menuResult = ConvertOSMenuResultToPPMenuResult(menuResult); + DoMenu(anEvent, menuResult); + } + } + break; } case inContent: @@ -462,7 +465,7 @@ void nsMacMessagePump::DoMouseDown(EventRecord &anEvent) Point oldPt = anEvent.where; while (::WaitMouseUp()) { - Repeater::DoRepeaters(anEvent); + Repeater::DoRepeaters(anEvent); Point origin = {0,0}; ::LocalToGlobal(&origin);