-
Notifications
You must be signed in to change notification settings - Fork 9.9k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
For text widgets, get the text from the AP stream instead of from the format callback (bug 1825002) #16433
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
r=me, with passing tests; thank you!
/botio test |
From: Bot.io (Linux m4)ReceivedCommand cmd_test from @calixteman received. Current queue size: 0 Live output at: http://54.241.84.105:8877/69a4d99a9efe4ef/output.txt |
From: Bot.io (Windows)ReceivedCommand cmd_test from @calixteman received. Current queue size: 1 Live output at: http://54.193.163.58:8877/3e770b532fa1a07/output.txt |
From: Bot.io (Windows)FailedFull output at http://54.193.163.58:8877/3e770b532fa1a07/output.txt Total script time: 13.87 mins
|
From: Bot.io (Linux m4)FailedFull output at http://54.241.84.105:8877/69a4d99a9efe4ef/output.txt Total script time: 26.97 mins
Image differences available at: http://54.241.84.105:8877/69a4d99a9efe4ef/reftest-analyzer.html#web=eq.log |
/botio test |
From: Bot.io (Windows)ReceivedCommand cmd_test from @calixteman received. Current queue size: 0 Live output at: http://54.193.163.58:8877/fca22bff7a0ee8d/output.txt |
From: Bot.io (Linux m4)ReceivedCommand cmd_test from @calixteman received. Current queue size: 0 Live output at: http://54.241.84.105:8877/1f5c02cd7b40728/output.txt |
From: Bot.io (Linux m4)FailedFull output at http://54.241.84.105:8877/1f5c02cd7b40728/output.txt Total script time: 27.04 mins
Image differences available at: http://54.241.84.105:8877/1f5c02cd7b40728/reftest-analyzer.html#web=eq.log |
Unfortunately the latest version of the patch still breaks the comb-field in the |
From: Bot.io (Windows)FailedFull output at http://54.193.163.58:8877/fca22bff7a0ee8d/output.txt Total script time: 35.48 mins
Image differences available at: http://54.193.163.58:8877/fca22bff7a0ee8d/reftest-analyzer.html#web=eq.log |
… format callback (bug 1825002) When fixing bug 1766987, I thought the field formatted value came from the result of the format callback: I was wrong. The format callback is ran but the value is unused (maybe it's useful to set some global vars... or it's just a bug in Acrobat). Anyway the value to display is the one rendered in the AP stream. The field value setter has been simplified and that fixes issue mozilla#16409.
/botio test |
From: Bot.io (Windows)ReceivedCommand cmd_test from @calixteman received. Current queue size: 0 Live output at: http://54.193.163.58:8877/a91e28196396e81/output.txt |
From: Bot.io (Linux m4)ReceivedCommand cmd_test from @calixteman received. Current queue size: 0 Live output at: http://54.241.84.105:8877/a40f29a666828bc/output.txt |
From: Bot.io (Linux m4)FailedFull output at http://54.241.84.105:8877/a40f29a666828bc/output.txt Total script time: 27.26 mins
Image differences available at: http://54.241.84.105:8877/a40f29a666828bc/reftest-analyzer.html#web=eq.log |
From: Bot.io (Windows)FailedFull output at http://54.193.163.58:8877/a91e28196396e81/output.txt Total script time: 34.88 mins
Image differences available at: http://54.193.163.58:8877/a91e28196396e81/reftest-analyzer.html#web=eq.log |
The last version of the patch seems fine, since all relevant tests pass now :-) |
/botio makeref |
From: Bot.io (Linux m4)ReceivedCommand cmd_makeref from @calixteman received. Current queue size: 0 Live output at: http://54.241.84.105:8877/173d6e6ac35c13f/output.txt |
From: Bot.io (Windows)ReceivedCommand cmd_makeref from @calixteman received. Current queue size: 0 Live output at: http://54.193.163.58:8877/60645fa5b36d83d/output.txt |
From: Bot.io (Linux m4)SuccessFull output at http://54.241.84.105:8877/173d6e6ac35c13f/output.txt Total script time: 23.31 mins
|
From: Bot.io (Windows)SuccessFull output at http://54.193.163.58:8877/60645fa5b36d83d/output.txt Total script time: 24.65 mins
|
When fixing bug 1766987, I thought the field formatted value came from the result of the format callback: I was wrong. The format callback is ran but the value is unused (maybe it's useful to set some global vars... or it's just a bug in Acrobat). Anyway the value to display is the one rendered in the AP stream.
The field value setter has been simplified and that fixes issue #16409.