open minibrowser without url encode

[datou0412]

Signed-off-by: 二凢 dingyu.wdy@alibaba-inc.com

What it does

open minibrowser without url encode

How to test

inject a MiniBrowserOpenHandler as miniBrowserOpenHandler

this.miniBrowserOpenHandler.open(new URI('https://your.site.com?q=test'), options);

expect open a minibrowser with url "https://your.site.com?q=test" but got "https://your.site.com?q%3Dtest"

Review checklist

• as an author, I have thoroughly tested my changes and carefully followed the review guidelines

Reminder for reviewers

• as a reviewer, I agree to behave in accordance with the review guidelines

[kittaakos]

Very nice catch, @datou0412. It happens only and if only you explicitly call the open method with a URI and without the params. Otherwise, it works. That's why we did not hit this issue before. Thank you for the fix!

[kittaakos]

I have verified it locally, as I had to modify the code a little bit to be able to reproduce the defect. It worked as expected.

Thank you for your contribution, @datou0412!