Anonymous users always have $cached enabled even if drupal_page_is_cacheable is FALSE

Thanks for opening this report. There's definitely something weird going on here.

What's *supposed* to happen in Webform is the following:

- When the page cache is enabled, Webform should show the form to anonymous users even if they have exceeded the *per-user* submission limit. This is to prevent the page from getting cached with the form not shown.
- When the page cache is enabled, Webform show *not* show the form to anonymous users if the *total* form limit has been reached.

Now, when the page cache is not enabled, it's supposed to treat anonymous users identical to logged-in users, hiding or showing the form based on both the per-user and total limit immediately.

I think that your adjustment to the code is probably the correct thing to do for that line, since it's supposed to just determine if the page is cacheable at all. However, we can really simplify it all the way down to just this:

$cached = drupal_page_is_cacheable();

There's code later in the webform_node_view() function that still isn't going to behave as desired:

    // Disable the form if the limit is exceeded and page cache is not active.
    if (($total_limit_exceeded = _webform_submission_total_limit_check($node)) && !$cached) {
      $enabled = FALSE;
    }

I think that the !$cached check should just be removed here, since if the entire form limit has been reached, it's supposed to hide the form entirely.

I tested quicksketch's changes and they seem correct. The $cached is still passed to the message theme function, which is then used to prevent one user's messages from being cached.

Committed to 7.x-4.x and 8.x