Review: Approve As mentioned on IRC, consider making the test more specific to ensure the phrase is actually in the banner and not somewhere else in browser.contents. Otherwise, this looks great. I particularly like the IInformationType approach to this. -- https://code.launchpad.net/~abentley/launchpad/fix-banner/+merge/124052 Your team Launchpad code reviewers is subscribed to branch lp:launchpad.
_______________________________________________ Mailing list: https://launchpad.net/~launchpad-reviewers Post to : [email protected] Unsubscribe : https://launchpad.net/~launchpad-reviewers More help : https://help.launchpad.net/ListHelp
