Skip to content
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

Firefox_nss: Revert unnecessary x11utils::turn_off_gnome_screensaver call #7874

Merged
merged 1 commit into from Jul 16, 2019

Conversation

jouyingbin
Copy link
Contributor

@jouyingbin jouyingbin commented Jul 12, 2019

Firefox_nss calls x11utils::turn_off_gnome_screensaver is a simple
script_run 'gsettings set org.gnome.desktop.session idle-delay 0';

  1. In order to avoid no terminal is shown the string which is just typed "into the void"
    of the desktop session. Therefore there is also never an output on the serial port.
  2. This patch can also fixed the side effect error about case running after firefox_nss.

Error run: https://openqa.suse.de/tests/3022911#step/firefox_nss/4

@jouyingbin jouyingbin changed the title Firefox_nss: revert unnecessary x11utils::turn_off_gnome_screensaver call Firefox_nss: Revert unnecessary x11utils::turn_off_gnome_screensaver call Jul 12, 2019
Copy link
Contributor

@lilyeyes lilyeyes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

tests/fips/mozilla_nss/firefox_nss.pm Show resolved Hide resolved
tests/fips/mozilla_nss/firefox_nss.pm Outdated Show resolved Hide resolved
@ghost
Copy link

ghost commented Jul 16, 2019

The change does not explain why turn_off_gnome_screensaver make following select_console failed. If we want to dive deeper we should check the code which implement turn_off_gnome_screensaver in x11utils. But as a work around it is acceptable to simply remove it from the code.

@jouyingbin jouyingbin changed the title Firefox_nss: Revert unnecessary x11utils::turn_off_gnome_screensaver call [WIP]Firefox_nss: Revert unnecessary x11utils::turn_off_gnome_screensaver call Jul 16, 2019
…call

Firefox_nss calls x11utils::turn_off_gnome_screensaver is a simple
script_run 'gsettings set org.gnome.desktop.session idle-delay 0';

1.In order to avoid no terminal is shown the string which is just typed "into the void"
  of the desktop session. Therefore there is also never an output on the serial port.
2.This patch can also fixed the side effect error about case running after firefox_nss.
@jouyingbin
Copy link
Contributor Author

The change does not explain why turn_off_gnome_screensaver make following select_console failed. If we want to dive deeper we should check the code which implement turn_off_gnome_screensaver in x11utils. But as a work around it is acceptable to simply remove it from the code.

Firefox_nss calls x11utils::turn_off_gnome_screensaver is a simple script_run 'gsettings set org.gnome.desktop.session idle-delay 0';

We can either add select_console 'root-console' or xterm to type the command script_run 'gsettings set org.gnome.desktop.session idle-delay 0'; e.g., add x11_start_program('xterm') and then run turn_off_gnome_screensaver if check_var('DESKTOP', 'gnome'), I think it can also fix the problem too.

Currently, I don't see the reason that we need to really turn off the screen saver, so I plan to revert the turn_off_gnome_screensaver function call.

Thanks.

@jouyingbin jouyingbin changed the title [WIP]Firefox_nss: Revert unnecessary x11utils::turn_off_gnome_screensaver call Firefox_nss: Revert unnecessary x11utils::turn_off_gnome_screensaver call Jul 16, 2019
Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@lemon-suse
Copy link
Contributor

LGTM.

@lemon-suse lemon-suse merged commit 23d29e2 into os-autoinst:master Jul 16, 2019
@jouyingbin jouyingbin deleted the firefox_nss branch July 16, 2019 09:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants