Closed
Bug 309702
Opened 19 years ago
Closed 19 years ago
Crashes when adblocking is enabled [@ nsPluginStreamListener::OnStartRequest]
Categories
(Core :: DOM: Core & HTML, defect)
Tracking
()
RESOLVED
FIXED
mozilla1.9alpha1
People
(Reporter: sugar.waffle, Assigned: Biesinger)
References
()
Details
(Keywords: crash, regression)
Crash Data
Attachments
(3 files)
1.83 KB,
text/plain
|
Details | |
1009 bytes,
patch
|
bzbarsky
:
superreview+
|
Details | Diff | Splinter Review |
231 bytes,
text/html
|
Details |
Incident ID: TB9634495X, TB9634658E, TB9634714G
It crashes by some pages when adblock is enable.
Reproducible: Always
Steps to Reproduce:
1. Adblock is turned on with Preferences.
2. open URL
3. The following links are clicked with a left frame.
http://live16.2ch.net/liventv/
Mac OS X 10.3.9
trunk build: 2005092208 (v1.0+)
Keywords: crash
Summary: It crashes when adblock is enable. → Crashes when adblocking is enabled [@ nsPluginStreamListener::OnStartRequest]
Comment 2•19 years ago
|
||
the crash is in the plugin code. why would ad blocking affect that?
Comment 3•19 years ago
|
||
(In reply to comment #2)
> the crash is in the plugin code. why would ad blocking affect that?
Maybe we block a Flash plugin, and an iframe ends up with no content or something.
I wasn't able to find the liventv link in the left frame (too many links!).
Assignee | ||
Comment 4•19 years ago
|
||
this should fix it, I believe...
Assignee: mikepinkerton → cbiesinger
Status: UNCONFIRMED → ASSIGNED
Attachment #197188 -
Flags: superreview?(bzbarsky)
Attachment #197188 -
Flags: review?(bzbarsky)
Comment 5•19 years ago
|
||
Can we at least get a testcase?
Updated•19 years ago
|
Attachment #197188 -
Flags: superreview?(bzbarsky)
Attachment #197188 -
Flags: superreview+
Attachment #197188 -
Flags: review?(bzbarsky)
Attachment #197188 -
Flags: review+
Assignee | ||
Comment 6•19 years ago
|
||
Testcase. This crashes in nsPluginStreamListener::OnStartRequest so I believe
it's this bug. (any my patch does fix it)
Assignee | ||
Updated•19 years ago
|
Component: General → DOM
Flags: review+
Product: Camino → Core
Assignee | ||
Comment 7•19 years ago
|
||
Checking in content/html/document/src/nsPluginDocument.cpp;
/cvsroot/mozilla/content/html/document/src/nsPluginDocument.cpp,v <--
nsPluginDocument.cpp
new revision: 1.23; previous revision: 1.22
done
Status: ASSIGNED → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
Assignee | ||
Updated•19 years ago
|
Target Milestone: --- → mozilla1.9alpha
Assignee | ||
Updated•19 years ago
|
Keywords: regression
Comment 9•19 years ago
|
||
Do we need this on the branch?
Assignee | ||
Comment 10•19 years ago
|
||
no, bug 1156 was trunk only.
Updated•14 years ago
|
Crash Signature: [@ nsPluginStreamListener::OnStartRequest]
Updated•6 years ago
|
Component: DOM → DOM: Core & HTML
You need to log in
before you can comment on or make changes to this bug.
Description
•