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

Issue 10388166: Construct a TCW for the login view's WebView. (Closed)

Created:
8 years, 7 months ago by Ben Goodger (Google)
Modified:
8 years, 7 months ago
Reviewers:
Nikita (slow)
CC:
chromium-reviews, stevenjb+watch_chromium.org, nkostylev+watch_chromium.org, davemoore+watch_chromium.org
Visibility:
Public.

Description

Construct a TCW for the login view's WebView. It needs this for Password Manager. It could be that we could just construct the Password Manager for this type, but this is the simplest fix. http://crbug.com/127110 TEST=See bug, configure proxy with auth required and open GAIA frame at sign in screen. Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=137512

Patch Set 1 #

Patch Set 2 : #

Patch Set 3 : #

Unified diffs Side-by-side diffs Delta from patch set Stats (+36 lines, -3 lines) Patch
M chrome/browser/chromeos/login/simple_web_view_dialog.h View 1 2 2 chunks +3 lines, -0 lines 0 comments Download
M chrome/browser/chromeos/login/simple_web_view_dialog.cc View 1 2 2 chunks +14 lines, -2 lines 0 comments Download
M chrome/browser/chromeos/login/webui_login_view.h View 2 chunks +7 lines, -0 lines 0 comments Download
M chrome/browser/chromeos/login/webui_login_view.cc View 2 chunks +12 lines, -1 line 0 comments Download

Messages

Total messages: 6 (0 generated)
Ben Goodger (Google)
8 years, 7 months ago (2012-05-16 17:25:37 UTC) #1
Avi (use Gerrit)
On 2012/05/16 17:25:37, Ben Goodger (Google) wrote: I'm a little concerned about this; TCWs have ...
8 years, 7 months ago (2012-05-16 17:28:10 UTC) #2
Ben Goodger (Google)
On 2012/05/16 17:28:10, Avi wrote: > I'm a little concerned about this; TCWs have a ...
8 years, 7 months ago (2012-05-16 17:39:50 UTC) #3
Nikita (slow)
lgtm, tested on fresh checkout with instructions https://sites.google.com/a/google.com/cros-connectivity-testing/proxy-testing This need to be merged into M20 ...
8 years, 7 months ago (2012-05-16 17:44:18 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/ben@chromium.org/10388166/3002
8 years, 7 months ago (2012-05-16 17:45:37 UTC) #5
commit-bot: I haz the power
8 years, 7 months ago (2012-05-16 21:17:29 UTC) #6
Change committed as 137512

Powered by Google App Engine
This is Rietveld 408576698