And then there are those non-crypto we might turn out to be investors or holders as well, so in the long term if might have swayed a few but I don't think it is the needed catalyst for a bull run though.
The keyword here is 'not immediate', so we really don't know what the effect in the market right now. But since they're going to release in the bearish market, we might see some little push though but not going to be the triggering factor for a bull-run.
Samsung has like 30% of the smart phones market globally though, but I don't think that people will just buy this phone because of its embedded crypto wallets, so it's a long way to go before we can really assess if this move from Samsung will do more good in crypto sphere.
Most buyers of that expensive would probably not mind the feature anyways. And though it's good to see these kinds of features for crypto being integrated by large companies like samsung, you're right that it'll likely have at best little push. I doubt it can start what everyone's hoping for.
I think many people do not know about this S10 feature. Samsung has a big market already and their new model will be sold massively as per their previous S series model. Many people will buy this phone because they love to buy latest model of their brand and they will never use that crypto feature being introduced in this phone.