Bug 14256 - Inspecting an element in an Image Document causes an assertion failure (hit while inspecting Acid3)
Summary: Inspecting an element in an Image Document causes an assertion failure (hit w...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Web Inspector (Deprecated) (show other bugs)
Version: 523.x (Safari 3)
Hardware: All All
: P2 Normal
Assignee: Nobody
URL:
Keywords: InRadar
: 16632 (view as bug list)
Depends on:
Blocks: 16755
  Show dependency tree / graph
 
Reported: 2007-06-20 20:45 PDT by Sam Weinig
Modified: 2008-11-14 07:42 PST (History)
7 users (show)

See Also:

Attachments
stack trace (34.02 KB, text/plain)
2007-06-20 20:50 PDT, Sam Weinig 		no flags Details
stack trace (34.02 KB, text/plain)
2007-06-20 20:50 PDT, Sam Weinig 		no flags Details
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Sam Weinig 2007-06-20 20:45:32 PDT 
To reproduce, right click an image and choose "Open image in New Window".  In the new window, activate the inspector and click the image left pane.  Await crash.  Crash log forthcoming.
Comment 1 Sam Weinig 2007-06-20 20:50:08 PDT 
Created attachment 15152 [details]
stack trace
Comment 2 Sam Weinig 2007-06-20 20:50:19 PDT 
Created attachment 15153 [details]
stack trace
Comment 3 Sam Weinig 2007-06-20 20:57:32 PDT 
This also happens with plugin documents, such as flash.
Comment 4 mitz 2007-12-30 11:27:59 PST 
*** Bug 16632 has been marked as a duplicate of this bug. ***
Comment 5 mitz 2008-01-08 09:12:13 PST 
The assertion failure results from calling FrameLoader::write(String) for a raw-data based document (an image) but ultimately to fix it one needs to know the expected behavior, i.e. what should the Inspector show in "source" view for non-text documents?
Comment 6 Adam Roben (:aroben) 2008-01-29 10:53:53 PST 
<rdar://problem/5712779>
Comment 7 Matt Lilek 2008-02-29 09:52:20 PST 
I hit this just loading LexisNexis Academic search results.
Comment 8 Darin Adler 2008-03-26 22:32:57 PDT 
Since people are seeing this when inspecting Acid3 we might want to fix it sooner rather than later.
Comment 9 Matt Lilek 2008-03-26 22:37:09 PDT 
(In reply to comment #8)
> Since people are seeing this when inspecting Acid3 we might want to fix it
> sooner rather than later.
> 

I don't actually see a crash in a release build, just a blank panel where the view source should be.

Actual assertion failure for bugzilla search posterity:
SHOULD NEVER BE REACHED
(/Users/matt/Code/WebKit/WebCore/loader/ImageDocument.cpp:91 virtual bool WebCore::ImageTokenizer::write(const WebCore::SegmentedString&, bool))
Comment 10 Adam Roben (:aroben) 2008-11-14 07:42:15 PST 
Seems to be fixed now.