This changes a bunch of ivars to __weak's, to avoid reference loops. Furthermore, in PBGitHistoryController, we need to call [webView close], otherwise some memory will never be freed?