TranslateManagerRenderViewHostTest.AlwaysTranslateLanguagePref and TranslateManagerRenderViewHostTest.BeforeTranslateExtraButtons do Bad Things |
||||
Issue descriptionThese two tests create duplicate TestProfiles for a single BrowserContext. Normally, the Profile *is* the BrowserContext, so this creates a weird situation. Generally, a BrowserContextKeyedService is used for only one Profile, which has a single StoragePartition. However, in these two tests, BrowserContextKeyedServices are shared by both Profiles (Since they have the same BrowserContext), but each profile also has its own StoragePartition. I'm working on separating out the network code (ProfileIOData) from being attached to the Profile to being attached to the StoragePartition, but I'm also using a BrowserContextKeyedService to hold onto objects Chrome-side. One service with two Profiles doesn't work very well. I've been trying to switch the tests over to use TestingProfile::BuildIncognito() - that does fixes the crashes the tests run into with my CL, but it makes the tests fail. :(
,
Aug 11 2017
Worth noting I've found a workaround (Going through the Profile on the way to the BrowserContextKeyedService, so the TestingProfile can bypass the network service). It would be nice if we don't have to rely on that long term. Either way, I still think this is something other BrowserContextKeyedServices may run into in tests.
,
Aug 22 2017
,
Mar 7 2018
,
Mar 7 2018
|
||||
►
Sign in to add a comment |
||||
Comment 1 by napper@chromium.org
, Aug 11 2017Status: assigned (was: Untriaged)