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

Issue 595443 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Apr 2016
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug



Sign in to add a comment

IPv6 address contain double brackets

Project Member Reported by allada@chromium.org, Mar 16 2016

Issue description

Bug with IPv6 adding double [[ ]] around IPv6 addresses in external interface.
 
ipv6_incorrect.png
15.7 KB View Download
ipv6_correct.png
13.6 KB View Download

Comment 1 by allada@chromium.org, Mar 16 2016

Components: Platform>DevTools>Network

Comment 2 by allada@chromium.org, Mar 16 2016

The following commit caused this issue: https://codereview.chromium.org/1754713006/patch/60001/70001

Comment 3 by allada@chromium.org, Mar 30 2016

Status: Started (was: Fixed)
Status: Fixed (was: Started)
Project Member

Comment 5 by bugdroid1@chromium.org, Apr 8 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/c0d55a9bbafc294cd1ab29521a3c7e45eb40800c

commit c0d55a9bbafc294cd1ab29521a3c7e45eb40800c
Author: allada <allada@chromium.org>
Date: Fri Apr 08 22:06:46 2016

[DevTools] Fix to double wrapping IPv6 in square brackets

Removed DevTools code that added square brackets around IPv6 addresses.

Also adjusted test script to allow IPv6 localhost's [::1] ip address for
testing.

BUG= 595443 
R=lushnikov

Review URL: https://codereview.chromium.org/1818923004

Cr-Commit-Position: refs/heads/master@{#386218}

[modify] https://crrev.com/c0d55a9bbafc294cd1ab29521a3c7e45eb40800c/components/test_runner/web_frame_test_client.cc
[modify] https://crrev.com/c0d55a9bbafc294cd1ab29521a3c7e45eb40800c/third_party/WebKit/LayoutTests/TestExpectations
[modify] https://crrev.com/c0d55a9bbafc294cd1ab29521a3c7e45eb40800c/third_party/WebKit/LayoutTests/http/tests/inspector/resource-parameters-expected.txt
[add] https://crrev.com/c0d55a9bbafc294cd1ab29521a3c7e45eb40800c/third_party/WebKit/LayoutTests/http/tests/inspector/resource-parameters-ipv6-expected.txt
[add] https://crrev.com/c0d55a9bbafc294cd1ab29521a3c7e45eb40800c/third_party/WebKit/LayoutTests/http/tests/inspector/resource-parameters-ipv6.html
[modify] https://crrev.com/c0d55a9bbafc294cd1ab29521a3c7e45eb40800c/third_party/WebKit/LayoutTests/http/tests/inspector/resource-parameters.html
[modify] https://crrev.com/c0d55a9bbafc294cd1ab29521a3c7e45eb40800c/third_party/WebKit/LayoutTests/platform/linux/http/tests/inspector/resource-parameters-expected.txt
[add] https://crrev.com/c0d55a9bbafc294cd1ab29521a3c7e45eb40800c/third_party/WebKit/LayoutTests/platform/linux/http/tests/inspector/resource-parameters-ipv6-expected.txt
[modify] https://crrev.com/c0d55a9bbafc294cd1ab29521a3c7e45eb40800c/third_party/WebKit/LayoutTests/platform/win/http/tests/inspector/resource-parameters-expected.txt
[add] https://crrev.com/c0d55a9bbafc294cd1ab29521a3c7e45eb40800c/third_party/WebKit/LayoutTests/platform/win/http/tests/inspector/resource-parameters-ipv6-expected.txt
[modify] https://crrev.com/c0d55a9bbafc294cd1ab29521a3c7e45eb40800c/third_party/WebKit/Source/devtools/front_end/sdk/NetworkRequest.js

Sign in to add a comment