Last modified: 2013-10-16 14:46:45 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T56854, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 54854 - VisualEditor: "Welcome to VisualEditor" iframe causes Chrome to re-render the page
VisualEditor: "Welcome to VisualEditor" iframe causes Chrome to re-render the...
Status: RESOLVED WORKSFORME
Product: VisualEditor
Classification: Unclassified
MediaWiki integration (Other open bugs)
unspecified
All All
: Normal major
: ---
Assigned To: Trevor Parscal
: browser-test-bug
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2013-10-01 23:33 UTC by Chris McMahon
Modified: 2013-10-16 14:46 UTC (History)
8 users (show)

See Also:
Web browser: Google Chrome
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description Chris McMahon 2013-10-01 23:33:07 UTC
Best seen on a slow connection e.g at crossbrowsertesting.com; I used Chrome 29 on OSX 10.7

Seen on beta labs and test2wiki with a fresh browser instance: 

* Click Edit/beta, invoke VE

** View of page appears, then disappears, refreshes and appears again before "Welcome to..." iframe appears

* Click X to dismiss iframe

** iframe disappears for a moment, then reappears, then disappears again

Given a slow enough connection, this behavior could be bothersome for regular users of VE.  

It definitely causes constant failures in the browser tests for Chrome on both beta labs and test2wiki. 

This is not seen in Firefox
Comment 1 Chris McMahon 2013-10-11 13:19:15 UTC
still seeing this in automated tests

also, I reproduced this again manually using the slow connection at crossbrowsertesting.com as noted, and in addition to seeing the double-load of the page and double-load of the welcome notice, after clearing the notice I am presented with a message "You are not logged in".  This is impossible of course, since I have to be logged in with the opt-in preference set in order to be able to invoke VE at all. 

The "You are not logged in" notice does not appear consistently.
Comment 2 James Forrester 2013-10-15 15:43:00 UTC
I'm confused as to how the browser tests for Chrome passed 9 days after this bug was filed - https://wmf.ci.cloudbees.com/view/r-ve/job/VisualEditor-en.wikipedia.beta.wmflabs.org-linux-chrome/72/ - only to fail again immediately thereafter, where the only intervening change in VisualEditor's code base was an alteration to the browser tests - https://wmf.ci.cloudbees.com/view/r-ve/job/VisualEditor-en.wikipedia.beta.wmflabs.org-linux-chrome/73/ - which feels like a different issue, and one where the problem is with the tests, not the code they are testing?
Comment 3 Chris McMahon 2013-10-15 16:07:22 UTC
Because both the underlying page and the Welcome to... overlay appear and disappear and reappear and then finally disappear for good, there is a chance that the correct elements will be in place at the times that the test looks for them.  On rare occasions the  build will hit everything with perfect timing. 

The other interesting artifact of this issue is that the tests in each build fail for different reasons, because the page and the overlay are appearing and disappearing and reappearing.

Watching the behavior in crossbrowsertesting as noted above is instructive.  The appear/disappear/reappear/disappear-again behavior is easily seen, I just checked it again.

And while the appearance of the 'You are not logged in' notice despite being logged in is more difficult to reproduce, it seems to be a symptom of the same issue.
Comment 4 Chris McMahon 2013-10-15 21:12:47 UTC
This may only happen in Chrome 29, I have yet to repro in Chrome 30
Comment 5 Chris McMahon 2013-10-16 14:46:45 UTC
This bug was reported and is reproducible against Chrome version 29 but does not appear in Chrome version 30.  Closing.

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links