Merge lp://staging/~mardy/signon-ui/end-request2 into lp://staging/signon-ui
Proposed by
Alberto Mardegan
Status: | Merged |
---|---|
Approved by: | David King |
Approved revision: | 58 |
Merged at revision: | 58 |
Proposed branch: | lp://staging/~mardy/signon-ui/end-request2 |
Merge into: | lp://staging/signon-ui |
Diff against target: |
15 lines (+6/-0) 1 file modified
src/browser-request.cpp (+6/-0) |
To merge this branch: | bzr merge lp://staging/~mardy/signon-ui/end-request2 |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
David King (community) | Approve | ||
jenkins (community) | continuous-integration | Approve | |
Review via email: mp+123921@code.staging.launchpad.net |
Description of the change
Ignore any webview signals since the authentication is completed
This is needed because QWebView might still emit loadFinished(false)
(which we would interpret as an error) on the final URL, which we don't
care about anymore.
To post a comment you must log in.
PASSED: Continuous integration, rev:58 s-jenkins: 8080/job/ online- accounts- signon- ui-ci/10/ s-jenkins: 8080/job/ online- accounts- signon- ui-ci/. /label= quantal/ 10/console
http://
Executed test runs:
SUCCESS: http://