75467 - bidi- change Bi-di option back and forth crash

Status: RESOLVED WORKSFORME

(Core :: Layout: Text and Fonts, defect, P2)

Description

[Frank Tang]

1. using bidi win 0404 build
2. visit http://home.netscape.com
3. select "View:Bidi Option:Default Direction:Right to left"
4. select "View:Bidi Option:Default Direction:Left to Right"
Crash

Comment 1

[Frank Tang]

The limited stack trace show it crash inside GKCONTENT

Comment 2

[Maha Abou El-Rous]

here is the stack trace
StyleSetImpl::FindMatchingContext(StyleSetImpl * const 0x02dda870, 
nsIStyleContext * 0x034363a0, nsIStyleContext * * 0x007cbb64) line 1863 + 9 
bytes
StyleContextImpl::ShareStyleData() line 3858 + 37 bytes
StyleContextImpl::RemapStyle(StyleContextImpl * const 0x034363a0, 
nsIPresContext * 0x02dd31e0, int 1) line 3617 + 8 bytes
StyleContextImpl::RemapStyle(StyleContextImpl * const 0x0342f150, 
nsIPresContext * 0x02dd31e0, int 1) line 3625
StyleContextImpl::RemapStyle(StyleContextImpl * const 0x0342f260, 
nsIPresContext * 0x02dd31e0, int 1) line 3625
StyleContextImpl::RemapStyle(StyleContextImpl * const 0x03423b80, 
nsIPresContext * 0x02dd31e0, int 1) line 3625
StyleContextImpl::RemapStyle(StyleContextImpl * const 0x03427030, 
nsIPresContext * 0x02dd31e0, int 1) line 3625
StyleContextImpl::RemapStyle(StyleContextImpl * const 0x035060a0, 
nsIPresContext * 0x02dd31e0, int 1) line 3625
StyleContextImpl::RemapStyle(StyleContextImpl * const 0x03502610, 
nsIPresContext * 0x02dd31e0, int 1) line 3625
StyleContextImpl::RemapStyle(StyleContextImpl * const 0x03502a90, 
nsIPresContext * 0x02dd31e0, int 1) line 3625
StyleContextImpl::RemapStyle(StyleContextImpl * const 0x03502f10, 
nsIPresContext * 0x02dd31e0, int 1) line 3625
StyleContextImpl::RemapStyle(StyleContextImpl * const 0x03500580, 
nsIPresContext * 0x02dd31e0, int 1) line 3625
StyleContextImpl::RemapStyle(StyleContextImpl * const 0x03500be0, 
nsIPresContext * 0x02dd31e0, int 1) line 3625
StyleContextImpl::RemapStyle(StyleContextImpl * const 0x03500480, 
nsIPresContext * 0x02dd31e0, int 1) line 3625
StyleContextImpl::RemapStyle(StyleContextImpl * const 0x02b17ed0, 
nsIPresContext * 0x02dd31e0, int 1) line 3625
StyleContextImpl::RemapStyle(StyleContextImpl * const 0x02b12490, 
nsIPresContext * 0x02dd31e0, int 1) line 3625
StyleContextImpl::RemapStyle(StyleContextImpl * const 0x02b0ff30, 
nsIPresContext * 0x02dd31e0, int 1) line 3625
StyleContextImpl::RemapStyle(StyleContextImpl * const 0x02aab320, 
nsIPresContext * 0x02dd31e0, int 1) line 3625
StyleContextImpl::RemapStyle(StyleContextImpl * const 0x02a30380, 
nsIPresContext * 0x02dd31e0, int 1) line 3625
nsPresContext::RemapStyleAndReflow(nsPresContext * const 0x02dd31e0) line 387
nsPresContext::SetBidi(nsPresContext * const 0x02dd31e0, unsigned int 16855313, 
int 1) line 1442
DocumentViewerImpl::SetBidiOptions(DocumentViewerImpl * const 0x02e0c3cc, 
unsigned int 16855313) line 3365
DocumentViewerImpl::SetBidiTextDirection(DocumentViewerImpl * const 0x02e0c3cc, 
unsigned char 1) line 3192
XPTC_InvokeByIndex(nsISupports * 0x02e0c3cc, unsigned int 18, unsigned int 1, 
nsXPTCVariant * 0x007cd880) line 139
nsXPCWrappedNativeClass::CallWrappedMethod(JSContext * 0x01ec76d0, 
nsXPCWrappedNative * 0x03551100, const XPCNativeMemberDescriptor * 0x035511cc, 
nsXPCWrappedNativeClass::CallMode CALL_SETTER, unsigned int 1, long * 
0x007ce51c, long * 0x00000000) line 934 + 42 bytes
nsXPCWrappedNativeClass::SetAttributeFromJSVal(JSContext * 0x01ec76d0, 
nsXPCWrappedNative * 0x03551100, const XPCNativeMemberDescriptor * 0x035511cc, 
long * 0x007ce51c) line 970
WrappedNative_SetProperty(JSContext * 0x01ec76d0, JSObject * 0x00ab8f78, long 
37294912, long * 0x007ce51c) line 447 + 24 bytes
js_Interpret(JSContext * 0x01ec76d0, long * 0x007ce6d4) line 2524 + 1939 bytes
js_Invoke(JSContext * 0x01ec76d0, unsigned int 1, unsigned int 2) line 803 + 13 
bytes
js_InternalInvoke(JSContext * 0x01ec76d0, JSObject * 0x00ae7c08, long 49480728, 
unsigned int 0, unsigned int 1, long * 0x007ce86c, long * 0x007ce7fc) line 875 
+ 20 bytes
JS_CallFunctionValue(JSContext * 0x01ec76d0, JSObject * 0x00ae7c08, long 
49480728, unsigned int 1, long * 0x007ce86c, long * 0x007ce7fc) line 3340 + 31 
bytes
nsJSContext::CallEventHandler(nsJSContext * const 0x01ec7880, void * 
0x00ae7c08, void * 0x02f30418, unsigned int 1, void * 0x007ce86c, int * 
0x007ce868, int 0) line 940 + 33 bytes
nsJSEventListener::HandleEvent(nsIDOMEvent * 0x03586474) line 154 + 64 bytes
nsEventListenerManager::HandleEventSubType(nsListenerStruct * 0x0258abd0, 
nsIDOMEvent * 0x03586474, nsIDOMEventTarget * 0x02591be8, unsigned int 8, 
unsigned int 7) line 920 + 19 bytes
nsEventListenerManager::HandleEvent(nsIPresContext * 0x02655ae0, nsEvent * 
0x007cf170, nsIDOMEvent * * 0x007cf01c, nsIDOMEventTarget * 0x02591be8, 
unsigned int 7, nsEventStatus * 0x007cf1b8) line 1848 + 39 bytes
nsXULElement::HandleDOMEvent(nsXULElement * const 0x02591be0, nsIPresContext * 
0x02655ae0, nsEvent * 0x007cf170, nsIDOMEvent * * 0x007cf01c, unsigned int 1, 
nsEventStatus * 0x007cf1b8) line 3671
PresShell::HandleDOMEventWithTarget(PresShell * const 0x02656b60, nsIContent * 
0x02591be0, nsEvent * 0x007cf170, nsEventStatus * 0x007cf1b8) line 5172 + 39 
bytes
nsMenuFrame::Execute() line 1422
nsMenuFrame::HandleEvent(nsMenuFrame * const 0x02fec6ec, nsIPresContext * 
0x02655ae0, nsGUIEvent * 0x007cf5c0, nsEventStatus * 0x007cf4b4) line 399
PresShell::HandleEventInternal(nsEvent * 0x007cf5c0, nsIView * 0x0356dbc0, 
unsigned int 1, nsEventStatus * 0x007cf4b4) line 5140 + 41 bytes
PresShell::HandleEvent(PresShell * const 0x02656b64, nsIView * 0x0356dbc0, 
nsGUIEvent * 0x007cf5c0, nsEventStatus * 0x007cf4b4, int 0, int & 1) line 5052 
+ 25 bytes
nsView::HandleEvent(nsView * const 0x0356dbc0, nsGUIEvent * 0x007cf5c0, 
unsigned int 8, nsEventStatus * 0x007cf4b4, int 0, int & 1) line 379
nsView::HandleEvent(nsView * const 0x02e0d970, nsGUIEvent * 0x007cf5c0, 
unsigned int 8, nsEventStatus * 0x007cf4b4, int 0, int & 1) line 352
nsView::HandleEvent(nsView * const 0x0353b230, nsGUIEvent * 0x007cf5c0, 
unsigned int 8, nsEventStatus * 0x007cf4b4, int 0, int & 1) line 352
nsView::HandleEvent(nsView * const 0x026552d0, nsGUIEvent * 0x007cf5c0, 
unsigned int 28, nsEventStatus * 0x007cf4b4, int 1, int & 1) line 352
nsViewManager::DispatchEvent(nsViewManager * const 0x02655470, nsGUIEvent * 
0x007cf5c0, nsEventStatus * 0x007cf4b4) line 1996
HandleEvent(nsGUIEvent * 0x007cf5c0) line 68
nsWindow::DispatchEvent(nsWindow * const 0x02d9f754, nsGUIEvent * 0x007cf5c0, 
nsEventStatus & nsEventStatus_eIgnore) line 695 + 10 bytes
nsWindow::DispatchWindowEvent(nsGUIEvent * 0x007cf5c0) line 716
nsWindow::DispatchMouseEvent(unsigned int 301, nsPoint * 0x00000000) line 4018 
+ 21 bytes
ChildWindow::DispatchMouseEvent(unsigned int 301, nsPoint * 0x00000000) line 
4263
nsWindow::ProcessMessage(unsigned int 514, unsigned int 0, long 655476, long * 
0x007cf978) line 3000 + 24 bytes
nsWindow::WindowProc(HWND__ * 0x00000474, unsigned int 514, unsigned int 0, 
long 655476) line 950 + 27 bytes
KERNEL32! bff7363b()
KERNEL32! bff9446b()
007c89fe()

