Bug 178745 - Uncaught Exception: TypeError: null is not an object (evaluating 'this.treeOutline.isXMLMimeType')
Summary: Uncaught Exception: TypeError: null is not an object (evaluating 'this.treeOu...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Web Inspector (show other bugs)
Version: WebKit Nightly Build
Hardware: All All
: P2 Normal
Assignee: Joseph Pecoraro
URL: http://www.mtv.com/live-tv
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2017-10-24 13:48 PDT by John Wilander
Modified: 2017-11-15 12:48 PST (History)
6 users (show)

See Also:


Attachments
[PATCH] Proposed Fix (1.69 KB, patch)
2017-10-24 14:26 PDT, Joseph Pecoraro
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description John Wilander 2017-10-24 13:48:10 PDT
-------
Inspected URL:        http://www.mtv.com/live-tv
Loading completed:    true
Frontend User Agent:  Mozilla/5.0 (Macintosh; Intel Mac OS X 10_13_2) AppleWebKit/605.1.11+ (KHTML, like Gecko)

Uncaught Exceptions:
 - TypeError: null is not an object (evaluating 'this.treeOutline.isXMLMimeType') (at DOMTreeElement.js:1340:76)
    _nodeTitleInfo @ DOMTreeElement.js:1340:76
    updateTitle @ DOMTreeElement.js:1180:55
    _updateModifiedNodes @ DOMTreeUpdater.js:123:42
    performWork @ Utilities.js:1472:47
-------

* STEPS TO REPRODUCE
1. Reload http://www.mtv.com/live-tv with Web Inspector open, in the console error view. I had previously right-clicked a URL in the error log and copied it.

* NOTES
OpenSource r223571
Comment 1 Joseph Pecoraro 2017-10-24 13:54:23 PDT
I was able to reproduce.
Comment 2 Joseph Pecoraro 2017-10-24 14:26:10 PDT
Created attachment 324722 [details]
[PATCH] Proposed Fix
Comment 3 BJ Burg 2017-10-24 14:32:19 PDT
Comment on attachment 324722 [details]
[PATCH] Proposed Fix

I'd prefer an early continue, but tomato tomato.
Comment 4 Devin Rousso 2017-10-24 14:33:38 PDT
Comment on attachment 324722 [details]
[PATCH] Proposed Fix

View in context: https://bugs.webkit.org/attachment.cgi?id=324722&action=review

> Source/WebInspectorUI/ChangeLog:11
> +        An earlier tree element update might have caused this other tree element's
> +        waiting to be updated to actually be removed. So just skip past them.

This sentence is very awkward.  I had to read it three times before I understood it :|  Can you rephrase it?
Comment 5 WebKit Commit Bot 2017-10-24 14:55:40 PDT
Comment on attachment 324722 [details]
[PATCH] Proposed Fix

Clearing flags on attachment: 324722

Committed r223927: <https://trac.webkit.org/changeset/223927>
Comment 6 WebKit Commit Bot 2017-10-24 14:55:41 PDT
All reviewed patches have been landed.  Closing bug.
Comment 7 Radar WebKit Bug Importer 2017-11-15 12:48:22 PST
<rdar://problem/35568182>