[framework-issues] [Issue 111452] framework: assert in Tran sactionManager when closing window with quicks tart enabled

2010-08-03 Thread dtardon
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=111452


User dtardon changed the following:

What|Old value |New value

  Status|VERIFIED  |CLOSED





--- Additional comments from dtar...@openoffice.org Tue Aug  3 06:58:13 
+ 2010 ---
integrated in DEV300_m84

-
Please do not reply to this automatically generated notification from
Issue Tracker. Please log onto the website and enter your comments.
http://qa.openoffice.org/issue_handling/project_issues.html#notification

-
To unsubscribe, e-mail: issues-unsubscr...@framework.openoffice.org
For additional commands, e-mail: issues-h...@framework.openoffice.org


-
To unsubscribe, e-mail: allbugs-unsubscr...@openoffice.org
For additional commands, e-mail: allbugs-h...@openoffice.org



[framework-issues] [Issue 111452] framework: assert in Tran sactionManager when closing window with quicks tart enabled

2010-06-27 Thread 400guy
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=111452





--- Additional comments from 400...@openoffice.org Sun Jun 27 19:20:48 
+ 2010 ---
*** Issue 110623 has been marked as a duplicate of this issue. ***

-
Please do not reply to this automatically generated notification from
Issue Tracker. Please log onto the website and enter your comments.
http://qa.openoffice.org/issue_handling/project_issues.html#notification

-
To unsubscribe, e-mail: issues-unsubscr...@framework.openoffice.org
For additional commands, e-mail: issues-h...@framework.openoffice.org


-
To unsubscribe, e-mail: allbugs-unsubscr...@openoffice.org
For additional commands, e-mail: allbugs-h...@openoffice.org



[framework-issues] [Issue 111452] framework: assert in Tran sactionManager when closing window with quicks tart enabled

2010-06-04 Thread mba
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=111452


User mba changed the following:

What|Old value |New value

 Assigned to|mba   |cd





--- Additional comments from m...@openoffice.org Fri Jun  4 11:27:08 + 
2010 ---
please verify

-
Please do not reply to this automatically generated notification from
Issue Tracker. Please log onto the website and enter your comments.
http://qa.openoffice.org/issue_handling/project_issues.html#notification

-
To unsubscribe, e-mail: issues-unsubscr...@framework.openoffice.org
For additional commands, e-mail: issues-h...@framework.openoffice.org


-
To unsubscribe, e-mail: allbugs-unsubscr...@openoffice.org
For additional commands, e-mail: allbugs-h...@openoffice.org



[framework-issues] [Issue 111452] framework: assert in Tran sactionManager when closing window with quicks tart enabled

2010-06-04 Thread cd
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=111452


User cd changed the following:

What|Old value |New value

  Status|RESOLVED  |VERIFIED





--- Additional comments from c...@openoffice.org Fri Jun  4 12:20:39 + 
2010 ---
cd: Verified with non-pro build.

-
Please do not reply to this automatically generated notification from
Issue Tracker. Please log onto the website and enter your comments.
http://qa.openoffice.org/issue_handling/project_issues.html#notification

-
To unsubscribe, e-mail: issues-unsubscr...@framework.openoffice.org
For additional commands, e-mail: issues-h...@framework.openoffice.org


-
To unsubscribe, e-mail: allbugs-unsubscr...@openoffice.org
For additional commands, e-mail: allbugs-h...@openoffice.org



[framework-issues] [Issue 111452] framework: assert in Tran sactionManager when closing window with quicks tart enabled

2010-05-18 Thread mba
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=111452


User mba changed the following:

What|Old value |New value

Target milestone|OOo 3.x   |OOo 3.3





--- Additional comments from m...@openoffice.org Tue May 18 15:55:08 + 
2010 ---
target 3.3

-
Please do not reply to this automatically generated notification from
Issue Tracker. Please log onto the website and enter your comments.
http://qa.openoffice.org/issue_handling/project_issues.html#notification

-
To unsubscribe, e-mail: issues-unsubscr...@framework.openoffice.org
For additional commands, e-mail: issues-h...@framework.openoffice.org


-
To unsubscribe, e-mail: allbugs-unsubscr...@openoffice.org
For additional commands, e-mail: allbugs-h...@openoffice.org



[framework-issues] [Issue 111452] framework: assert in Tran sactionManager when closing window with quicks tart enabled

2010-05-14 Thread 400guy
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=111452


User 400guy changed the following:

What|Old value |New value

  CC|''|'400guy'





-
Please do not reply to this automatically generated notification from
Issue Tracker. Please log onto the website and enter your comments.
http://qa.openoffice.org/issue_handling/project_issues.html#notification

-
To unsubscribe, e-mail: issues-unsubscr...@framework.openoffice.org
For additional commands, e-mail: issues-h...@framework.openoffice.org


-
To unsubscribe, e-mail: allbugs-unsubscr...@openoffice.org
For additional commands, e-mail: allbugs-h...@openoffice.org



[framework-issues] [Issue 111452] framework: assert in Tran sactionManager when closing window with quicks tart enabled

2010-05-11 Thread mba
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=111452


User mba changed the following:

What|Old value |New value

  Status|NEW   |RESOLVED

  Resolution|  |FIXED

Target milestone|---   |OOo 3.x





--- Additional comments from m...@openoffice.org Tue May 11 12:47:27 + 
2010 ---
Thanks for the patch, it helped me to find the culprit. OTOH I solved the
problem differently. It took some time to analyze this problem.

The reason for the assertion is that getController() is called at a disposed
frame and its TransactionGuard complains about that by showing the assertion
message. The offending call happens in the AsyncCallback of the CloseDispatcher
(the code that you have patched).

The idea of this code is that once a Controller has been suspended in
preparation for closing the frame, this suspension must be removed when the
closing of the frame failed. Otherwise the UI inside the frame would be
dysfunctional and perhaps a crash is not far away.

The code makes a mistake by assuming that bSuccess being sal_False is enough
to detect the need to unsuspend, but unfortunately the variable is can have
this value also if the frame was closed successfully, but the Desktop could not
be terminated. This is the case e.g. if a Quickstarter is active.

Your patch just checks if the frame was actually destroyed, but it will fail in
case any other code still has a hard reference to the frame.

My first attempt was to have two variables bSuccess (for the overall access of
the whole method) and bCloseSuccess that will be true only if the frame was
closed successfully by the CloseDispatcher. Unfortunately after some thinking I
discovered scenarios where the Controller was suspended, but the frame was
closed or tried to be closed elsewhere, so bCloseSuccess is not able to
provide the needed information.

What we want to know is: was the frame disposed so that calling getController()
is not necessary? A possible solution would be to register an EventListener at
the frame that notifies the CloseDispatcher about the disposing event. But IMHO
that would be overkill. I think it is OK to remove the TransactionGuard from the
Frame::getController() method. This makes clear that calling getController() at
a disposed frame is not a mistake. I can't think of a situation where this would
cause trouble: getController() just returns a copy of a member variable that is
not guaranteed to be not empty.

-
Please do not reply to this automatically generated notification from
Issue Tracker. Please log onto the website and enter your comments.
http://qa.openoffice.org/issue_handling/project_issues.html#notification

-
To unsubscribe, e-mail: issues-unsubscr...@framework.openoffice.org
For additional commands, e-mail: issues-h...@framework.openoffice.org


-
To unsubscribe, e-mail: allbugs-unsubscr...@openoffice.org
For additional commands, e-mail: allbugs-h...@openoffice.org



[framework-issues] [Issue 111452] framework: assert in Tran sactionManager when closing window with quicks tart enabled

2010-05-07 Thread dtardon
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=111452
 Issue #|111452
 Summary|framework: assert in TransactionManager when closing w
|indow with quickstart enabled
   Component|framework
 Version|DEV300m77
Platform|All
 URL|
  OS/Version|Linux
  Status|NEW
   Status whiteboard|
Keywords|
  Resolution|
  Issue type|PATCH
Priority|P3
Subcomponent|code
 Assigned to|mba
 Reported by|dtardon





--- Additional comments from dtar...@openoffice.org Fri May  7 11:19:44 
+ 2010 ---
More specifically, I'm getting the following two asserts:

Error: ASSERT:
TransactionManager...
Owner instance already closed. Call was rejected!
 From File
/home/dtardon/work/upstream/openoffice.org/build/devel/framework/source/threadhelp/transactionmanager.cxx
at Line 396
Error: ASSERT:
CloseDispatcher::impl_asyncCallback
Congratulation! You found the reason for bug #120310#. Please contact 
the
right developer and show him a scenario, which trigger this bug. THX.
 From File
/home/dtardon/work/upstream/openoffice.org/build/devel/framework/source/dispatch/closedispatcher.cxx
at Line 454


How to reproduce:

1. start oo.o and ensure that you don't have opened anything but start center
2. enable quickstarter
3. open one writer window
4. close the window

-
Please do not reply to this automatically generated notification from
Issue Tracker. Please log onto the website and enter your comments.
http://qa.openoffice.org/issue_handling/project_issues.html#notification

-
To unsubscribe, e-mail: issues-unsubscr...@framework.openoffice.org
For additional commands, e-mail: issues-h...@framework.openoffice.org


-
To unsubscribe, e-mail: allbugs-unsubscr...@openoffice.org
For additional commands, e-mail: allbugs-h...@openoffice.org



[framework-issues] [Issue 111452] framework: assert in Tran sactionManager when closing window with quicks tart enabled

2010-05-07 Thread dtardon
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=111452


User dtardon changed the following:

What|Old value |New value

 Attachment is patch|  |Created an attachment (id=
|  |69358)
possible fix






--- Additional comments from dtar...@openoffice.org Fri May  7 11:20:48 
+ 2010 ---
Created an attachment (id=69358)
possible fix


-
Please do not reply to this automatically generated notification from
Issue Tracker. Please log onto the website and enter your comments.
http://qa.openoffice.org/issue_handling/project_issues.html#notification

-
To unsubscribe, e-mail: issues-unsubscr...@framework.openoffice.org
For additional commands, e-mail: issues-h...@framework.openoffice.org


-
To unsubscribe, e-mail: allbugs-unsubscr...@openoffice.org
For additional commands, e-mail: allbugs-h...@openoffice.org