Comment 3

[Frank Tang]

set this to moz0.9.1. We could fix this after we turn IBMBIDI on and before we 
land the xpfe ui changes. Without the xpfe ui change, this bug won't be 
reproduceable anyway. 

Comment 4

[Frank Tang]

erik resign. reassign all his bug to ftang for now.

Comment 5

[Frank Tang]

mark this assign

Comment 6

[Frank Tang]

take out moz0.9.1

Comment 7

[Frank Tang]

reassign back to simon

Comment 8

[Andreas Becker]

Changing QA contact to [e-mail] for now.

Comment 9

[Frank Tang]

move to "Bidi:Hebrew/Arabic"

Comment 10

[Zach Lipton [:zach]]

Mass-move all BiDi Hebrew and Arabic qa to me, [e-mail]. 
Thank you Gilad for your service to this component, and best of luck to you 
in the future.

Sholom. 

Comment 11

[Eyal Rozenberg]

The menu items mentioned in the bug description does not exist by now... and
switching page direction works, specifically for home.netscape.com (although
it's probably not the same page as it was 4 years ago). So unless someone says
otherwise, this looks like it should be marked invalid.

Comment 12

[Uri Bernstein]

Resolvig WORKSFORME per comment #11, and per the fact that the classes in the
stacktrace from comment #2 no longer exist. Also, no duplicates over 4 years
seem to indicate the problem no longer exists.

Please re-open if reproducable.