Comment on attachment 8381157 protocol.patch Looks good, thanks. Can you adjust the spacing a bit to match the surrounding code, as noted below, please?
>+ >+ Only one blank line, please. >+ if ( mClass == eProtocolInfo ) { No space after '(' or before ')'. Only one space after "==". >+ *_retval = nsGNOMERegistry::HandlerExists (mSchemeOrType.get()); No space before '('. -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1197559 Title: Regression: Firefox forgot how to open apt urls To manage notifications about this bug go to: https://bugs.launchpad.net/firefox/+bug/1197559/+subscriptions -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs