Comment #1 on issue 7369 by [email protected]: browsing_instance.cc needs a unit test http://code.google.com/p/chromium/issues/detail?id=7369
BrowsingInstance is pretty thoroughly unit-tested in site_instance_unittest.cc, because the two classes are tightly coupled. (The class comment in browsing_instance.h explains that BrowsingInstances are not visible outside the SiteInstance class.) Not sure it makes sense to give BrowsingInstance its own unit test. At best, perhaps a comment in browsing_instance.h pointing developers to the site_instance_unittest.cc file? What do you think? -- You received this message because you are listed in the owner or CC fields of this issue, or because you starred this issue. You may adjust your issue notification preferences at: http://code.google.com/hosting/settings --~--~---------~--~----~------------~-------~--~----~ Automated mail from issue updates at http://crbug.com/ Subscription options: http://groups.google.com/group/chromium-bugs -~----------~----~----~----~------~----~------~--~---
