New issue
Advanced search Search tips

Issue 624038 link

Starred by 3 users

Issue metadata

Status: Fixed
Owner:
Closed: Feb 2017
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 2
Type: Bug



Sign in to add a comment

Dev tools reopens drawer on each page load

Reported by philipp...@gmail.com, Jun 28 2016

Issue description

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

Steps to reproduce the problem:
1. Open Dev tools
2. Hide drawer to safe screen estate
3. Reload the page
4. Drawer with console is there again

What is the expected behavior?
If one closes the drawer, it should stay closed. Otherwise, one needs to close it after each page load in order to enlarge the rest of the dev tools window (e.g., to see elements or sources).

What went wrong?
The drawer shows again

Did this work before? Yes It does right now on Windows

Chrome version: 51.0.2704.103  Channel: stable
OS Version: OS X 10.10.5
Flash Version: Shockwave Flash 22.0 r0

This seems to be OS X related.

http://stackoverflow.com/questions/21420516/google-chrome-dev-tools-automatically-opens-the-drawer-pane
> This does not work on 51.0.2704.103 (64-bit) OS-X.
> Confirmed, the latest 51 update on OSX resets the drawer view on every reload. 
> Works on 51.0.2704.106 Windows. Unfortunately does not work on OS X
 
SkglH.png
9.3 KB View Download
Owner: l...@chromium.org
Interesting. I'm surprised you're having this behavior in Chrome 51.

Is there anything being logged to the console?

can you take a screenshot when this happens? 

Comment 2 by l...@chromium.org, Jul 1 2016

Labels: Needs-Feedback
Hi Paul,

here you go. There are two errors.

The second error seems to not count as a website error: there is a "1" on the right hand side of the red cross error indicator directly when the page loads, but the console appears only later in the drawer when the second error is also shown in the console tab then above the drawer while the 1 is not incremented.

I cross checked with Google Mail. There, I receive one error which is also indicated besides the red cross ("GET https://plus.google.com/serviceworker.js net::ERR_INSECURE_RESPONSE"), but the drawer is not forcefully shown.

So maybe it's the SourceMap error, which is handled by dev tools in a different way than the other errors? That source map is indeed a monster, with the original js being 6 MB. We will split that up later using require js... The framework of choice, Ember CLI, unfortunately throws everything into one file by default.
vendor.map.zip
1.8 MB Download
chrome-console.png
201 KB View Download

Comment 4 by l...@chromium.org, Jul 1 2016

Labels: -Needs-Feedback
Status: Assigned (was: Unconfirmed)
Hi philipp2k7@, thank you for the screenshot and the really helpful cross-check!

Our "Failed to parse sourcemap" is rigged to add a console error AND open the drawer, but it really shouldn't.

This core issue popped up in an earlier ticket #603750, but I made the wrong call back then.  Users who aren't interested in console errors shouldn't have a drawer forced open.  I'll start a CL to make it just add the error without opening the drawer.

Comment 5 by l...@chromium.org, Jul 1 2016

Labels: -OS-Mac OS-All
Disabling the source maps actually fixed the issue for me. The console is now not coming up on every page load, yeah :-)

Thanks for your support!
Status: WontFix (was: Assigned)
Status: Assigned (was: WontFix)
This is a really annoying problem :(

Any updates on when this will be fixed?

Disabling sourcemaps is definitely not a solution for me since I need sourcemaps
Owner: paulir...@chromium.org
Status: Started (was: Assigned)
The underlying problem is the sourcemap wasn't valid so we couldn't apply it. So even if we sort the error message being annoying, your sourcemap is still broken.

I've put in a fix for the annoying message though: https://codereview.chromium.org/2169213002
Project Member

Comment 11 by bugdroid1@chromium.org, Jul 22 2016

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

commit 6577e9c2199e226fb59c1b15fcc8dc89e245752b
Author: paulirish <paulirish@chromium.org>
Date: Fri Jul 22 06:19:30 2016

DevTools: Sourcemap failure shouldn't open console drawer.

BUG= 624038 , 497094 

Review-Url: https://codereview.chromium.org/2169213002
Cr-Commit-Position: refs/heads/master@{#407079}

[modify] https://crrev.com/6577e9c2199e226fb59c1b15fcc8dc89e245752b/third_party/WebKit/Source/devtools/front_end/sdk/SourceMap.js

Status: Fixed (was: Started)

Sign in to add a comment