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

Issue 169443003: Added support for additional popup-blocking policies on Android. (Closed)

Created:
6 years, 10 months ago by Joao da Silva
Modified:
6 years, 10 months ago
CC:
chromium-reviews, joaodasilva+watch_chromium.orgm, Bernhard Bauer
Visibility:
Public.

Description

Added support for additional popup-blocking policies on Android. This platform also supports PopupsAllowedForUrls and PopupsBlockedForUrls. If these policies are enabled on the template then they'll be loaded by the cloud policy provider. BUG=341493 Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=252043

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+2 lines, -0 lines) Patch
M components/policy/resources/policy_templates.json View 2 chunks +2 lines, -0 lines 0 comments Download

Messages

Total messages: 8 (0 generated)
Joao da Silva
Bernhard, PTAL. These 2 escaped during review of your earlier CL adding support for the ...
6 years, 10 months ago (2014-02-17 15:46:20 UTC) #1
Joao da Silva
Bernhard is OOO. Daniel, can you have a look?
6 years, 10 months ago (2014-02-19 08:33:52 UTC) #2
dconnelly
lgtm
6 years, 10 months ago (2014-02-19 08:36:38 UTC) #3
Joao da Silva
The CQ bit was checked by joaodasilva@chromium.org
6 years, 10 months ago (2014-02-19 08:57:08 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/joaodasilva@chromium.org/169443003/1
6 years, 10 months ago (2014-02-19 08:57:18 UTC) #5
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/joaodasilva@chromium.org/169443003/1
6 years, 10 months ago (2014-02-19 14:52:23 UTC) #6
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/joaodasilva@chromium.org/169443003/1
6 years, 10 months ago (2014-02-19 15:26:53 UTC) #7
commit-bot: I haz the power
6 years, 10 months ago (2014-02-19 19:34:54 UTC) #8
Message was sent while issue was closed.
Change committed as 252043

Powered by Google App Engine
This is Rietveld 408576698