mirrors/electron
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages 1 Wiki Activity Actions

revert: compensate for title bar height when setting bounds on BrowserView (#38981)

Browse source 
This reverts commit 75f9573e53.
This commit is contained in:
Shelley Vohr 2023-07-06 09:50:08 +02:00 committed by GitHub
parent ec4c9024b9
commit 5a77c75753
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 9 additions and 39 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

39
shell/browser/native_browser_view_mac.mm
View file

@ -58,21 +58,9 @@ void NativeBrowserViewMac::SetBounds(const gfx::Rect& bounds) {
auto* view = iwc_view->GetNativeView().GetNativeNSView(); auto* view = iwc_view->GetNativeView().GetNativeNSView();
auto* superview = view.superview; auto* superview = view.superview;
const auto superview_height = superview ? superview.frame.size.height : 0; const auto superview_height = superview ? superview.frame.size.height : 0;
// We need to use the content rect to calculate the titlebar height if the
// superview is an framed NSWindow, otherwise it will be offset incorrectly by
// the height of the titlebar.
auto titlebar_height = 0;
if (auto* win = [superview window]) {
const auto content_rect_height =
[win contentRectForFrameRect:superview.frame].size.height;
titlebar_height = superview_height - content_rect_height;
}
auto new_height =
superview_height - bounds.y() - bounds.height() + titlebar_height;
view.frame = view.frame =
NSMakeRect(bounds.x(), new_height, bounds.width(), bounds.height()); NSMakeRect(bounds.x(), superview_height - bounds.y() - bounds.height(),
bounds.width(), bounds.height());
} }
gfx::Rect NativeBrowserViewMac::GetBounds() { gfx::Rect NativeBrowserViewMac::GetBounds() {
@ -80,23 +68,12 @@ gfx::Rect NativeBrowserViewMac::GetBounds() {
if (!iwc_view) if (!iwc_view)
return gfx::Rect(); return gfx::Rect();
NSView* view = iwc_view->GetNativeView().GetNativeNSView(); NSView* view = iwc_view->GetNativeView().GetNativeNSView();
auto* superview = view.superview; const int superview_height =
const int superview_height = superview ? superview.frame.size.height : 0; (view.superview) ? view.superview.frame.size.height : 0;
return gfx::Rect(
// We need to use the content rect to calculate the titlebar height if the view.frame.origin.x,
// superview is an framed NSWindow, otherwise it will be offset incorrectly by superview_height - view.frame.origin.y - view.frame.size.height,
// the height of the titlebar. view.frame.size.width, view.frame.size.height);
auto titlebar_height = 0;
if (auto* win = [superview window]) {
const auto content_rect_height =
[win contentRectForFrameRect:superview.frame].size.height;
titlebar_height = superview_height - content_rect_height;
}
auto new_height = superview_height - view.frame.origin.y -
view.frame.size.height + titlebar_height;
return gfx::Rect(view.frame.origin.x, new_height, view.frame.size.width,
view.frame.size.height);
} }
void NativeBrowserViewMac::SetBackgroundColor(SkColor color) { void NativeBrowserViewMac::SetBackgroundColor(SkColor color) {

9
spec/api-browser-view-spec.ts
View file

@ -146,14 +146,7 @@ describe('BrowserView module', () => {
}); });
describe('BrowserView.getBounds()', () => { describe('BrowserView.getBounds()', () => {
it('returns correct bounds on a framed window', () => { it('returns the current bounds', () => {
view = new BrowserView();
const bounds = { x: 10, y: 20, width: 30, height: 40 };
view.setBounds(bounds);
expect(view.getBounds()).to.deep.equal(bounds);
});
it('returns correct bounds on a frameless window', () => {
view = new BrowserView(); view = new BrowserView();
const bounds = { x: 10, y: 20, width: 30, height: 40 }; const bounds = { x: 10, y: 20, width: 30, height: 40 };
view.setBounds(bounds); view.setBounds(bounds);