NSBundle bundleWithIdentifier causes slowdown when creating FlutterEngine

[xster]

The addition of traversing the bundles by searching for an identifier added in https://github.com/flutter/engine/pull/5986/files#diff-8e54aca7e9d4fd69a57fb86d6ec3dee2R36 adds a 100ms latency when instantiating a FlutterEngine (7ms -> 106ms on an iPod).

It's hard to figure out the original feature request from the PR. Can we revise the approach?

[jamesderlin]

Internally we wanted to support having multiple bundles and not rely on the main bundle when looking up Flutter assets. (I'm still trying to refresh my memory about exactly why... I think we considered it a burden for add2app?)

The +[NSBundle bundleForClass]: call to initialize engineBundle could be moved to where it's used. I'm not sure what to do about the +[NSBundle bundleWithIdentifier:] call. Possibly we could replace it with +[NSBundle bundleWithPath:] (although since the point of finding the bundle is to extract paths from it, probably not). Maybe another possibility would be to change the ordering so that the expensive lookup is done only after checking cheaper sources.

Anyway, I'm happy to ponder this and assist however I can, but I currently am no longer set up to make and test changes, so this probably would be better owned by someone else.

[xster]

For context, the original feature request was b/112004455

[gaaclarke]

FWIW [FlutterDartProject init] takes 4ms on iPhone SE. Debug build at dd77b73

Edit: This was recorded on a full Flutter app, not Add-to-app.

[xster]

Could have been fixed already. Feel free to close this.

Previous observations with https://github.com/flutter/engine/pull/5986/files

and without

[gaaclarke]

I measured with the same previous settings, but an Add-to-app scenario and didn't notice any problem either.

The slowdown is plausible. I just don't know what conditions lead to [NSBundle bundleWithIdentifier:] being so slow. Any ideas?

[chinmaygarde]

The [discussion on [NSBundle bundleWithIdentifier:]](https://developer.apple.com/documentation/foundation/nsbundle/1411929-bundlewithidentifier?language=objc) has a section on performance implications of this call and suggests alternatives. Specifically:

As an optimization, you can use the bundleWithPath: or bundleWithURL: method instead to avoid file system traversal.

[cailianqing]

hi，the problem caused watchdog crash.Is there a repair plan？