Bug 346068

Summary: Google Signon UI takes ages to load
Product: [Frameworks and Libraries] telepathy Reporter: David Edmundson <kde>
Component: accounts-kcmAssignee: Telepathy Bugs <kde-telepathy-bugs>
Status: RESOLVED NOT A BUG    
Severity: normal CC: mklapetek
Priority: NOR    
Version: git-latest   
Target Milestone: Future   
Platform: Other   
OS: Linux   
Latest Commit: Version Fixed In:

Description David Edmundson 2015-04-11 11:19:07 UTC
Went to set up a google account, clicked google. Nothing happened for about 20 seconds. I had thought it was broken and closed the window before the signon UI appeared. Happens every time.
Comment 1 Martin Klapetek 2015-04-13 10:40:34 UTC
Looks like a dbus timeout.

Does that happen everytime? Can you provide a bustle/dferry log and possibly also the output of kcmshell5 kaccounts?
Comment 2 David Edmundson 2015-04-13 10:48:08 UTC
Think it might have just been a slow network. seems fine today.

Would be nice if it showed the window whilst it loaded.
Comment 3 Martin Klapetek 2015-04-13 14:03:30 UTC
Indeed, signon-ui shows the dialog only on QWebView::loadFinished() signal, which isn't ideal.

I'll see if that can be improved.
Comment 4 Martin Klapetek 2015-04-13 19:51:01 UTC
For the record keeping purposes:

<mck182> hey, what would you think about this patch for signon-ui https://paste.kde.org/p8w8onz9a
<mck182> this way it appears immediately with the spinning wheel and it just happily spins away while the page is being loaded
<mardy> I don't like it, because it often happens that once the webpage is loaded, it immediately redirects you to the callback page, and in that case the current code doesn't show any UI at all
<mardy> the idea is that the client would use a spinner or some other indicator while performing the authentication
<mardy> there's also an underused feature in signond, that lets authentication plugins send progress reports to the client
<mardy> anyway, if you disagree, I don't mind having some #ifdef there
<mardy> ah, but no, ifdef is not very good
<mardy> there might be an Ubuntu machine where signon-ui is used by both KDE and Unity
<mardy> maybe an enviornment variable, or config file?

I think I'll just resort to set the whole UI to disabled and add some spinning wheels here and there.