CR feedback for filters

src/Microsoft.AspNet.Mvc.Core/Properties/Resources.Designer.cs

@@ -619,7 +619,7 @@ namespace Microsoft.AspNet.Mvc.Core
         }
 
         /// <summary>
-        /// If an {0} provides a result value by setting the {1} property of {2} to a non-null value, then the it should not call the next filter by invoking {3}.
+        /// If an {0} provides a result value by setting the {1} property of {2} to a non-null value, then it cannot call the next filter by invoking {3}.
         /// </summary>
         internal static string AsyncActionFilter_InvalidShortCircuit
         {
@@ -627,7 +627,7 @@ namespace Microsoft.AspNet.Mvc.Core
         }
 
         /// <summary>
-        /// If an {0} provides a result value by setting the {1} property of {2} to a non-null value, then the it should not call the next filter by invoking {3}.
+        /// If an {0} provides a result value by setting the {1} property of {2} to a non-null value, then it cannot call the next filter by invoking {3}.
         /// </summary>
         internal static string FormatAsyncActionFilter_InvalidShortCircuit(object p0, object p1, object p2, object p3)
         {
@@ -635,7 +635,7 @@ namespace Microsoft.AspNet.Mvc.Core
         }
 
         /// <summary>
-        /// If an {0} cancels execution by setting the {1} property of {2} to 'true', then the it should not call the next filter by invoking {3}.
+        /// If an {0} cancels execution by setting the {1} property of {2} to 'true', then it cannot call the next filter by invoking {3}.
         /// </summary>
         internal static string AsyncResultFilter_InvalidShortCircuit
         {
@@ -643,7 +643,7 @@ namespace Microsoft.AspNet.Mvc.Core
         }
 
         /// <summary>
-        /// If an {0} cancels execution by setting the {1} property of {2} to 'true', then the it should not call the next filter by invoking {3}.
+        /// If an {0} cancels execution by setting the {1} property of {2} to 'true', then it cannot call the next filter by invoking {3}.
         /// </summary>
         internal static string FormatAsyncResultFilter_InvalidShortCircuit(object p0, object p1, object p2, object p3)
         {

src/Microsoft.AspNet.Mvc.Core/ReflectedActionInvoker.cs

@@ -350,7 +350,6 @@ namespace Microsoft.AspNet.Mvc
                 // Exceptions thrown by the action method OR filters bubble back up through ActionExcecutedContext.
                 _actionExecutedContext = new ActionExecutedContext(_actionExecutingContext, _filters)
                 {
-                    Exception = exception,
                     ExceptionDispatchInfo = ExceptionDispatchInfo.Capture(exception)
                 };
             }
@@ -467,7 +466,6 @@ namespace Microsoft.AspNet.Mvc
             {
                 _resultExecutedContext = new ResultExecutedContext(_resultExecutingContext, _filters, _resultExecutingContext.Result)
                 {
-                    Exception = exception,
                     ExceptionDispatchInfo = ExceptionDispatchInfo.Capture(exception)
                 };
             }

src/Microsoft.AspNet.Mvc.Core/Resources.resx

@@ -232,10 +232,10 @@
     <value>No route matches the supplied values.</value>
   </data>
   <data name="AsyncActionFilter_InvalidShortCircuit" xml:space="preserve">
-    <value>If an {0} provides a result value by setting the {1} property of {2} to a non-null value, then the it should not call the next filter by invoking {3}.</value>
+    <value>If an {0} provides a result value by setting the {1} property of {2} to a non-null value, then it cannot call the next filter by invoking {3}.</value>
   </data>
   <data name="AsyncResultFilter_InvalidShortCircuit" xml:space="preserve">
-    <value>If an {0} cancels execution by setting the {1} property of {2} to 'true', then the it should not call the next filter by invoking {3}.</value>
+    <value>If an {0} cancels execution by setting the {1} property of {2} to 'true', then it cannot call the next filter by invoking {3}.</value>
   </data>
   <data name="ViewEngine_PartialViewNotFound" xml:space="preserve">
     <value>The partial view '{0}' was not found. The following locations were searched:{1}</value>

test/Microsoft.AspNet.Mvc.Core.Test/ReflectedActionInvokerTest.cs

@@ -623,7 +623,7 @@ namespace Microsoft.AspNet.Mvc
 
             var message =
                 "If an IAsyncActionFilter provides a result value by setting the Result property of " +
-                "ActionExecutingContext to a non-null value, then the it should not call the next filter by invoking " +
+                "ActionExecutingContext to a non-null value, then it cannot call the next filter by invoking " +
                 "ActionExecutionDelegate.";
 
             var invoker = CreateInvoker(actionFilter.Object);
@@ -972,7 +972,7 @@ namespace Microsoft.AspNet.Mvc
 
             var message = 
                 "If an IAsyncResultFilter cancels execution by setting the Cancel property of " +
-                "ResultExecutingContext to 'true', then the it should not call the next filter by invoking " +
+                "ResultExecutingContext to 'true', then it cannot call the next filter by invoking " +
                 "ResultExecutionDelegate.";
 
             var invoker = CreateInvoker(filter.Object);