• Christos Froussios's avatar
    Reland "OOR-CORS: store more information to CORSErrorStatus on preflight errors" · 4741ed4f
    Christos Froussios authored
    This reverts commit c4189fd6.
    
    Reason for revert: Reverting caused other, bigger failures. I will
    disable the test instead.
    
    Original change's description:
    > Revert "OOR-CORS: store more information to CORSErrorStatus on preflight errors"
    > 
    > This reverts commit 1f675b46.
    > 
    > Reason for revert: Breaks builder WebKit Linux Trusty ASAN
    > 
    > Original change's description:
    > > OOR-CORS: store more information to CORSErrorStatus on preflight errors
    > > 
    > > Current implementation does not store rejected method or header
    > > information to CORSErrorStatus on CORS-preflight checks, and
    > > results in missing hint parameter on CreateErrorString() in Blink.
    > > This patch provides required information to generate right error
    > > messages, and fixes some crashed layout tests.
    > > 
    > > Bug: 803766, 836741
    > > Cq-Include-Trybots: master.tryserver.chromium.linux:linux_mojo
    > > Change-Id: If4de6b916a77d204812a7b010e39080bf059bc9e
    > > Tbr: kinuko@chromium.org
    > > Reviewed-on: https://chromium-review.googlesource.com/1030670
    > > Commit-Queue: Takashi Toyoshima <toyoshim@chromium.org>
    > > Reviewed-by: Yutaka Hirano <yhirano@chromium.org>
    > > Cr-Commit-Position: refs/heads/master@{#554642}
    > 
    > TBR=kinuko@chromium.org,toyoshim@chromium.org,yhirano@chromium.org
    > 
    > # Not skipping CQ checks because original CL landed > 1 day ago.
    > 
    > Bug: 803766, 836741, 838057
    > Change-Id: Ia8eb9bf63b1b0c6edda93eb7d45c1bb20d26a7b7
    > Cq-Include-Trybots: master.tryserver.chromium.linux:linux_mojo
    > Reviewed-on: https://chromium-review.googlesource.com/1034532
    > Reviewed-by: Christos Froussios <cfroussios@chromium.org>
    > Commit-Queue: Christos Froussios <cfroussios@chromium.org>
    > Cr-Commit-Position: refs/heads/master@{#554712}
    
    TBR=kinuko@chromium.org,toyoshim@chromium.org,yhirano@chromium.org,cfroussios@chromium.org
    
    Change-Id: Iaf20d807cf9f71976a3d1c8a46678afb48682029
    No-Presubmit: true
    No-Tree-Checks: true
    No-Try: true
    Bug: 803766, 836741, 838057
    Cq-Include-Trybots: master.tryserver.chromium.linux:linux_mojo
    Reviewed-on: https://chromium-review.googlesource.com/1034702Reviewed-by: default avatarChristos Froussios <cfroussios@chromium.org>
    Commit-Queue: Christos Froussios <cfroussios@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#554726}
    4741ed4f
TestExpectations 364 KB