Add the ability to resize the window when over the custom frame.

Add the ability to resize the window when over the custom frame.

I also switched from 3px borders to 4px borders because that's what they are
on windows.  On linux, the borders look bigger because our webcontent area
doesn't drop a shadow.

For this to work for resizing on top, I had to remove the event box from
the browser titlebar and move the mouse event handling to the window.

BUG=14645

[Evan Martin, 2009-06-19 23:27:46]

LGTM

http://codereview.chromium.org/140026/diff/1/3
File chrome/browser/gtk/browser_titlebar.h (right):

http://codereview.chromium.org/140026/diff/1/3#newcode40
Line 40: void ShowContextMenu();
Why move this?

http://codereview.chromium.org/140026/diff/1/4
File chrome/browser/gtk/browser_toolbar_gtk.h (right):

http://codereview.chromium.org/140026/diff/1/4#newcode51
Line 51: GtkWidget* widget() {
const

[tony, 2009-06-20 22:54:17]

I also fixed the tab offset bug.  Now the tabs are located in the right place
regardless of frame size.

http://codereview.chromium.org/140026/diff/1/3
File chrome/browser/gtk/browser_titlebar.h (right):

http://codereview.chromium.org/140026/diff/1/3#newcode40
Line 40: void ShowContextMenu();
On 2009/06/19 23:27:46, Evan Martin wrote:
> Why move this?

I needed the method to be public since it is now being called from
browser_window_gtk.cc.

http://codereview.chromium.org/140026/diff/1/4
File chrome/browser/gtk/browser_toolbar_gtk.h (right):

http://codereview.chromium.org/140026/diff/1/4#newcode51
Line 51: GtkWidget* widget() {
On 2009/06/19 23:27:46, Evan Martin wrote:
> const

I tried it and it didn't work.  I later pass the widget into a gtk method which
doesn't expect it to be const (although certainly could have been).

[pfeldman, 2009-06-29 12:08:38]

It looks like as of this change click events in DevTools window are considered
drag events. Could it be the case?

Thanks
Pavel

[tony, 2009-06-29 17:04:57]

I filed bug 15576 and will fix it today.

On 2009/06/29 12:08:38, pfeldman wrote:
> It looks like as of this change click events in DevTools window are considered
> drag events. Could it be the case?
> 
> Thanks
> Pavel