mPosition is not correctly reset at the start of a new request

VERIFIED FIXED

Status

()

Core
Plug-ins
VERIFIED FIXED
17 years ago
17 years ago

People

(Reporter: dougt, Assigned: dougt)

Tracking

Trunk
x86
Windows 2000
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: PDT+)

Attachments

(1 attachment)

(Assignee)

Description

17 years ago
mPosition is not correctly reset at the start of a new request
(Assignee)

Comment 1

17 years ago
Created attachment 41268 [details] [diff] [review]
fixes mPosition reset problem
(Assignee)

Updated

17 years ago
Blocks: 76892

Comment 2

17 years ago
r=peterl
No longer blocks: 76892

Comment 3

17 years ago
r=av

Comment 4

17 years ago
Patch looks fine - sr=attinasi
(Assignee)

Comment 5

17 years ago
landed on trunk

Checking in ns4xPluginInstance.cpp;
/cvsroot/mozilla/modules/plugin/nglsrc/ns4xPluginInstance.cpp,v  <--  
ns4xPluginInstance.cpp
new revision: 1.54; previous revision: 1.53
done
Checking in ns4xPluginStreamListener.h;
/cvsroot/mozilla/modules/plugin/nglsrc/ns4xPluginStreamListener.h,v  <--  
ns4xPluginStreamListener.h
new revision: 1.3; previous revision: 1.2
done
Keywords: vbranch

Comment 6

17 years ago
I'm giving this PDT+ please check in tonite on the branch.  

Shouldn't this currently have the vtrunk keyword instead of the vbranch keyword?
 (Maybe I've *also* misunderstood this vtrunk/vbranch system, but I thought
vtrunk meant "verify this on the trunk" which is the current status of this bug.)
Whiteboard: PDT+
(Assignee)

Comment 7

17 years ago
fixed on branch.
Status: NEW → RESOLVED
Last Resolved: 17 years ago
Resolution: --- → FIXED

Comment 8

17 years ago
stamping VERIFIED
Status: RESOLVED → VERIFIED
Keywords: vbranch
You need to log in before you can comment on or make changes to this bug.