Bug 1736175 Comment 31 Edit History

Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.

### Beta/Release Uplift Approval Request
* **User impact if declined**: Firefox profiles will continue to not work on network shares.
* **Is this code covered by automated tests?**: No
* **Has the fix been verified in Nightly?**: Yes
* **Needs manual test from QE?**: No
* **If yes, steps to reproduce**: 
* **List of other uplifts needed**: None
* **Risk to taking this patch**: Low
* **Why is the change risky/not risky? (and alternatives if risky)**: This patch changes some methods to do path joining through an API instead of using forward slashes. The path joining API has been well tested for several versions.
* **String changes made/needed**:
### Beta/Release Uplift Approval Request
* **User impact if declined**: Firefox profiles will continue to not work on network shares.
* **Is this code covered by automated tests?**: Yes
* **Has the fix been verified in Nightly?**: Yes
* **Needs manual test from QE?**: No
* **If yes, steps to reproduce**: 
* **List of other uplifts needed**: None
* **Risk to taking this patch**: Low
* **Why is the change risky/not risky? (and alternatives if risky)**: This patch changes some methods to do path joining through an API instead of using forward slashes. The path joining API has been well tested for several versions.
* **String changes made/needed**:


EDIT: yes this is covered by unit tests for the sync engines module, which all passed.

Back to Bug 1736175 Comment 31