New issue
Advanced search Search tips

Issue 708561 link

Starred by 3 users

Issue metadata

Status: Verified
Owner:
Closed: Apr 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug



Sign in to add a comment

Proxy cannot be set up specifically for other protocols except HTTP

Project Member Reported by emaxx@chromium.org, Apr 5 2017

Issue description

Chrome Version: 59.0.3055.0
OS: Chrome OS

What steps will reproduce the problem?
(1) Open network settings window.
(2) Under Advanced>Proxy, fill any or all of the following fields: SOCKS / FTP / HTTPS.

What is the expected result?
The "SAVE" button is enabled and allows to apply the configured proxies.

What happens instead?
The "SAVE" button is disabled. It becomes enabled only after the field with the HTTP proxy gets filled.


This bug prevents from configuring, for instance, a sole SOCKS proxy. But this was possible in M58 and before.


UPD: Please see the attached screenshots for the "BEFORE" (M-58) and "AFTER" (M-59).
 
BEFORE__correct_m58.png
653 KB View Download
AFTER__wrong_m59.png
499 KB View Download

Comment 1 by emaxx@chromium.org, Apr 5 2017

Description: Show this description

Comment 2 by emaxx@chromium.org, Apr 5 2017

Description: Show this description
Labels: Hotlist-MD-Settings-Internet
Owner: steve...@chromium.org
Status: Assigned (was: Untriaged)
YouTube
xPrintServer-CP_QS.pdf
1.2 MB Download
Status: Started (was: Assigned)
Project Member

Comment 6 by bugdroid1@chromium.org, Apr 13 2017

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

commit 8bc32204bc129658adab8e1fb8be3058c723e6ca
Author: stevenjb <stevenjb@chromium.org>
Date: Thu Apr 13 06:24:49 2017

MD Settings: Internet: Proxy: Support partial proxies

The MD Settings proxy implementation did not support
proxy configurations that did not include a HTTP proxy.
This is a bug, we should support, e.g., just setting a
FTP proxy.

BUG= 708561 
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2815923002
Cr-Commit-Position: refs/heads/master@{#464320}

[modify] https://crrev.com/8bc32204bc129658adab8e1fb8be3058c723e6ca/chrome/browser/resources/settings/internet_page/network_proxy.js
[modify] https://crrev.com/8bc32204bc129658adab8e1fb8be3058c723e6ca/chrome/browser/resources/settings/internet_page/network_proxy_input.js

Status: Fixed (was: Started)
Status: Verified (was: Fixed)
9534.0.0, 60.0.3092.0

Sign in to add a comment