[Webkit-unassigned] [Bug 63591] New: [Chromium] isRedirect is incorrectly determined for decidePolicyForNavigation

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=63591

           Summary: [Chromium] isRedirect is incorrectly determined for
                    decidePolicyForNavigation
           Product: WebKit
           Version: 528+ (Nightly build)
          Platform: Unspecified
        OS/Version: Unspecified
            Status: UNCONFIRMED
          Severity: Normal
          Priority: P2
         Component: New Bugs
        AssignedTo: webkit-unassigned at lists.webkit.org
        ReportedBy: binji at chromium.org



Ben Smith <binji at chromium.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #99017|                            |review?, commit-queue?
               Flag|                            |


Created an attachment (id=99017)
 --> (https://bugs.webkit.org/attachment.cgi?id=99017&action=review)
Patch to fix this bug.

In FrameLoaderClientImpl::dispatchDecidePolicyForNavigationAction()

isRedirect is currently determined by hasRedirectChain(), which checks for an non-empty redirect chain. Instead, it should check for a redirect chain with more than one element. All valid redirects will have at least the source URL and the destination URL in the redirect chain.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.