Chromium Code Reviews
Help | Chromium Project | Sign in
(203)

Issue 11301016: [cros] Prevent extensions from opening dialogs on Login screen. (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
2 years, 7 months ago by Ivan Korotkov
Modified:
2 years, 7 months ago
Reviewers:
Aaron Boodman
CC:
chromium-reviews, chromium-apps-reviews_chromium.org
Visibility:
Public.

Description

[cros] Prevent extensions from opening dialogs on Login screen. BUG=chrome-os-partner:13898 Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=164806

Patch Set 1 #

Messages

Total messages: 9 (0 generated)
Ivan Korotkov
Hi Aaron, could you please take a look?
2 years, 7 months ago (2012-10-29 12:58:26 UTC) #1
Aaron Boodman
Weird layering to have this class know so much about the context it is used ...
2 years, 7 months ago (2012-10-29 15:52:59 UTC) #2
Ivan Korotkov
Looking at ExtensionHostManager, it seems that a new ExtensionHost instance is created for each extension's ...
2 years, 7 months ago (2012-10-29 16:11:09 UTC) #3
Nikita (OOO from May 25)
On 2012/10/29 16:11:09, Ivan Korotkov wrote: > We need some trivial change here, as we ...
2 years, 7 months ago (2012-10-29 18:52:22 UTC) #4
Aaron Boodman
OK, temporary change LGTM
2 years, 7 months ago (2012-10-29 19:12:54 UTC) #5
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/ivankr@chromium.org/11301016/1
2 years, 7 months ago (2012-10-29 19:15:14 UTC) #6
commit-bot: I haz the power
Retried try job too often for step(s) content_browsertests
2 years, 7 months ago (2012-10-29 21:07:48 UTC) #7
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/ivankr@chromium.org/11301016/1
2 years, 7 months ago (2012-10-29 21:32:26 UTC) #8
commit-bot: I haz the power
2 years, 7 months ago (2012-10-30 01:07:16 UTC) #9
Change committed as 164806
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld ec887be