New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 795075 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Dec 2017
Cc:
EstimatedDays: ----
NextAction: ----
OS: iOS
Pri: 1
Type: Bug



Sign in to add a comment

PushAndReplaceStateNavigationTestCase testHtml5HistoryPushUnicodeCharacters] failing

Project Member Reported by peterlaurens@chromium.org, Dec 14 2017

Issue description

PushAndReplaceStateNavigationTestCase testHtml5HistoryPushUnicodeCharacters] failing on simulator and device:

Test Case '-[PushAndReplaceStateNavigationTestCase testHtml5HistoryPushUnicodeCharacters]' failed (5.791 seconds).


https://logs.chromium.org/v/?s=chrome%2Fbb%2Finternal.bling.main%2Fipad10-simulator%2F9228%2F%2B%2Frecipes%2Fsteps%2Fios_chrome_web_egtests__iPad_Retina_iOS_10.0__on_Mac%2F0%2Fstdout
 
Labels: -Pri-3 Pri-1
f8625da2e7215aaac6c0c76a636ab8483090943d is the first bad commit
commit f8625da2e7215aaac6c0c76a636ab8483090943d
Author: Eric Lawrence <elawrence@chromium.org>
Date:   Thu Dec 14 18:36:37 2017 +0000

    Unescape fragment for display in Omnibox
    
    A change in Chrome 63 causes Chrome to urlencode the fragment component
    of URLs. The Chrome omnibox should decode the fragment component as it
    does for other URL components and matching Firefox.
    
    Bug:  789163 
    Change-Id: If82d72445d904495c8d3e4129ab63be141745328
    Reviewed-on: https://chromium-review.googlesource.com/822819
    Commit-Queue: Eric Lawrence <elawrence@chromium.org>
    Reviewed-by: Peter Kasting <pkasting@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#524123}

:040000 040000 379deb35dc01659901470341a52218258babb6f9 24df10597b057701f2fad5fcd27dfb580e6825ce M	chrome
:040000 040000 24545798ece61b75907ca8407502b7e2cc3cb986 524bcca422ef55dd5da642d46163bf97b7098ac4 M	components

Status: Started (was: Untriaged)
Labels: OS-iOS
Status: Fixed (was: Started)
The original change was reverted with:

https://chromium-review.googlesource.com/c/chromium/src/+/828080

Sign in to add a comment