The way this is used, I think `canViewObsolete()` would be more natural. As it is, `if (checkViewObsolete())` could be doing something if we are able to check the condition, or if we have checked the condition. Let's be definite about what this is saying.