Filling as a good first bug to learn workflows.

do not use 'else' after 'return':
https://searchfox.org/mozilla-central/source/toolkit/components/places/nsNavHistoryResult.cpp#3418-3437

As the change is trivial, it is just to learn how to contribute to Firefox.

Found by http://clang.llvm.org/extra/clang-tidy/checks/readability-else-after-return.html

Tutorial to contribute:
https://firefox-source-docs.mozilla.org/contributing/contribution_quickref.html

Please don't ask for the bug to be assigned. It will be automatically assigned to the first patch.

Removed the 'else' block after the return statement in the 'if' block

This is my first time contributing here. Please let me know if any other process needs to be followed.

You did it right, thank you. If you have further questions, feel free to ask.

Thanks for your review. Now that the commit has been pushed to the codebase, should I Resolve the bug?

It will be done automatically :)

autoland is a staging branch, the bug will be closed when the change is moved to the nightly branch (called mozilla-central)

Now the patch is on the integration branch, autoland (https://treeherder.mozilla.org/#/jobs?repo=autoland). About twice a day the integration branch is merged into the main branch, mozilla-central (https://treeherder.mozilla.org/jobs?repo=mozilla-central).
So, if this doesn't cause unexpected test failures (it shouldn't), shortly it will be merged and automatically resolved.
You don't have to do anything since it's all automated.

Thanks a lot to both of you for the explanation. Looking forward to contribute more to the project :)

https://hg.mozilla.org/mozilla-central/rev/45e9724d1432