Summary: | Web Inspector: REGRESSION (r171645): up/down key navigation of timeline sidebar tree elements is broken when scope bar filters are applied | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Brian Burg <burg> | ||||
Component: | Web Inspector | Assignee: | Matt Baker <mattbaker> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | commit-queue, graouts, joepeck, jonowells, mattbaker, nvasilyev, timothy, webkit-bug-importer | ||||
Priority: | P2 | Keywords: | InRadar | ||||
Version: | 528+ (Nightly build) | ||||||
Hardware: | All | ||||||
OS: | All | ||||||
Attachments: |
|
Description
Brian Burg
2015-03-04 17:31:15 PST
Caused by http://trac.webkit.org/changeset/171645. Looking over TreeOutline.js, the real problem seems to be with traverseNextTreeElement and traversePreviousTreeElement. The above revision changed the criteria for tree elements being considered revealed, causing the latent bug in element traversal to manifest. Created attachment 254609 [details]
[Patch] Proposed Fix
Comment on attachment 254609 [details] [Patch] Proposed Fix View in context: https://bugs.webkit.org/attachment.cgi?id=254609&action=review > Source/WebInspectorUI/UserInterface/Views/TreeOutline.js:1039 > + var depthChange = 0; Is this indent right? (In reply to comment #4) > Comment on attachment 254609 [details] > [Patch] Proposed Fix > > View in context: > https://bugs.webkit.org/attachment.cgi?id=254609&action=review > > > Source/WebInspectorUI/UserInterface/Views/TreeOutline.js:1039 > > + var depthChange = 0; > > Is this indent right? Could you clarify? Comment on attachment 254609 [details] [Patch] Proposed Fix View in context: https://bugs.webkit.org/attachment.cgi?id=254609&action=review >>> Source/WebInspectorUI/UserInterface/Views/TreeOutline.js:1039 >>> + var depthChange = 0; >> >> Is this indent right? > > Could you clarify? The patch on my phone looks like this like is a 4 spaces less than it should be. It could be the rendering or maybe there is a tab? Comment on attachment 254609 [details] [Patch] Proposed Fix View in context: https://bugs.webkit.org/attachment.cgi?id=254609&action=review >>>> Source/WebInspectorUI/UserInterface/Views/TreeOutline.js:1039 >>>> + var depthChange = 0; >>> >>> Is this indent right? >> >> Could you clarify? > > The patch on my phone looks like this like is a 4 spaces less than it should be. It could be the rendering or maybe there is a tab? Weird, it looks that way on my phone too. It looks correct on my laptop, and I confirmed that the indentation is identical to the line below it. Comment on attachment 254609 [details] [Patch] Proposed Fix Clearing flags on attachment: 254609 Committed r185629: <http://trac.webkit.org/changeset/185629> All reviewed patches have been landed. Closing bug. |