The weekly report for week 30.
General news: In the time originally allocated for Cairo/Quartz work, I’ll be doing Cocoa widget work instead, due to the current status of the code in these areas being more amenable to this. The first item is rewriting the clipboard in Cocoa. (I’ve also been asked to review the Cocoa widget code for performance issues, but I am not a Cocoa performance expert, so it remains to be seen how that goes.)
Fixed the innerHTML issues that were crashing
Planet Intertwingly.
Pulled a fresh tree and started to move the changes over.
It turned out that Sicking had killed
nsITextContent while I was taking a vacation.
Undid the DummyParserRequest refactoring. (Took
more effort than I hoped.)
Browsed through the front pages of X-Philes.
Cleaned up the patch.
Uploaded the patch to the bug.
Requested r=.
Built with ac_add_options
--enable-default-toolkit=cocoa.
Change of plans: Will work on Cocoa widget integration instead of Cairo/Quartz integration.
Started reading the clipboard code and Cocoa NSPasteboard
documentation.
Debugged the font zoom.
Started reading the Cocoa widget code noting down anything that I think may be a perf problem. (I was asked to review the code for perf problems.)
The line numbers between XCode and gdb didn’t match, which
made things difficult. Spent quite some time tackling this issue.
Making it work seemed futile. printf time!
It appears that nsTArray doesn’t auto-stretch
the way I expected.
Still more debugging needed.
Read Cocoa widget code.
Got the font zoom working.
Moved the zoom code to nsDocumentViewer.cpp.
Ported the rest of the code from JavaScript to C++ to make the full feature available for apps that embed Gecko.
Uploaded a new patch to the bug.
Read more Cocoa widget code.
Read more Cocoa widget code.
Compiled the Carbon clipboard code as Objective-C++ to make sure I had edited the makefile the right way.
Started a fresh Cocoa implementation of nsClipboard.
Wrestled with finding a syntax error that caused very unhelpful compiler errors in a different place.
Filed bug 346345.