Closed GoogleCodeExporter closed 9 years ago
Pretty much complete at r70. Take a look.
Original comment by aale...@gmail.com
on 17 Nov 2010 at 8:14
See this thread:
http://groups.google.com/group/genxdm/browse_thread/thread/39c70b5ca8650673
Summarizing: the initial implementation introduced a new parameter, F, to
handle DOM stupidity. It almost certainly shouldn't have, and Eric pushed
back. The discussion suggested that DOM handle its own conflation of
"document" and "factory" internally, and that all APIs have "getNodeFactory" on
the context and "getFactoryForContext" on model and cursor.
So I'm marking this "open" again, although I'm going to mark it "fixed" again
in a minute. :-)
Original comment by aale...@gmail.com
on 29 Nov 2010 at 4:46
Changes committed at r71, on xquf branch. Marking "fixed" again, and we'll see
if this one makes it. :-)
Summary of fixes:
1) MutableContext lost the "F" parameter defined for the method call
getNodeFactory, and the argument to the method.
2) NodeFactory lost its "F" parameter, and two methods associated with
"getting" and "setting" the API factory.
3) MutableModel and MutableCursor added a method, getFactoryForContext (with N
argument for model, of course).
4) the bridgekit (MutableCursorOnMutableModel) was modified (hmmm, could
actually cache the factory there) to add the new method (delegating to model).
5) the DOM bridge was modified to implement the new methods and keep track of
which Document was associated with this NodeFactory.
Sending mail to the thread mentioned above in a moment, too.
Original comment by aale...@gmail.com
on 29 Nov 2010 at 4:51
Original comment by eric%tib...@gtempaccount.com
on 29 Apr 2011 at 11:15
Original issue reported on code.google.com by
aale...@gmail.com
on 17 Nov 2010 at 3:34