1443660 - Rename NavigationDelegate.onLoadUri() to onLoadUriRequest()

Status: RESOLVED FIXED

(GeckoView :: General, enhancement)

Description

[James Willcox (:snorp) ([e-mail]) (he/him)]

This is more consistent with other "can I do this" delegate methods.

Comment 1

[James Willcox (:snorp) ([e-mail]) (he/him)]

Attachment: Bug 1443660 - Rename NavigationDelegate.onLoadUri() to onLoadRequest()

Review commit: https://reviewboard.mozilla.org/r/225594/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/225594/

Comment 2

[(inactive) Jim Chen [:jchen] [:darchons]]

Comment on attachment 8956634 [details]
Bug 1443660 - Rename NavigationDelegate.onLoadUri() to onLoadRequest()

https://reviewboard.mozilla.org/r/225594/#review231466

Comment 3

[Eugen Sawin [:esawin]]

Comment on attachment 8956634 [details]
Bug 1443660 - Rename NavigationDelegate.onLoadUri() to onLoadRequest()

https://reviewboard.mozilla.org/r/225594/#review231840

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/bbb5759485fe
Rename NavigationDelegate.onLoadUri() to onLoadRequest() r=jchen,esawin

Comment 5

[Noemi Erli[:noemi_erli]]

Backed out 3 changesets (bug 1443660, bug 1443659, bug 1443658) for android-checkstyle bustage on a CLOSED TREE
 
Push with the failure: https://treeherder.mozilla.org/#/jobs?repo=mozilla-inbound&revision=c8275e1f531edac26f613b99499714708af3b01e&filter-resultStatus=testfailed&filter-resultStatus=busted&filter-resultStatus=exception&filter-resultStatus=retry&filter-resultStatus=usercancel&filter-resultStatus=runnable

Log failure example: https://treeherder.mozilla.org/logviewer.html#?job_id=166634548&repo=mozilla-inbound

Backout: https://hg.mozilla.org/integration/mozilla-inbound/rev/dd3beebf0ce3dda4f359907379aaa372449b022d

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/5e54022a7c56
Rename NavigationDelegate.onLoadUri() to onLoadRequest() r=jchen,esawin

Comment 7

[Natalia Csoregi [:nataliaCs]]

https://hg.mozilla.org/mozilla-central/rev/5e54022a7c56