Bug 1093128 - Fix checking for the viewer to open when the user clicks a...

... link to protected content
There is already a test called "does not allow a choice that would subvert forward lock", so I added the "take 2".  I don't know whether it's OK to have two tests with the same name.  If it isn't OK, how would you prefer to distinguish the,?