WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
Bug 162850
Web Inspector: Exception thrown when hovering network waterfall graphs during reload
https://bugs.webkit.org/show_bug.cgi?id=162850
Summary
Web Inspector: Exception thrown when hovering network waterfall graphs during...
Matt Baker
Reported
2016-10-01 18:07:45 PDT
Summary: Exception thrown when hovering network waterfall graphs during reload. Steps to Reproduce: 1. Goto Network tab, reload inspected page 2. Hover over a record bar element in the grid 3. ⌘R to reload inspected page, while moving cursor in and out of the graph element => TypeError: null is not an object (evaluating 'this.dataGrid') (at ResourceTimelineDataGridNode.js:377:26) Note: The record bar's mouseleave event listener doesn't check that the node's grid reference is valid before using it. The node's grid reference is set to null when a node is removed from the grid, but before it's removed from the DOM. This occurs during page load as resources are reordered in the grid.
Attachments
Patch
(1.80 KB, patch)
2016-10-01 18:12 PDT
,
Matt Baker
no flags
Details
Formatted Diff
Diff
View All
Add attachment
proposed patch, testcase, etc.
Radar WebKit Bug Importer
Comment 1
2016-10-01 18:07:56 PDT
<
rdar://problem/28579653
>
Matt Baker
Comment 2
2016-10-01 18:12:19 PDT
Created
attachment 290457
[details]
Patch
Blaze Burg
Comment 3
2016-10-02 13:40:29 PDT
Comment on
attachment 290457
[details]
Patch View in context:
https://bugs.webkit.org/attachment.cgi?id=290457&action=review
rs=me
> Source/WebInspectorUI/ChangeLog:12 > + (WebInspector.ResourceTimelineDataGridNode):
This line seems unnecessary.
WebKit Commit Bot
Comment 4
2016-10-02 14:01:32 PDT
Comment on
attachment 290457
[details]
Patch Clearing flags on attachment: 290457 Committed
r206722
: <
http://trac.webkit.org/changeset/206722
>
WebKit Commit Bot
Comment 5
2016-10-02 14:01:35 PDT
All reviewed patches have been landed. Closing bug.
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug