<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugs.webkit.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.4.1"
          urlbase="https://bugs.webkit.org/"
          
          maintainer="admin@webkit.org"
>

    <bug>
          <bug_id>6637</bug_id>
          
          <creation_ts>2006-01-17 19:51:20 -0800</creation_ts>
          <short_desc>selecting node in Inspector after closing window crashes Safari</short_desc>
          <delta_ts>2006-06-11 04:57:40 -0700</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>Web Inspector (Deprecated)</component>
          <version>420+</version>
          <rep_platform>Mac</rep_platform>
          <op_sys>OS X 10.4</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords>InRadar</keywords>
          <priority>P1</priority>
          <bug_severity>Critical</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Michael Croft">mcroft</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>alice.barraclough</cc>
    
    <cc>ddkilzer</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>28761</commentid>
    <comment_count>0</comment_count>
    <who name="Michael Croft">mcroft</who>
    <bug_when>2006-01-17 19:51:20 -0800</bug_when>
    <thetext>Select an element in Safari and Inspect it.
Select the element in the node selection window
Close Safari window.
Select another element in the Web Inspector.
Crash.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>28762</commentid>
    <comment_count>1</comment_count>
    <who name="Michael Croft">mcroft</who>
    <bug_when>2006-01-17 19:51:36 -0800</bug_when>
    <thetext>May be related to 6635</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>28792</commentid>
    <comment_count>2</comment_count>
    <who name="Mark Rowe (bdash)">mrowe</who>
    <bug_when>2006-01-17 22:11:54 -0800</bug_when>
    <thetext>Confirmed with ToT.  This looks unrelated to bug 6635.  Backtrace from the crash is as follows:

Exception:  EXC_BAD_ACCESS (0x0001)
Codes:      KERN_PROTECTION_FAILURE (0x0002) at 0x00000140

Thread 0 Crashed:
0   com.apple.WebCore        	0x0113bc3c Frame::jScript() + 20
1   libobjc.A.dylib          	0x909c01f4 objc_msgSend + 244
2   com.apple.WebCore        	0x01191bf0 -[WebScriptObject(WebScriptObjectInternal) 
_initializeScriptDOMNodeImp] + 152
3   com.apple.JavaScriptCore 	0x0014f754 -[WebScriptObject _imp] + 60
4   com.apple.JavaScriptCore 	0x0014c78c KJS::Bindings::convertObjcValueToValue(KJS::ExecState*, 
void*, KJS::Bindings::ObjcValueType) + 516
5   com.apple.JavaScriptCore 	0x0014b9f8 KJS::Bindings::ObjcInstance::invokeMethod(KJS::ExecState*, 
KJS::Bindings::MethodList const&amp;, KJS::List const&amp;) + 820
6   com.apple.JavaScriptCore 	0x001482a8 KJS::RuntimeMethod::callAsFunction(KJS::ExecState*, 
KJS::JSObject*, KJS::List const&amp;) + 252
7   com.apple.JavaScriptCore 	0x0012baec KJS::JSObject::call(KJS::ExecState*, KJS::JSObject*, KJS::List 
const&amp;) + 96
8   com.apple.JavaScriptCore 	0x00125a90 KJS::FunctionCallDotNode::evaluate(KJS::ExecState*) + 464
9   com.apple.JavaScriptCore 	0x001236d4 KJS::VarDeclNode::evaluate(KJS::ExecState*) + 88
10  com.apple.JavaScriptCore 	0x0011f8f0 KJS::VarDeclListNode::evaluate(KJS::ExecState*) + 84
11  com.apple.JavaScriptCore 	0x00122684 KJS::VarStatementNode::execute(KJS::ExecState*) + 104
12  com.apple.JavaScriptCore 	0x00122d5c KJS::SourceElementsNode::execute(KJS::ExecState*) + 188
13  com.apple.JavaScriptCore 	0x00122604 KJS::BlockNode::execute(KJS::ExecState*) + 132
14  com.apple.JavaScriptCore 	0x00111f90 KJS::DeclaredFunctionImp::execute(KJS::ExecState*) + 56
15  com.apple.JavaScriptCore 	0x001121d0 KJS::FunctionImp::callAsFunction(KJS::ExecState*, 
KJS::JSObject*, KJS::List const&amp;) + 444
16  com.apple.JavaScriptCore 	0x0012baec KJS::JSObject::call(KJS::ExecState*, KJS::JSObject*, KJS::List 
const&amp;) + 96
17  com.apple.JavaScriptCore 	0x00126b94 KJS::FunctionCallResolveNode::evaluate(KJS::ExecState*) + 
456
18  com.apple.JavaScriptCore 	0x00122474 KJS::ExprStatementNode::execute(KJS::ExecState*) + 104
19  com.apple.JavaScriptCore 	0x00122d5c KJS::SourceElementsNode::execute(KJS::ExecState*) + 188
20  com.apple.JavaScriptCore 	0x00122604 KJS::BlockNode::execute(KJS::ExecState*) + 132
21  com.apple.JavaScriptCore 	0x001127ec KJS::GlobalFuncImp::callAsFunction(KJS::ExecState*, 
KJS::JSObject*, KJS::List const&amp;) + 652
22  com.apple.JavaScriptCore 	0x0012baec KJS::JSObject::call(KJS::ExecState*, KJS::JSObject*, KJS::List 
const&amp;) + 96
23  com.apple.JavaScriptCore 	0x00126b94 KJS::FunctionCallResolveNode::evaluate(KJS::ExecState*) + 
456
24  com.apple.JavaScriptCore 	0x00122474 KJS::ExprStatementNode::execute(KJS::ExecState*) + 104
25  com.apple.JavaScriptCore 	0x00122e04 KJS::SourceElementsNode::execute(KJS::ExecState*) + 356
26  com.apple.JavaScriptCore 	0x00122604 KJS::BlockNode::execute(KJS::ExecState*) + 132
27  com.apple.JavaScriptCore 	0x00111f90 KJS::DeclaredFunctionImp::execute(KJS::ExecState*) + 56
28  com.apple.JavaScriptCore 	0x001121d0 KJS::FunctionImp::callAsFunction(KJS::ExecState*, 
KJS::JSObject*, KJS::List const&amp;) + 444
29  com.apple.JavaScriptCore 	0x0012baec KJS::JSObject::call(KJS::ExecState*, KJS::JSObject*, KJS::List 
const&amp;) + 96
30  com.apple.JavaScriptCore 	0x00150b28 -[WebScriptObject callWebScriptMethod:withArguments:] + 
376
31  com.apple.WebKit         	0x00378d4c -[WebInspector(WebInspectorPrivate) _update] + 96
32  com.apple.WebKit         	0x00377294 -[WebInspector setFocusedDOMNode:] + 304
33  com.apple.WebKit         	0x00379864 -[WebInspector(WebInspectorPrivate) 
outlineViewSelectionDidChange:] + 136
34  com.apple.Foundation     	0x928d4018 _nsnote_callback + 180
35  com.apple.CoreFoundation 	0x907854c4 __CFXNotificationPost + 368
36  com.apple.CoreFoundation 	0x9077d5a0 _CFXNotificationPostNotification + 684
37  com.apple.Foundation     	0x928be420 -[NSNotificationCenter 
postNotificationName:object:userInfo:] + 92
38  com.apple.AppKit         	0x93721518 -[NSTableView _enableSelectionPostingAndPost] + 424
39  com.apple.AppKit         	0x937365f4 -[NSTableView mouseDown:] + 2768
40  com.apple.AppKit         	0x937f5a30 -[NSOutlineView mouseDown:] + 680
41  com.apple.WebCore        	0x01147014 MacFrame::passMouseDownEventToWidget(QWidget*) + 
832
42  com.apple.WebCore        	0x0114a7a0 MacFrame::khtmlMousePressEvent
(WebCore::MousePressEvent*) + 144
43  com.apple.WebCore        	0x01278194 ObjectContents::event(QEvent*) + 28
44  com.apple.WebCore        	0x01154f64 KHTMLView::viewportMousePressEvent(QMouseEvent*) + 
564
45  com.apple.WebCore        	0x0114d9c0 MacFrame::mouseDown(NSEvent*) + 544
46  com.apple.WebKit         	0x00340210 -[WebHTMLView mouseDown:] + 272
47  com.apple.AppKit         	0x936d5eb0 -[NSWindow sendEvent:] + 4616
48  com.apple.WebKit         	0x0037c8fc -[WebInspectorPanel sendEvent:] + 764
49  com.apple.AppKit         	0x9367eef4 -[NSApplication sendEvent:] + 4172
50  com.apple.Safari         	0x00021c64 0x1000 + 134244
51  com.apple.AppKit         	0x93676330 -[NSApplication run] + 508
52  com.apple.AppKit         	0x93766e68 NSApplicationMain + 452
53  com.apple.Safari         	0x0005cfdc 0x1000 + 376796
54  com.apple.Safari         	0x0005ce80 0x1000 + 376448
</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>30232</commentid>
    <comment_count>3</comment_count>
    <who name="Alice Liu">alice.barraclough</who>
    <bug_when>2006-01-26 18:26:56 -0800</bug_when>
    <thetext>&lt;rdar://problem/4417355&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>33408</commentid>
    <comment_count>4</comment_count>
    <who name="Darin Adler">darin</who>
    <bug_when>2006-02-19 12:59:06 -0800</bug_when>
    <thetext>I tried, and could not reproduce this problem today.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>33437</commentid>
    <comment_count>5</comment_count>
    <who name="Michael Croft">mcroft</who>
    <bug_when>2006-02-19 20:23:29 -0800</bug_when>
    <thetext>(In reply to comment #4)
&gt; I tried, and could not reproduce this problem today.
&gt; 
Still happening for me with current TOT.

Date/Time:      2006-02-19 23:18:27.652 -0500
OS Version:     10.4.5 (Build 8H14)
Report Version: 4

Command: Safari
Path:    /Applications/Safari.app/Contents/MacOS/Safari
Parent:  bash [277]

Version:        2.0.3 (417.8)
Build Version:  1
Project Name:   WebBrowser
Source Version: 4170800

PID:    4265
Thread: 0

Exception:  EXC_BAD_ACCESS (0x0001)
Codes:      KERN_PROTECTION_FAILURE (0x0002) at 0x00000044

Thread 0 Crashed:
0   com.apple.WebCore        	0x0199139c WebCore::Frame::jScript() + 28 (Frame.cpp:375)
1   com.apple.WebCore        	0x01a0cbe4 -[WebScriptObject(WebScriptObjectInternal) _initializeScriptDOMNodeImp] + 148 (DOMInternal.mm:183)
2   com.apple.JavaScriptCore 	0x01068c54 -[WebScriptObject _imp] + 68 (WebScriptObject.mm:96)
3   com.apple.JavaScriptCore 	0x01065314 KJS::Bindings::convertObjcValueToValue(KJS::ExecState*, void*, KJS::Bindings::ObjcValueType) + 500 (objc_utility.mm:238)
4   com.apple.JavaScriptCore 	0x01064348 KJS::Bindings::ObjcInstance::invokeMethod(KJS::ExecState*, KJS::Bindings::MethodList const&amp;, KJS::List const&amp;) + 1000 (objc_instance.mm:230)
5   com.apple.JavaScriptCore 	0x0105f9e4 KJS::RuntimeMethod::callAsFunction(KJS::ExecState*, KJS::JSObject*, KJS::List const&amp;) + 276 (runtime_method.cpp:93)
6   com.apple.JavaScriptCore 	0x0103bc24 KJS::JSObject::call(KJS::ExecState*, KJS::JSObject*, KJS::List const&amp;) + 100 (object.cpp:99)
7   com.apple.JavaScriptCore 	0x0102d7bc KJS::FunctionCallDotNode::evaluate(KJS::ExecState*) + 524 (nodes.cpp:695)
8   com.apple.JavaScriptCore 	0x0102b6d8 KJS::VarDeclNode::evaluate(KJS::ExecState*) + 88 (nodes.cpp:1523)
9   com.apple.JavaScriptCore 	0x0102b61c KJS::VarDeclListNode::evaluate(KJS::ExecState*) + 76 (nodes.cpp:1570)
10  com.apple.JavaScriptCore 	0x01031ef8 KJS::VarStatementNode::execute(KJS::ExecState*) + 104 (nodes.cpp:1594)
11  com.apple.JavaScriptCore 	0x0103545c KJS::SourceElementsNode::execute(KJS::ExecState*) + 252 (nodes.cpp:2390)
12  com.apple.JavaScriptCore 	0x01032098 KJS::BlockNode::execute(KJS::ExecState*) + 152 (nodes.cpp:1627)
13  com.apple.JavaScriptCore 	0x0101a0f8 KJS::DeclaredFunctionImp::execute(KJS::ExecState*) + 56 (completion.h:53)
14  com.apple.JavaScriptCore 	0x01019a10 KJS::FunctionImp::callAsFunction(KJS::ExecState*, KJS::JSObject*, KJS::List const&amp;) + 464 (interpreter.h:470)
15  com.apple.JavaScriptCore 	0x0103bc24 KJS::JSObject::call(KJS::ExecState*, KJS::JSObject*, KJS::List const&amp;) + 100 (object.cpp:99)
16  com.apple.JavaScriptCore 	0x0102df5c KJS::FunctionCallResolveNode::evaluate(KJS::ExecState*) + 524 (nodes.cpp:602)
17  com.apple.JavaScriptCore 	0x01032168 KJS::ExprStatementNode::execute(KJS::ExecState*) + 104 (nodes.cpp:1651)
18  com.apple.JavaScriptCore 	0x0103545c KJS::SourceElementsNode::execute(KJS::ExecState*) + 252 (nodes.cpp:2390)
19  com.apple.JavaScriptCore 	0x01032098 KJS::BlockNode::execute(KJS::ExecState*) + 152 (nodes.cpp:1627)
20  com.apple.JavaScriptCore 	0x01018b7c KJS::GlobalFuncImp::callAsFunction(KJS::ExecState*, KJS::JSObject*, KJS::List const&amp;) + 748 (interpreter.h:470)
21  com.apple.JavaScriptCore 	0x0103bc24 KJS::JSObject::call(KJS::ExecState*, KJS::JSObject*, KJS::List const&amp;) + 100 (object.cpp:99)
22  com.apple.JavaScriptCore 	0x0102df5c KJS::FunctionCallResolveNode::evaluate(KJS::ExecState*) + 524 (nodes.cpp:602)
23  com.apple.JavaScriptCore 	0x01032168 KJS::ExprStatementNode::execute(KJS::ExecState*) + 104 (nodes.cpp:1651)
24  com.apple.JavaScriptCore 	0x01035548 KJS::SourceElementsNode::execute(KJS::ExecState*) + 488 (completion.h:53)
25  com.apple.JavaScriptCore 	0x01032098 KJS::BlockNode::execute(KJS::ExecState*) + 152 (nodes.cpp:1627)
26  com.apple.JavaScriptCore 	0x0101a0f8 KJS::DeclaredFunctionImp::execute(KJS::ExecState*) + 56 (completion.h:53)
27  com.apple.JavaScriptCore 	0x01019a10 KJS::FunctionImp::callAsFunction(KJS::ExecState*, KJS::JSObject*, KJS::List const&amp;) + 464 (interpreter.h:470)
28  com.apple.JavaScriptCore 	0x0103bc24 KJS::JSObject::call(KJS::ExecState*, KJS::JSObject*, KJS::List const&amp;) + 100 (object.cpp:99)
29  com.apple.JavaScriptCore 	0x0106a570 -[WebScriptObject callWebScriptMethod:withArguments:] + 448 (WebScriptObject.mm:210)
30  com.apple.WebKit         	0x00383104 -[WebInspector(WebInspectorPrivate) _update] + 100 (WebInspector.m:644)
31  com.apple.WebKit         	0x00381414 -[WebInspector setFocusedDOMNode:] + 324 (WebInspector.m:226)
32  com.apple.WebKit         	0x00383d38 -[WebInspector(WebInspectorPrivate) outlineViewSelectionDidChange:] + 136 (WebInspector.m:917)
33  com.apple.Foundation     	0x928e6cf8 _nsnote_callback + 180
34  com.apple.CoreFoundation 	0x907844c4 __CFXNotificationPost + 368
35  com.apple.CoreFoundation 	0x9077c5a0 _CFXNotificationPostNotification + 684
36  com.apple.Foundation     	0x928d1100 -[NSNotificationCenter postNotificationName:object:userInfo:] + 92
37  com.apple.AppKit         	0x93734518 -[NSTableView _enableSelectionPostingAndPost] + 424
38  com.apple.AppKit         	0x937495f4 -[NSTableView mouseDown:] + 2768
39  com.apple.AppKit         	0x93808a30 -[NSOutlineView mouseDown:] + 680
40  com.apple.WebCore        	0x019a07f8 WebCore::MacFrame::passMouseDownEventToWidget(WebCore::Widget*) + 888 (MacFrame.mm:1542)
41  com.apple.WebCore        	0x019a6870 WebCore::MacFrame::khtmlMousePressEvent(WebCore::MousePressEvent*) + 240 (MacFrame.mm:1466)
42  com.apple.WebCore        	0x019b6614 WebCore::FrameView::viewportMousePressEvent(QMouseEvent*) + 452 (RefPtr.h:50)
43  com.apple.WebCore        	0x019aeccc WebCore::MacFrame::mouseDown(NSEvent*) + 620 (MacFrame.mm:2041)
44  com.apple.WebKit         	0x00344600 -[WebHTMLView mouseDown:] + 272 (WebHTMLView.m:2636)
45  com.apple.AppKit         	0x936e8eb0 -[NSWindow sendEvent:] + 4616
46  com.apple.WebKit         	0x00386c48 -[WebInspectorPanel sendEvent:] + 808 (WebInspectorPanel.m:154)
47  com.apple.AppKit         	0x93691ef4 -[NSApplication sendEvent:] + 4172
48  com.apple.Safari         	0x00021c64 0x1000 + 134244
49  com.apple.AppKit         	0x93689330 -[NSApplication run] + 508
50  com.apple.AppKit         	0x93779e68 NSApplicationMain + 452
51  com.apple.Safari         	0x0005cfdc 0x1000 + 376796
52  com.apple.Safari         	0x0005ce80 0x1000 + 376448

Thread 1:
0   libSystem.B.dylib        	0x9000b1e8 mach_msg_trap + 8
1   libSystem.B.dylib        	0x9000b13c mach_msg + 60
2   com.apple.CoreFoundation 	0x9075d114 __CFRunLoopRun + 832
3   com.apple.CoreFoundation 	0x9075ca18 CFRunLoopRunSpecific + 268
4   com.apple.Foundation     	0x928f6384 -[NSRunLoop runMode:beforeDate:] + 172
5   com.apple.Foundation     	0x928f62bc -[NSRunLoop run] + 76
6   com.apple.WebKit         	0x0036ca74 +[WebFileDatabase _syncLoop:] + 180 (WebFileDatabase.m:291)
7   com.apple.Foundation     	0x928e73b4 forkThreadForFunction + 108
8   libSystem.B.dylib        	0x9002b1e0 _pthread_body + 96

Thread 2:
0   libSystem.B.dylib        	0x9000b1e8 mach_msg_trap + 8
1   libSystem.B.dylib        	0x9000b13c mach_msg + 60
2   com.apple.CoreFoundation 	0x9075d114 __CFRunLoopRun + 832
3   com.apple.CoreFoundation 	0x9075ca18 CFRunLoopRunSpecific + 268
4   com.apple.Foundation     	0x9290e8bc +[NSURLConnection(NSURLConnectionInternal) _resourceLoadLoop:] + 264
5   com.apple.Foundation     	0x928e73b4 forkThreadForFunction + 108
6   libSystem.B.dylib        	0x9002b1e0 _pthread_body + 96

Thread 3:
0   libSystem.B.dylib        	0x9000b1e8 mach_msg_trap + 8
1   libSystem.B.dylib        	0x9000b13c mach_msg + 60
2   com.apple.CoreFoundation 	0x9075d114 __CFRunLoopRun + 832
3   com.apple.CoreFoundation 	0x9075ca18 CFRunLoopRunSpecific + 268
4   com.apple.Foundation     	0x9290f9fc +[NSURLCache _diskCacheSyncLoop:] + 152
5   com.apple.Foundation     	0x928e73b4 forkThreadForFunction + 108
6   libSystem.B.dylib        	0x9002b1e0 _pthread_body + 96

Thread 4:
0   libSystem.B.dylib        	0x9001f1ec select + 12
1   com.apple.CoreFoundation 	0x9076f9a8 __CFSocketManager + 472
2   libSystem.B.dylib        	0x9002b1e0 _pthread_body + 96

Thread 5:
0   libSystem.B.dylib        	0x9002b888 semaphore_wait_signal_trap + 8
1   libSystem.B.dylib        	0x9002fffc pthread_cond_wait + 488
2   com.apple.Foundation     	0x928ee520 -[NSConditionLock lockWhenCondition:] + 68
3   com.apple.Syndication    	0x99f3850c -[AsyncDB _run:] + 192
4   com.apple.Foundation     	0x928e73b4 forkThreadForFunction + 108
5   libSystem.B.dylib        	0x9002b1e0 _pthread_body + 96

Thread 0 crashed with PPC Thread State 64:
  srr0: 0x000000000199139c srr1: 0x100000000200f030                        vrsave: 0x0000000000000000
    cr: 0x24022202          xer: 0x0000000000000000   lr: 0x0000000001a0cbe4  ctr: 0x00000000019e7df0
    r0: 0x0000000001a0cbe4   r1: 0x00000000bfffbe30   r2: 0x0000000000000000   r3: 0x0000000000000000
    r4: 0x00000000909ec0a0   r5: 0x0000000001bf779c   r6: 0x0000000000000001   r7: 0x0000000000000001
    r8: 0x000000004000758b   r9: 0x00000000010be1cc  r10: 0x0000000000000041  r11: 0x000000000000007c
   r12: 0x00000000019e7df0  r13: 0x00000000a36b7ca8  r14: 0x0000000000000100  r15: 0x0000000000000000
   r16: 0x00000000a36b7ca8  r17: 0x0000000000000000  r18: 0x0000000022e883a0  r19: 0x00000000220cb060
   r20: 0x00000000bfffd32c  r21: 0x0000000000685d10  r22: 0x0000000000000001  r23: 0x000000000109800c
   r24: 0x00000000248f4670  r25: 0x00000000010c5130  r26: 0x00000000010c5130  r27: 0x00000000bfffcb64
   r28: 0x0000000000000000  r29: 0x00000000006ad580  r30: 0x0000000023345220  r31: 0x0000000001a0cb50

</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>39747</commentid>
    <comment_count>6</comment_count>
      <attachid>7813</attachid>
    <who name="James G. Speth">speth</who>
    <bug_when>2006-04-18 19:04:35 -0700</bug_when>
    <thetext>Created attachment 7813
patch for review

when the window it was inspecting is closed, the inspector goes into its no-selection state.  choosing to inspect another element activates it again.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>39851</commentid>
    <comment_count>7</comment_count>
      <attachid>7813</attachid>
    <who name="Timothy Hatcher">timothy</who>
    <bug_when>2006-04-19 12:00:47 -0700</bug_when>
    <thetext>Comment on attachment 7813
patch for review

Looks good, tested and works great. r=me

One style note for the future.

+        }
+        else

This should be:

    + } else

I will fix this when I land.

Please attach a ChangeLog so I can land this.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>39887</commentid>
    <comment_count>8</comment_count>
    <who name="Timothy Hatcher">timothy</who>
    <bug_when>2006-04-19 18:40:56 -0700</bug_when>
    <thetext>Landed in r13976.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>45311</commentid>
    <comment_count>9</comment_count>
    <who name="David Kilzer (:ddkilzer)">ddkilzer</who>
    <bug_when>2006-06-11 04:57:40 -0700</bug_when>
    <thetext>*** Bug 6922 has been marked as a duplicate of this bug. ***</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>7813</attachid>
            <date>2006-04-18 19:04:35 -0700</date>
            <delta_ts>2006-04-19 12:00:47 -0700</delta_ts>
            <desc>patch for review</desc>
            <filename>inspector-patch</filename>
            <type>text/plain</type>
            <size>5292</size>
            <attacher name="James G. Speth">speth</attacher>
            
              <data encoding="base64">SW5kZXg6IFdlYkluc3BlY3Rvci5tCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFdlYkluc3BlY3Rvci5tCShyZXZp
c2lvbiAxMzg4MCkKKysrIFdlYkluc3BlY3Rvci5tCSh3b3JraW5nIGNvcHkpCkBAIC0xNjQsMTAg
KzE2NCwyMCBAQCBzdGF0aWMgTlNNYXBUYWJsZSAqbGFzdENoaWxkSWdub3JpbmdXaGl0CiAgICAg
aWYgKFt3ZWJGcmFtZSBpc0VxdWFsOl9wcml2YXRlLT53ZWJGcmFtZV0pCiAgICAgICAgIHJldHVy
bjsKIAorICAgIGlmIChfcHJpdmF0ZS0+d2ViRnJhbWUpIHsKKyAgICAgICAgW1tOU05vdGlmaWNh
dGlvbkNlbnRlciBkZWZhdWx0Q2VudGVyXSByZW1vdmVPYnNlcnZlcjpzZWxmIG5hbWU6V2ViVmll
d1Byb2dyZXNzRmluaXNoZWROb3RpZmljYXRpb24gb2JqZWN0OltfcHJpdmF0ZS0+d2ViRnJhbWUg
d2ViVmlld11dOworICAgICAgICBbW05TTm90aWZpY2F0aW9uQ2VudGVyIGRlZmF1bHRDZW50ZXJd
IHJlbW92ZU9ic2VydmVyOnNlbGYgbmFtZTpOU1dpbmRvd1dpbGxDbG9zZU5vdGlmaWNhdGlvbiBv
YmplY3Q6W1tfcHJpdmF0ZS0+d2ViRnJhbWUgd2ViVmlld10gd2luZG93XV07CisgICAgfQorICAg
IAogICAgIFt3ZWJGcmFtZSByZXRhaW5dOwogICAgIFtfcHJpdmF0ZS0+d2ViRnJhbWUgcmVsZWFz
ZV07CiAgICAgX3ByaXZhdGUtPndlYkZyYW1lID0gd2ViRnJhbWU7CiAKKyAgICBpZiAoX3ByaXZh
dGUtPndlYkZyYW1lKSB7CisgICAgICAgIFtbTlNOb3RpZmljYXRpb25DZW50ZXIgZGVmYXVsdENl
bnRlcl0gYWRkT2JzZXJ2ZXI6c2VsZiBzZWxlY3RvcjpAc2VsZWN0b3IoaW5zcGVjdGVkV2ViVmll
d1Byb2dyZXNzRmluaXNoZWQ6KSBuYW1lOldlYlZpZXdQcm9ncmVzc0ZpbmlzaGVkTm90aWZpY2F0
aW9uIG9iamVjdDpbX3ByaXZhdGUtPndlYkZyYW1lIHdlYlZpZXddXTsKKyAgICAgICAgW1tOU05v
dGlmaWNhdGlvbkNlbnRlciBkZWZhdWx0Q2VudGVyXSBhZGRPYnNlcnZlcjpzZWxmIHNlbGVjdG9y
OkBzZWxlY3RvcihpbnNwZWN0ZWRXaW5kb3dXaWxsQ2xvc2U6KSBuYW1lOk5TV2luZG93V2lsbENs
b3NlTm90aWZpY2F0aW9uIG9iamVjdDpbW19wcml2YXRlLT53ZWJGcmFtZSB3ZWJWaWV3XSB3aW5k
b3ddXTsKKyAgICB9CisgICAgW19wcml2YXRlLT50cmVlT3V0bGluZVZpZXcgc2V0QWxsb3dzRW1w
dHlTZWxlY3Rpb246Tk9dOwogICAgIFtzZWxmIHNldEZvY3VzZWRET01Ob2RlOlt3ZWJGcmFtZSBE
T01Eb2N1bWVudF1dOwogfQogCkBAIC0xODAsNyArMTkwLDcgQEAgc3RhdGljIE5TTWFwVGFibGUg
Kmxhc3RDaGlsZElnbm9yaW5nV2hpdAogCiAtICh2b2lkKXNldFJvb3RET01Ob2RlOihET01Ob2Rl
ICopbm9kZQogewotICAgIGlmICghW25vZGUgaGFzQ2hpbGROb2Rlc10gfHwgW25vZGUgaXNTYW1l
Tm9kZTpfcHJpdmF0ZS0+cm9vdE5vZGVdKQorICAgIGlmIChbbm9kZSBpc1NhbWVOb2RlOl9wcml2
YXRlLT5yb290Tm9kZV0pCiAgICAgICAgIHJldHVybjsKIAogICAgIFtub2RlIHJldGFpbl07CkBA
IC01NDgsNyArNTU4LDcgQEAgc3RhdGljIE5TTWFwVGFibGUgKmxhc3RDaGlsZElnbm9yaW5nV2hp
dAogCiAtICh2b2lkKV9yZXZlYWxBbmRTZWxlY3ROb2RlSW5UcmVlOihET01Ob2RlICopbm9kZQog
ewotICAgIGlmICghX3ByaXZhdGUtPndlYlZpZXdMb2FkZWQgfHwgIW5vZGUpCisgICAgaWYgKCFf
cHJpdmF0ZS0+d2ViVmlld0xvYWRlZCkKICAgICAgICAgcmV0dXJuOwogCiAgICAgaWYgKCFfcHJp
dmF0ZS0+cHJldmVudFJldmVhbE9uRm9jdXMpIHsKQEAgLTU3Miw4ICs1ODIsMTAgQEAgc3RhdGlj
IE5TTWFwVGFibGUgKmxhc3RDaGlsZElnbm9yaW5nV2hpdAogICAgIGlmIChpbmRleCAhPSAtMSkg
ewogICAgICAgICBbX3ByaXZhdGUtPnRyZWVPdXRsaW5lVmlldyBzZWxlY3RSb3dJbmRleGVzOltO
U0luZGV4U2V0IGluZGV4U2V0V2l0aEluZGV4OmluZGV4XSBieUV4dGVuZGluZ1NlbGVjdGlvbjpO
T107CiAgICAgICAgIFtfcHJpdmF0ZS0+dHJlZU91dGxpbmVWaWV3IHNjcm9sbFJvd1RvVmlzaWJs
ZTppbmRleF07Ci0gICAgICAgIFtzZWxmIF91cGRhdGVUcmVlU2Nyb2xsYmFyXTsKKyAgICB9IGVs
c2UgeworICAgICAgICBbX3ByaXZhdGUtPnRyZWVPdXRsaW5lVmlldyBkZXNlbGVjdEFsbDpzZWxm
XTsKICAgICB9CisgICAgW3NlbGYgX3VwZGF0ZVRyZWVTY3JvbGxiYXJdOwogfQogCiAtICh2b2lk
KV9yZWZyZXNoU2VhcmNoCkBAIC02NDEsOCArNjUzLDE0IEBAIHN0YXRpYyBOU01hcFRhYmxlICps
YXN0Q2hpbGRJZ25vcmluZ1doaXQKIAogLSAodm9pZClfdXBkYXRlCiB7Ci0gICAgaWYgKF9wcml2
YXRlLT53ZWJWaWV3TG9hZGVkICYmIFtzZWxmIGZvY3VzZWRET01Ob2RlXSkKLSAgICAgICAgW1tf
cHJpdmF0ZS0+d2ViVmlldyB3aW5kb3dTY3JpcHRPYmplY3RdIGNhbGxXZWJTY3JpcHRNZXRob2Q6
QCJ1cGRhdGVQYW5lcyIgd2l0aEFyZ3VtZW50czpuaWxdOworICAgIGlmIChfcHJpdmF0ZS0+d2Vi
Vmlld0xvYWRlZCkgeworICAgICAgICBpZiAoW3NlbGYgZm9jdXNlZERPTU5vZGVdKSB7CisgICAg
ICAgICAgICBbW19wcml2YXRlLT53ZWJWaWV3IHdpbmRvd1NjcmlwdE9iamVjdF0gY2FsbFdlYlNj
cmlwdE1ldGhvZDpAInRvZ2dsZU5vU2VsZWN0aW9uIiB3aXRoQXJndW1lbnRzOltOU0FycmF5IGFy
cmF5V2l0aE9iamVjdDpbTlNOdW1iZXIgbnVtYmVyV2l0aEJvb2w6Tk9dXV07CisgICAgICAgICAg
ICBbW19wcml2YXRlLT53ZWJWaWV3IHdpbmRvd1NjcmlwdE9iamVjdF0gY2FsbFdlYlNjcmlwdE1l
dGhvZDpAInVwZGF0ZVBhbmVzIiB3aXRoQXJndW1lbnRzOm5pbF07CisgICAgICAgIH0KKyAgICAg
ICAgZWxzZQorICAgICAgICAgICAgW1tfcHJpdmF0ZS0+d2ViVmlldyB3aW5kb3dTY3JpcHRPYmpl
Y3RdIGNhbGxXZWJTY3JpcHRNZXRob2Q6QCJ0b2dnbGVOb1NlbGVjdGlvbiIgd2l0aEFyZ3VtZW50
czpbTlNBcnJheSBhcnJheVdpdGhPYmplY3Q6W05TTnVtYmVyIG51bWJlcldpdGhCb29sOllFU11d
XTsKKyAgICB9CiB9CiAKIC0gKHZvaWQpX3VwZGF0ZVRyYXZlcnNhbEJ1dHRvbnMKQEAgLTY3Myw3
ICs2OTEsNyBAQCBzdGF0aWMgTlNNYXBUYWJsZSAqbGFzdENoaWxkSWdub3JpbmdXaGl0CiAKIC0g
KHZvaWQpX3VwZGF0ZVJvb3QKIHsKLSAgICBpZiAoIV9wcml2YXRlLT53ZWJWaWV3TG9hZGVkIHx8
ICFbc2VsZiByb290RE9NTm9kZV0gfHwgX3ByaXZhdGUtPnNlYXJjaFJlc3VsdHNWaXNpYmxlKQor
ICAgIGlmICghX3ByaXZhdGUtPndlYlZpZXdMb2FkZWQgfHwgX3ByaXZhdGUtPnNlYXJjaFJlc3Vs
dHNWaXNpYmxlKQogICAgICAgICByZXR1cm47CiAKICAgICBET01IVE1MRWxlbWVudCAqdGl0bGVB
cmVhID0gKERPTUhUTUxFbGVtZW50ICopW19wcml2YXRlLT5kb21Eb2N1bWVudCBnZXRFbGVtZW50
QnlJZDpAInRyZWVQb3B1cFRpdGxlQXJlYSJdOwpAQCAtNjkzLDcgKzcxMSw4IEBAIHN0YXRpYyBO
U01hcFRhYmxlICpsYXN0Q2hpbGRJZ25vcmluZ1doaXQKICAgICBfcHJpdmF0ZS0+cHJldmVudFNl
bGVjdGlvblJlZm9jdXMgPSBZRVM7CiAgICAgRE9NTm9kZSAqZm9jdXNlZE5vZGUgPSBbW3NlbGYg
Zm9jdXNlZERPTU5vZGVdIHJldGFpbl07CiAgICAgW19wcml2YXRlLT50cmVlT3V0bGluZVZpZXcg
cmVsb2FkRGF0YV07Ci0gICAgW19wcml2YXRlLT50cmVlT3V0bGluZVZpZXcgZXhwYW5kSXRlbTpb
c2VsZiByb290RE9NTm9kZV1dOworICAgIGlmIChbc2VsZiByb290RE9NTm9kZV0pCisgICAgICAg
IFtfcHJpdmF0ZS0+dHJlZU91dGxpbmVWaWV3IGV4cGFuZEl0ZW06W3NlbGYgcm9vdERPTU5vZGVd
XTsKICAgICBbc2VsZiBfcmV2ZWFsQW5kU2VsZWN0Tm9kZUluVHJlZTpmb2N1c2VkTm9kZV07CiAg
ICAgW2ZvY3VzZWROb2RlIHJlbGVhc2VdOwogICAgIF9wcml2YXRlLT5wcmV2ZW50U2VsZWN0aW9u
UmVmb2N1cyA9IE5POwpAQCAtNzQ4LDggKzc2NywzMSBAQCBzdGF0aWMgTlNNYXBUYWJsZSAqbGFz
dENoaWxkSWdub3JpbmdXaGl0CiAKICNwcmFnbWEgbWFyayAtCiAKKy0gKHZvaWQpaW5zcGVjdGVk
V2ViVmlld1Byb2dyZXNzRmluaXNoZWQ6KE5TTm90aWZpY2F0aW9uICopbm90aWZpY2F0aW9uCit7
CisgICAgaWYgKFtub3RpZmljYXRpb24gb2JqZWN0XSA9PSBbW3NlbGYgd2ViRnJhbWVdIHdlYlZp
ZXddKSB7CisgICAgICAgIFtzZWxmIHNldEZvY3VzZWRET01Ob2RlOltbc2VsZiB3ZWJGcmFtZV0g
RE9NRG9jdW1lbnRdXTsKKyAgICAgICAgW3NlbGYgX3VwZGF0ZV07CisgICAgICAgIFtzZWxmIF91
cGRhdGVSb290XTsKKyAgICAgICAgW3NlbGYgX2hpZ2hsaWdodE5vZGU6W3NlbGYgZm9jdXNlZERP
TU5vZGVdXTsKKyAgICB9Cit9CisKKy0gKHZvaWQpaW5zcGVjdGVkV2luZG93V2lsbENsb3NlOihO
U05vdGlmaWNhdGlvbiAqKW5vdGlmaWNhdGlvbgoreworICAgIFtzZWxmIHNldEZvY3VzZWRET01O
b2RlOm5pbF07CisgICAgW3NlbGYgc2V0V2ViRnJhbWU6bmlsXTsKKyAgICBbX3ByaXZhdGUtPnRy
ZWVPdXRsaW5lVmlldyBzZXRBbGxvd3NFbXB0eVNlbGVjdGlvbjpZRVNdOworICAgIFtfcHJpdmF0
ZS0+dHJlZU91dGxpbmVWaWV3IGRlc2VsZWN0QWxsOnNlbGZdOworICAgIFtzZWxmIF91cGRhdGVd
OworICAgIFtzZWxmIF91cGRhdGVSb290XTsKK30KKworI3ByYWdtYSBtYXJrIC0KKwogLSAodm9p
ZCl3ZWJWaWV3OihXZWJWaWV3ICopc2VuZGVyIGRpZEZpbmlzaExvYWRGb3JGcmFtZTooV2ViRnJh
bWUgKilmcmFtZQogeworICAgIC8vIG5vdGU6IHRoaXMgaXMgdGhlIEluc3BlY3RvcidzIG93biBX
ZWJWaWV3LCBub3QgdGhlIG9uZSBiZWluZyBpbnNwZWN0ZWQKICAgICBfcHJpdmF0ZS0+d2ViVmll
d0xvYWRlZCA9IFlFUzsKICAgICBbW3NlbmRlciB3aW5kb3dTY3JpcHRPYmplY3RdIHNldFZhbHVl
OnNlbGYgZm9yS2V5OkAiSW5zcGVjdG9yIl07CiAKSW5kZXg6IFdlYkluc3BlY3RvclBhbmVsLm0K
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PQotLS0gV2ViSW5zcGVjdG9yUGFuZWwubQkocmV2aXNpb24gMTM4ODApCisrKyBX
ZWJJbnNwZWN0b3JQYW5lbC5tCSh3b3JraW5nIGNvcHkpCkBAIC00Myw3ICs0Myw4IEBACiAKIC0g
KEJPT0wpY2FuQmVjb21lTWFpbldpbmRvdwogewotICAgIHJldHVybiBZRVM7CisgICAgLy8gcHJl
dmVudCB0aGUgaW5zcGVjdG9yIGZyb20gY29uZnVzaW5nIG90aGVyIHBhbmVscworICAgIHJldHVy
biBOTzsKIH0KIAogLSAodm9pZCltb3ZlV2luZG93OihOU0V2ZW50ICopZXZlbnQK
</data>
<flag name="review"
          id="2144"
          type_id="1"
          status="+"
          setter="timothy"
    />
          </attachment>
      

    </bug>

</bugzilla>