Context menu for title bar doesn't appear when a tab's context menu was open....

Context menu for title bar doesn't appear when a tab's context menu was open.

---------------------------------------------------------------------------
* About the Bug
---------------------------------------------------------------------------

- What steps will reproduce the problem?

1. Open the context menu for a tab of your choice.
2. Right-click on the title bar.

- What is the expected result?

The application context menu should show up.

- What happens instead?

Nothing (The tab's context menu is correctly closed, of course).

- What is the reason?

The context menu for an App is displayed when WM_NCRBUTTONDOWN event is processed on the Window::OnNCRButtonDown() handler.

In normal case, when the right mouse button is clicked on the NC area, this event is generated. But in the case that the context menu for a tab is being displayed,  WM_NCRBUTTONDOWN is not received. Instead, WM_NCLBUTTONDOWN is generated. (This is wrong because left mouse button is never pressed.)

So the right handler (Window::OnNCRButtonDown()) cannot be called.

---------------------------------------------------------------------------
* About the Fix
---------------------------------------------------------------------------

What is modified?

Modified MenuController::RepostEvent() method. This function is called when the mouse button is pressed while the context menu for a tab is being displayed.

In this function, following two steps occur in order.

1st. Determine the event type.
2nd. Post this event using PostMessage().

On the 2nd step,

In the case that the determined event on the 1st step is for the client area, there's no problem. But in the case of the non-client area, it doesn't use the right event type but use the fixed event type (WM_NCLBUTTONDOWN). This is the wrong implementation.

So, I modified this part so that it use the right event type.


Bug=5695
http://crbug.com/5695

[Finnur, 2009-01-12 19:25:00]

LGTM. I'm not familiar with this code, but I tested it on the tip of tree (few
days old) on my machine and it seems to fix the problem.

Lets see if Ben agrees.

[sky, 2009-01-12 20:14:34]

LGTM

[DeArto20, 2009-02-13 04:08:33]

It seems that one of the reviewer (Ben) looks very busy. So If there are not any
problems, can you commit this patch to the server for me?

On 2009/01/12 20:14:34, sky wrote:
> LGTM

[sky, 2009-02-13 17:47:08]

OK