BrowserSwitcherUseIeSitelist is not Windows-only |
|||||
Issue descriptionThe BrowserSwitcherUseIeSitelist is only used on Windows in the code, but the policy itself says it's supported on other platforms. We should update policy data so the documentation gets generated properly.
,
Nov 13
Requesting a merge into M71
,
Nov 13
Before we approve merge to M71, please answer followings: * Is this M71 regression? Is it critical? * Is the change well baked/verified in Canary, having enough automation tests coverage and safe to merge to M71? (The change is not in canary yet as it landed 4 hrs back) * Any other important details to justify the merge. Please note M71 is already in Beta, so merge bar is very high. Thank you.
,
Nov 13
Also is this require a string change? If yes, it is too late for M71.
,
Nov 13
> Is this M71 regression? The new policy that's buggy was introduced in M71. > Is it critical? No, but it is just a documentation change. It is low-risk (single line change), and can help avoid confusion for admins. > Is the change well baked/verified in Canary, having enough automation tests coverage and safe to merge to M71? It's not landed in Canary yet, and there are no automated tests for this. It's still low-risk, since it mostly affects documentation, and is for a feature that only affects a small subset of users.
,
Nov 13
Approving merge to M71 branch 3578 based on comment #5.
,
Nov 13
Ah, that was my mistake. It looks like the bug was introduced in M72, not M71. So, there's no need for a merge after all, it looks like. :-)
,
Nov 13
Rejecting merge to M71 based on comment #7. Thank you. |
|||||
►
Sign in to add a comment |
|||||
Comment 1 by bugdroid1@chromium.org
, Nov 13