New issue
Advanced search Search tips

Issue 594567 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner: ----
Closed: Apr 2016
Components:
EstimatedDays: ----
NextAction: ----
OS: Mac
Pri: 2
Type: Bug



Sign in to add a comment

MacViews: Tab contents is not visible when devtools is open.

Reported by k...@yandex-team.ru, Mar 14 2016

Issue description

UserAgent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_10_5) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/47.0.2526.73 YaBrowser/16.2.0.2567 (beta) Safari/537.36

Steps to reproduce the problem:
1.  Open docked DevTools on any page.

What is the expected behavior?
DevTools and tab contents is visible.
Tab contents is shown above DevTools view.

What went wrong?
Tab contents is not visible because it is positioned below DevTools native view.

Did this work before? No 

Chrome version: 47.0.2526.73  Channel: dev
OS Version: OS X 10.10.5
Flash Version: Shockwave Flash 20.0 r0
 

Comment 1 by meh...@chromium.org, Mar 14 2016

Components: -UI Internals>Views Platform>Apps>DevTools

Comment 2 by tapted@chromium.org, Mar 15 2016

Labels: Hotlist-MacViews
Status: Started (was: Unconfirmed)
Project Member

Comment 3 by bugdroid1@chromium.org, Mar 21 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/852a2ec06f9aac8f1b048d241a5bb8f9c5559d13

commit 852a2ec06f9aac8f1b048d241a5bb8f9c5559d13
Author: kirr <kirr@yandex-team.ru>
Date: Mon Mar 21 08:34:06 2016

Implement NativeWidgetMac::ReorderNativeViews.

NativeWidgetMac doesn't currently take into account native view order
when attaching NSViews in NativeViewHost and in view::View::ReorderChildView.
It is important when showing DevTools.

BUG= 594567 

Review URL: https://codereview.chromium.org/1796773003

Cr-Commit-Position: refs/heads/master@{#382259}

[modify] https://crrev.com/852a2ec06f9aac8f1b048d241a5bb8f9c5559d13/ui/views/cocoa/bridged_native_widget.h
[modify] https://crrev.com/852a2ec06f9aac8f1b048d241a5bb8f9c5559d13/ui/views/cocoa/bridged_native_widget.mm
[modify] https://crrev.com/852a2ec06f9aac8f1b048d241a5bb8f9c5559d13/ui/views/controls/native/native_view_host_mac.mm
[modify] https://crrev.com/852a2ec06f9aac8f1b048d241a5bb8f9c5559d13/ui/views/widget/native_widget_mac.mm
[modify] https://crrev.com/852a2ec06f9aac8f1b048d241a5bb8f9c5559d13/ui/views/widget/native_widget_mac_unittest.mm

Comment 4 by tapted@chromium.org, Apr 14 2016

Status: Fixed (was: Started)
Labels: -Hotlist-MacViews Proj-MacViews

Sign in to add a comment