-
Notifications
You must be signed in to change notification settings - Fork 10k
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
[api-minor][Editor] Don't use the editor parent which can be null. #15782
Conversation
a9ef7ae
to
482616a
Compare
Also, something that just occurred to me: Is this perhaps something that we want to try and uplift? |
Good question, let me think (chat with @marco-c) about it. |
/botio test |
From: Bot.io (Linux m4)ReceivedCommand cmd_test from @Snuffleupagus received. Current queue size: 0 Live output at: http://54.241.84.105:8877/36b28e684d7f658/output.txt |
From: Bot.io (Windows)ReceivedCommand cmd_test from @Snuffleupagus received. Current queue size: 0 Live output at: http://54.193.163.58:8877/64098853c2d09d6/output.txt |
From: Bot.io (Linux m4)FailedFull output at http://54.241.84.105:8877/36b28e684d7f658/output.txt Total script time: 25.44 mins
Image differences available at: http://54.241.84.105:8877/36b28e684d7f658/reftest-analyzer.html#web=eq.log |
From: Bot.io (Windows)FailedFull output at http://54.193.163.58:8877/64098853c2d09d6/output.txt Total script time: 31.19 mins
Image differences available at: http://54.193.163.58:8877/64098853c2d09d6/reftest-analyzer.html#web=eq.log |
/botio integrationtest |
From: Bot.io (Linux m4)ReceivedCommand cmd_integrationtest from @calixteman received. Current queue size: 0 Live output at: http://54.241.84.105:8877/235270412fdf8f0/output.txt |
From: Bot.io (Windows)ReceivedCommand cmd_integrationtest from @calixteman received. Current queue size: 0 Live output at: http://54.193.163.58:8877/dd550a8d561013d/output.txt |
From: Bot.io (Linux m4)SuccessFull output at http://54.241.84.105:8877/235270412fdf8f0/output.txt Total script time: 4.18 mins
|
From: Bot.io (Windows)FailedFull output at http://54.193.163.58:8877/dd550a8d561013d/output.txt Total script time: 12.47 mins
|
An annotation editor layer can be destroyed when it's invisible, hence some annotations can have a null parent but when printing/saving or when changing font size, color, ... of all added annotations (when selected with ctrl+a) we still need to have some parent properties especially the page dimensions, global scale factor and global rotation angle. This patch aims to remove all the references to the parent in the editor instances except in some cases where an editor should obviously have one. It fixes mozilla#15780.
/botio integrationtest |
From: Bot.io (Windows)ReceivedCommand cmd_integrationtest from @calixteman received. Current queue size: 0 Live output at: http://54.193.163.58:8877/01d7d5c6cabf114/output.txt |
From: Bot.io (Linux m4)ReceivedCommand cmd_integrationtest from @calixteman received. Current queue size: 0 Live output at: http://54.241.84.105:8877/f48d818a142a104/output.txt |
From: Bot.io (Linux m4)SuccessFull output at http://54.241.84.105:8877/f48d818a142a104/output.txt Total script time: 4.25 mins
|
From: Bot.io (Windows)FailedFull output at http://54.193.163.58:8877/01d7d5c6cabf114/output.txt Total script time: 10.48 mins
|
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, since I'm assuming that you've tested this carefully locally (given the amount of changes); thank you!
An annotation editor layer can be destroyed when it's invisible, hence some
annotations can have a null parent but when printing/saving or when changing
font size, color, ... of all added annotations (when selected with ctrl+a) we
still need to have some parent properties especially the page dimensions, global
scale factor and global rotation angle.
This patch aims to remove all the references to the parent in the editor instances
except in some cases where an editor should obviously have one.
It fixes #15780.