Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Disable PrettyPrintJSONDocument #754

Merged
merged 1 commit into from
Sep 29, 2023
Merged

Conversation

tmancey
Copy link
Contributor

@tmancey tmancey commented Sep 29, 2023

No description provided.

@tmancey tmancey self-assigned this Sep 29, 2023
@tmancey tmancey requested a review from a team as a code owner September 29, 2023 19:29
@tmancey tmancey force-pushed the disable_pretty_print_json_document branch from 2f2e7d5 to f0b8fee Compare September 29, 2023 19:30
@tmancey tmancey requested a review from aseren September 29, 2023 19:30
@tmancey tmancey force-pushed the disable_pretty_print_json_document branch 4 times, most recently from e2cd9c8 to f59236f Compare September 29, 2023 19:37
@tmancey tmancey force-pushed the disable_pretty_print_json_document branch from f59236f to 297ef79 Compare September 29, 2023 19:38
@tmancey tmancey merged commit 64f2a87 into main Sep 29, 2023
3 checks passed
@tmancey tmancey deleted the disable_pretty_print_json_document branch September 29, 2023 19:48
@kjozwiak
Copy link
Member

kjozwiak commented Oct 2, 2023

Nightly - PrettyPrintJSONDocument:Disabled_EmergencyKillSwitch

Verification PASSED on Win 11 x64 using the following build(s):

Brave | 1.60.73 Chromium: 118.0.5993.21 (Official Build) nightly (64-bit)
-- | --
Revision | 3152feb8ff2ed6443c63daf2bac09b9cd9a04798
OS | Windows 11 Version 22H2 (Build 22621.2361)
  • downloaded/installed 1.60.73 Chromium: 118.0.5993.21 and started Brave using the following
    • brave.exe --enable-logging=stderr --variations-server-url=https://variations.bravesoftware.com/seed
  • ensured that PrettyPrintJSONDocument isn't visible under brave://version
Example Example Example
image image image
Example Example
image image

BETA - PrettyPrintJSONDocument:Disabled_EmergencyKillSwitch

Verification PASSED on Win 11 x64 using the following build(s):

Brave | 1.59.107 Chromium: 117.0.5938.140 (Official Build) beta (64-bit)
-- | --
Revision | 5bbd53ed3e41461e15555b7ce0490464656b236f
OS | Windows 11 Version 22H2 (Build 22621.2361)
  • downloaded/installed 1.59.107 Chromium: 117.0.5938.140 and started Brave using the following
    • brave.exe --enable-logging=stderr --variations-server-url=https://variations.bravesoftware.com/seed
  • ensured that PrettyPrintJSONDocument isn't visible under brave://version
Example Example Example
image image image
Example Example
image image

Release - PrettyPrintJSONDocument:Disabled_EmergencyKillSwitch

Verification PASSED on Win 11 x64 using the following build(s):

Brave | 1.58.135 Chromium: 117.0.5938.140 (Official Build) (64-bit)
-- | --
Revision | 325e8592cad8c6badb656cab238b722ae46bf7bc
OS | Windows 11 Version 22H2 (Build 22621.2361)
  • downloaded/installed 1.58.135 Chromium: 117.0.5938.140 and started Brave using the following
    • brave.exe --enable-logging=stderr --variations-server-url=https://variations.bravesoftware.com/seed
  • ensured that PrettyPrintJSONDocument isn't visible under brave://version
Example Example Example
image image image
Example Example
image image

Release - min_version (checking PrettyPrintJSONDocument not being set with C116)

Verification PASSED on Win 11 x64 using the following build(s):

Brave | 1.57.64 Chromium: 116.0.5845.188 (Official Build) (64-bit)
-- | --
Revision | 25c5f0154a27707e95614f4ae5c79975958e2d09
OS | Windows 11 Version 22H2 (Build 22621.2361)
  • downloaded/installed 1.57.64 Chromium: 116.0.5845.188 and started Brave using the following
    • brave.exe --enable-logging=stderr --variations-server-url=https://variations.bravesoftware.com/seed
  • ensured that PrettyPrintJSONDocument isn't visible under brave://version
Example Example
image image
  • restarted Brave using brave.exe --enable-logging=stderr --variations-server-url=https://variations.bravesoftware.com/seed
  • ensured that PrettyPrintJSONDocument:Disabled_EmergencyKillSwitch is not being set via brave://version

image

@kjozwiak
Copy link
Member

kjozwiak commented Oct 2, 2023

Quick Note: As of brave/brave-core#20244, the Griffin studies will be applied on first launch for Android.

Nightly - PrettyPrintJSONDocument:Disabled_EmergencyKillSwitch

Verification PASSED on Pixel 6 running Android 14 using the following build(s):

Brave | 1.60.74 Chromium: 118.0.5993.21 (Official Build) canary (64-bit)
--- | ---
Revision | 2167fe18e911a62325d2124823a7b4a32f3966a7
OS | Android 14; Build/U1B1.230908.003; 34; REL
Example Example
Screenshot_20231002-185034 Screenshot_20231002-185058

BETA - PrettyPrintJSONDocument:Disabled_EmergencyKillSwitch

Verification PASSED on Pixel 6 running Android 14 using the following build(s):

Brave | 1.59.107 Chromium: 117.0.5938.140 (Official Build) beta (64-bit)
--- | ---
Revision | 5bbd53ed3e41461e15555b7ce0490464656b236f
OS | Android 14; Build/U1B1.230908.003; 34; REL
Example Example
Screenshot_20231002-190954 Screenshot_20231002-191016

Release - PrettyPrintJSONDocument:Disabled_EmergencyKillSwitch

Verification PASSED on Pixel 6 running Android 14 using the following build(s):

Brave | 1.58.135 Chromium: 117.0.5938.140 (Official Build) (64-bit)
--- | ---
Revision | 325e8592cad8c6badb656cab238b722ae46bf7bc
OS | Android 14; Build/U1B1.230908.003; 34; REL
Example Example
Screenshot_20231002-191536 Screenshot_20231002-191601

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants