I have taken a look at how much effort this would be. This class and most of the code in this directory are currently not directly tested via unit tests. Since the method in question is not exposed in the public API, testing this code path explicitly would involve mocking out how data requests are handled via PP::URLLoader, which is a relatively large amount of engineering work.

The specific change in this CL was removing an instance of NOTREACHED, which should have been removed when handling for the case was added. I don't reasonably expect there to be a regression of this behaviour.

At this time I don't think it is worth the effort to put in a test for just this issue due to the complexity involve and low value.