Remove request policy attribute from dynamic page cache subscriber

Makes sense to me.

+++ b/core/modules/dynamic_page_cache/src/EventSubscriber/DynamicPageCacheSubscriber.php
@@ -104,6 +96,13 @@ class DynamicPageCacheSubscriber implements EventSubscriberInterface {
+  protected $requestPolicyResult;

Needs trailing s. Can be fixed on commit.

Looks great. Less request attributes the better.

Committed/pushed to 8.1.x only because this is internal refactoring, removes the constant, adds the protected property etc. I can't see a reason this would need to go into 8.0.x but if there is one please re-open for backport.

Fixed the missing s on the protected property.

Looks great. Less request attributes the better.

Exactly :)

This failed to update one comment. Created a minor follow-up: #2759247: Request attribute removed from DynamicPageCacheSubscriber, but comment was not updated.