Last modified: 2013-08-28 18:56:01 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 T55360, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 53360 - VisualEditor: Displayed content and DM out of sync with fast insertion (primarily from test scripts?)
VisualEditor: Displayed content and DM out of sync with fast insertion (prima...
Status: RESOLVED FIXED
Product: VisualEditor
Classification: Unclassified
ContentEditable (Other open bugs)
unspecified
All All
: High major
: VE-deploy-2013-08-29
Assigned To: D Chan
: browser-test-bug
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2013-08-26 18:15 UTC by Chris McMahon
Modified: 2013-08-28 18:56 UTC (History)
7 users (show)

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


Attachments
example 1 (69.94 KB, image/png)
2013-08-26 18:15 UTC, Chris McMahon
Details
example 1 (89.55 KB, image/png)
2013-08-26 18:16 UTC, Chris McMahon
Details
example 2 (64.38 KB, image/png)
2013-08-26 18:16 UTC, Chris McMahon
Details
example 2 (92.36 KB, image/png)
2013-08-26 18:17 UTC, Chris McMahon
Details
example 3 (76.21 KB, image/png)
2013-08-26 18:17 UTC, Chris McMahon
Details
example 3 (84.02 KB, image/png)
2013-08-26 18:17 UTC, Chris McMahon
Details

Description Chris McMahon 2013-08-26 18:15:27 UTC
Created attachment 13171 [details]
example 1

new issue as of Aug 26, seen on test2wiki

this is easily triggered by an automated browser test but can also be triggered manually: 

while typing edits, click the Save page button.  
click Review 

Edited page and diff view contents are not the same. See screen shot examples 1 and 2. 

ex 1: "asfdaEdit" in edited page vs "asEdadit" in diff view where original page has "Edit"

ex 2: "aaqaaaStarting" in edited page vs "aSqaaatarting" in diff view where original page has "Starting"

in at least one case, the actual saved contents of the page differs from the buffer that appears upon clicking Edit to invoke VE, see example 3
Comment 1 Chris McMahon 2013-08-26 18:16:23 UTC
Created attachment 13172 [details]
example 1
Comment 2 Chris McMahon 2013-08-26 18:16:46 UTC
Created attachment 13173 [details]
example 2
Comment 3 Chris McMahon 2013-08-26 18:17:09 UTC
Created attachment 13174 [details]
example 2
Comment 4 Chris McMahon 2013-08-26 18:17:32 UTC
Created attachment 13175 [details]
example 3
Comment 5 Chris McMahon 2013-08-26 18:17:57 UTC
Created attachment 13176 [details]
example 3
Comment 6 James Forrester 2013-08-27 19:58:44 UTC
I still can't reproduce these in Chrome/Firefox/Safari/Opera testing on local master, on production test2, or production enwiki/mediawikiwiki.

What does "can also be triggered manually" mean - how? Or is this an unreliable/occasional thing? Possibly a system-load issue?

Is this specific to test2?

From the screenshots, the displayed content (from CE) and the data model (in DM) appear to have gotten out of sync, which Shouldn't Ever Happen(tm).
Comment 7 Chris McMahon 2013-08-27 21:27:24 UTC
I can reproduce this consistently on test2wiki most of the time manually and automatedly all of the time as of the last VE deploy.  To repro manually (I use Chrome): 

* Pick a random page on test2wiki that has existing text
* click Edit for VE. click no other thing.
* Poise your right hand so that you are prepared to click 'Save page'
* With your left hand type randomly very very quickly. While typing with your left hand, click 'Save page' with your right hand

* take a look at the VE page contents (CE I assume).  It will appear to be accurate.  It will be different than what is in the diff view (DM I assume). 

Note: the automated test moves quickly enough that *nothing* typed into the VE edit interface is reflected in the diff view.  I can demo this if you'd like.

Again, this is new behavior as of Aug 26.
Comment 8 Chris McMahon 2013-08-27 21:28:54 UTC
I should mention that the (corrupted) diff view contents is what is actually saved when finally saving the page in VE
Comment 9 James Forrester 2013-08-28 02:30:08 UTC
This has been fixed in VisualEditor master as part of Gerrit change #81213, and we will do an emergency push of this to wmf14 tomorrow to fix it in product; sorry for the disruption.

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


Navigation
Links