Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(198)

Issue 9315011: Quick-and-dirty hacks for getting policy working behind proxies. (Closed)

Created:
8 years, 10 months ago by Mattias Nissler (ping if slow)
Modified:
8 years, 10 months ago
CC:
chromium-reviews, stevenjb+watch_chromium.org, nkostylev+watch_chromium.org, davemoore+watch_chromium.org
Visibility:
Public.

Description

Quick-and-dirty hacks for getting policy working behind proxies. Note that this hard-wires policy OAuth fetching in login_utils.cc to the authentication profile. I'm not entirely sure this is a good idea, but required since the user profile isn't in a state to allow successful requests at this point (the use-shared-proxies flag is off). Other stuff in login_utils.cc might need the hack too (sync?). BUG= TEST=

Patch Set 1 #

Total comments: 2
Unified diffs Side-by-side diffs Delta from patch set Stats (+13 lines, -7 lines) Patch
M chrome/browser/chromeos/login/login_utils.cc View 3 chunks +11 lines, -5 lines 2 comments Download
M chrome/browser/chromeos/proxy_config_service_impl.cc View 2 chunks +2 lines, -2 lines 0 comments Download

Messages

Total messages: 7 (0 generated)
Mattias Nissler (ping if slow)
Hi folks, this hack gets us back into the game. I've verified user policy fetching ...
8 years, 10 months ago (2012-01-31 21:05:19 UTC) #1
Mattias Nissler (ping if slow)
+pastarmovj
8 years, 10 months ago (2012-01-31 21:06:46 UTC) #2
kuan
lgtm for proxy config service part; i'm also ok with login part, but zel would ...
8 years, 10 months ago (2012-01-31 21:13:38 UTC) #3
zel
lgtm
8 years, 10 months ago (2012-01-31 21:30:40 UTC) #4
xiyuan
FYI, I've committed this CL on trunk. https://src.chromium.org/viewvc/chrome?view=rev&revision=120013
8 years, 10 months ago (2012-02-01 02:25:31 UTC) #5
Nikita (slow)
https://chromiumcodereview.appspot.com/9315011/diff/1/chrome/browser/chromeos/login/login_utils.cc File chrome/browser/chromeos/login/login_utils.cc (right): https://chromiumcodereview.appspot.com/9315011/diff/1/chrome/browser/chromeos/login/login_utils.cc#newcode792 chrome/browser/chromeos/login/login_utils.cc:792: // Make sure we flip every profile to not ...
8 years, 10 months ago (2012-02-01 09:42:34 UTC) #6
pastarmovj
8 years, 10 months ago (2012-02-01 09:57:31 UTC) #7
https://chromiumcodereview.appspot.com/9315011/diff/1/chrome/browser/chromeos...
File chrome/browser/chromeos/login/login_utils.cc (right):

https://chromiumcodereview.appspot.com/9315011/diff/1/chrome/browser/chromeos...
chrome/browser/chromeos/login/login_utils.cc:792: // Make sure we flip every
profile to not share proxies if the user hasn't
On 2012/02/01 09:42:34, Nikita Kostylev wrote:
> Seems that if Chrome crashes inside user session, this setting would have
> incorrect default value.

Yes, theoretically if Chrome crashes right after this code and before the
preferences were written to disk it might lead to flipping back to the default
being now enabled, until the user logs out and back in. Then this code will get
re-executed and flip the bit again. However you are right that we might want to
have this code on the "crashed browser restore" code path too.

Powered by Google App Engine
This is Rietveld 408576698