Oh, I see my oversight.  I was reading this as:

        return ((foundStartURL && foundValidEcValue) || urlMatchesCommLoginRedirectPattern);

Which would have been the same as before.  Can we add some comments in this method to briefly describe the workflow of these changes?  If I find it confusing, I'm guessing others will as well.