It's trying to make a content view out of it, but that ain't gonna work. It's inside a try-catch block, so it doesn't actually break anything.
Sounds like we just need to add the class to the early return in ReourceSidebarPanel._treeElementSelected. Or come up with a special function on these tree elements (isFolder()?) that we can check.
Created attachment 258014 [details] Proposed Fix
Comment on attachment 258014 [details] Proposed Fix This seems fine. I guess I wonder why ApplicationCacheManifestTreeElement is not a subclass of FolderTreeElement if it acts like a folder.
(In reply to comment #3) > Comment on attachment 258014 [details] > Proposed Fix > > This seems fine. I guess I wonder why ApplicationCacheManifestTreeElement is > not a subclass of FolderTreeElement if it acts like a folder. It probably should be, or we should have a content view for the appcache manifest and change what appears for each frame that uses the same manifest. But there are larger problems with our app cache support (there are no real model objects to speak of; the content view directly unpacks protocol payloads), so I'd defer on further refactoring.
Comment on attachment 258014 [details] Proposed Fix Clearing flags on attachment: 258014 Committed r187872: <http://trac.webkit.org/changeset/187872>
All reviewed patches have been landed. Closing bug.