New issue
Advanced search Search tips

Issue 698103 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: Apr 2017
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux
Pri: 3
Type: Bug



Sign in to add a comment

DevTools: [regression] 17 messages hidden by filters

Project Member Reported by pfeldman@chromium.org, Mar 3 2017

Issue description

It is unclear why messages are filtered although user did not set up filtering manually.
 
Screenshot from 2017-03-02 17:33:40.png
10.1 KB View Download
Project Member

Comment 2 by bugdroid1@chromium.org, Mar 3 2017

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

commit 759ef7f0d55e443ac58ba3acd2d867d483713dfb
Author: pfeldman <pfeldman@chromium.org>
Date: Fri Mar 03 03:24:04 2017

DevTools: move the filtered items caption into the toolbar.

BUG= 698103 

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

[modify] https://crrev.com/759ef7f0d55e443ac58ba3acd2d867d483713dfb/third_party/WebKit/Source/devtools/front_end/console/ConsoleView.js

Labels: Merge-Request-58
Status: Started (was: Assigned)
Project Member

Comment 4 by sheriffbot@chromium.org, Mar 9 2017

Labels: -Merge-Request-58 Hotlist-Merge-Approved Merge-Approved-58
Your change meets the bar and is auto-approved for M58. Please go ahead and merge the CL to branch 3029 manually. Please contact milestone owner if you have questions.
Owners: amineer@(clank), cmasso@(bling), bhthompson@(cros), govind@(desktop)

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Project Member

Comment 5 by bugdroid1@chromium.org, Mar 10 2017

Labels: -merge-approved-58 merge-merged-3029
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/6d8ab8c125bc50b63af3417a1ada3ed6cc4cb6c8

commit 6d8ab8c125bc50b63af3417a1ada3ed6cc4cb6c8
Author: Pavel Feldman <pfeldman@chromium.org>
Date: Fri Mar 10 01:17:11 2017

DevTools: move the filtered items caption into the toolbar.

BUG= 698103 

Review-Url: https://codereview.chromium.org/2733533002
Cr-Commit-Position: refs/heads/master@{#454500}
(cherry picked from commit 759ef7f0d55e443ac58ba3acd2d867d483713dfb)

Review-Url: https://codereview.chromium.org/2743573004 .
Cr-Commit-Position: refs/branch-heads/3029@{#104}
Cr-Branched-From: 939b32ee5ba05c396eef3fd992822fcca9a2e262-refs/heads/master@{#454471}

[modify] https://crrev.com/6d8ab8c125bc50b63af3417a1ada3ed6cc4cb6c8/third_party/WebKit/Source/devtools/front_end/console/ConsoleView.js

Components: Infra>Git Platform>DevTools
Labels: Needs-Feedback
Tested this issue on Ubuntu 14.04 with chrome #58.0.3029.14

These are the steps followed.

1. Launch chrome
2. Open New Tab and go to DevTools->Elements then pressed Esc to open the console
   Observe there is anything in the Filter box.
3. Search anything in the Google Search.

Observed that even though filter box is empty, we are seeing filtered messages.

pfeldman@ could you please look into the attached screen cast and provide us the steps and expected behavior to verify the issue.

Thank You...


Issue 698103.mp4
1.6 MB View Download
Status: Fixed (was: Started)
This is now expected behavior - the status 'hidden by filters' message is next to the filter itself.

Sign in to add a comment