text: Title: Failed keywords inside skipped tests are not expanded
  Body: We are just upgrading from 3.x to 6.0.2. We are moving our test framework to use the "skipped"-concept instead of the deprecated "critical/non-critical"-concept. We noticed that in log.html, the failed KWs are not automatically expanded for a test skipped using skipOnFailure.
  
  Reproducing:
  - click a  failed skipped test in report.html.
  - Failed nested KWs are not automatically expanded in log.html
  
  For a failed non-skipped test, the failed KWs are automatically recursively expanded. The same happens for failed non-critical test in 3.x. We would expect and like the same behaviour for the failed skipped tests. Especially for nested KWs, it's irritating to manually expand the failed nested KWs to get to the lowest level failed KW. 