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

(mac os 10.7.5) disable print preview/prevent crashes upon print #52

Closed
mrkapqa opened this issue Mar 15, 2022 · 13 comments
Closed

(mac os 10.7.5) disable print preview/prevent crashes upon print #52

mrkapqa opened this issue Mar 15, 2022 · 13 comments

Comments

@mrkapqa
Copy link

mrkapqa commented Mar 15, 2022

Hello,

thank you for your help with this software.

Could you please show how to disable print preview as the browser always crashes when trying to print a webpage, on mac osx 10.7.5.
i think it has to do with print preview, so id like to disable it.
otherwise software seem to function well.
downloaded straight from the releases page.

@Wowfunhappy
Copy link

Wowfunhappy commented Mar 15, 2022

Could you please show how to disable print preview as the browser always crashes when trying to print a webpage, on mac osx 10.7.5.

If this is indeed the cause of the crash, you can run:

defaults write org.chromium.Chromium DisablePrintPreview -bool true

in the Terminal and then restart Chromium to disable Chromium's print preview. (Note: this will cause Chromium to say it is "managed by your organization". This is just what happens when you set policies via the Terminal; no one can see your browsing history.)

@mrkapqa
Copy link
Author

mrkapqa commented Mar 15, 2022

thank you for your quick help,
greatly appreciated, and it worked.

please show also on how to ddisable the key confidential request at the start, as i choose "deny" and would like to keep it so.
Screen Shot 2022-03-15 at 5 08 35 PM

@Wowfunhappy
Copy link

Wowfunhappy commented Mar 15, 2022

I don't think you can choose Deny without it asking again. Note, "Chromium Safe Storage" is an application password created by Chromium for its own use, there's really no reason to deny access. (But good on you for being vigilant!)

@Wowfunhappy
Copy link

As of build 986492, Chromium's print preview does not cause Chromium to crash on OS X Lion, so I think this issue has been fixed. If it's working for you too @mrkapqa, you should be able to close this issue.

That said, if the Chromium print dialog ever causes issues again in the future, I'd personally recommend just disabling it for everyone. It's an unnecessary anti-feature.

@mrkapqa
Copy link
Author

mrkapqa commented Mar 30, 2022

thanks again,
i have Version 98.0.4758.109 (Official Build) (x86_64)
and noticed that the Preview is no more enabled by default? hence it would not crash?
on Firefox, it used to be that the way to "enable" print preview was by clicking on the toolbar button, and cmd + P would instead just open the print dialogue, without print preview.
a quite elegant approach, once you get aware of it (print is normally not in the toolbar, so one has to customize it first to see the "hidden" feature of print preview")
on chromium, however, the print preview seemed active since long, and never had problems with it, but i have to admit that i dont use it regularly;

if i had to choose, i would prefer to have it enabled, unless, it causes crashes, of course.
dont know if this is now supposed to be disabled by default, if yes,iwill check later again enabling it manually, and see if it still crashes or not, cause i ilike the feature and do a lot of pdf-printing; firefox is still my preferred browser, although i have to admit that chromium is better at rendering webpages for pdf printing, and quite by a margin; will see how this evolves.

@mrkapqa
Copy link
Author

mrkapqa commented Mar 30, 2022

ah well, i think on osx there is not even a print preview on firefox for mac, at least not on 10.7.5, just checked.
maybe then on newer versions like firefox 91 esr or similar

@blueboxd
Copy link
Owner

Hi, sorry for the late.

I haven't experienced crashes on print previews on 10.7 with both canary and stable builds.
Do you have crash logs?

@mrkapqa
Copy link
Author

mrkapqa commented Mar 31, 2022

hey,sorry,
i had upgraded to mavericks,
would consider this case closed.
thanks for the tip, might come in handy.

@mrkapqa
Copy link
Author

mrkapqa commented Mar 31, 2022

only remaining is ,
now i dont have print preview anymore (neither on lion had it, nor on mavericks 10.9.1 am currently on).
please show the proper way to enable it, thanks!
Screen Shot 2022-03-31 at 21 41 28
Screen Shot 2022-03-31 at 21 42 05

@Wowfunhappy
Copy link

Wowfunhappy commented Apr 1, 2022

If you disabled Chromium's print dialog via the defaults write command I posted previously, you have to undo that:

defaults delete org.chromium.Chromium DisablePrintPreview

@mrkapqa
Copy link
Author

mrkapqa commented Apr 1, 2022

thanks, dont know what happened, but after re-installing the OS X,
now on 10.9.5, the Print Preview on CHromium Legacy on my computer is gone;
i had disabled it in the previous installation, but on the new installation it never materialized, which is bit strange.

the print windows looks like this
Screen Shot 2022-04-01 at 08 59 08

@Wowfunhappy
Copy link

Huh. I don't know why that's happening, but fwiw, you can expand the "PDF" drop-down and click "Open PDF in Preview" in order to see a preview. (This is why I've never liked the Chromium dialog anyway; this functionality already exists in the OS, and the native version can be integrated with other Mac features such as Automator.)

@mrkapqa
Copy link
Author

mrkapqa commented Apr 1, 2022

it seems the line above
defaults write org.chromium.Chromium DisablePrintPreview -bool true
persisted through several new-installations (on the same Mac), which is quite telling.
now, after againevoking defaults delete org.chromium.Chromium DisablePrintPreview
on a new insllation of Mac OS Mountain lion it would finally restore it; withouth it, every chromium-legacy i tr (from releases, or via downloader.pane) would always just show the normal osx printing window;

@mrkapqa mrkapqa closed this as completed Apr 1, 2022
blueboxd pushed a commit that referenced this issue Jun 22, 2022
This should be a >= instead.

(cherry picked from commit 4989947)

Change-Id: I14814657607a3ab80ef529b4eec40e9b63ea67e6
Bug: 1325042
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3646330
Auto-Submit: Kramer Ge <fangzhoug@chromium.org>
Commit-Queue: Peter McNeeley <petermcneeley@chromium.org>
Commit-Queue: Maksim Sisov <msisov@igalia.com>
Commit-Queue: Kramer Ge <fangzhoug@chromium.org>
Reviewed-by: Peter McNeeley <petermcneeley@chromium.org>
Reviewed-by: Maksim Sisov <msisov@igalia.com>
Cr-Original-Commit-Position: refs/heads/main@{#1003185}
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3651106
Cr-Commit-Position: refs/branch-heads/5060@{#52}
Cr-Branched-From: b83393d-refs/heads/main@{#1002911}
blueboxd pushed a commit that referenced this issue Aug 31, 2022
Change-Id: I2f58795166c0ddc891aa935a363c1a5c9b97d24f
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3787001
Commit-Queue: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com>
Auto-Submit: Ben Mason <benmason@chromium.org>
Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com>
Cr-Commit-Position: refs/branch-heads/5195@{#52}
Cr-Branched-From: 7aa3f07-refs/heads/main@{#1027018}
blueboxd pushed a commit that referenced this issue Sep 29, 2022
…ettings""

This reverts commit ff9407b.

Reason for revert: The sign-in row is hidden when promos are shown if
none of SYNC_ANDROID_PROMOS_WITH_ILLUSTRATION, SYNC_ANDROID_PROMOS_WITH_SINGLE_BUTTON or SYNC_ANDROID_PROMOS_WITH_TITLE is enabled.
This cl uses one of these flags - SYNC_ANDROID_PROMOS_WITH_SINGLE_BUTTON - to fix the tests.

Original change's description:
> Revert "Reland "Add Tangible sync flow for signin row in main settings""
>
> This reverts commit df17791.
>
> Reason for revert: Failing bots for another reason
> https://bugs.chromium.org/p/chromium/issues/detail?id=1353505
>
> Original change's description:
> > Reland "Add Tangible sync flow for signin row in main settings"
> >
> > This reverts commit 7935edc.
> >
> > Reason for revert:
> > Fixes the tests. Sync promo uses the same string as the signin
> > preference when there are no accounts.
> > In order to fix the tests an account is added to the device so that
> > the promo has a different text.
> >
> > Original change's description:
> > > Revert "Add Tangible sync flow for signin row in main settings"
> > >
> > > This reverts commit 4cee937.
> > >
> > > Reason for revert: newly added tests fail on bots (see
> > > https://crbug.com/1353505).
> > >
> > > Original change's description:
> > > > Add Tangible sync flow for signin row in main settings
> > > >
> > > > This cl adds the tangible sync flow launch capability to the signin
> > > > preference in main settings page. This flow is guarded by a flag.
> > > >
> > > > Bug: 1332527
> > > > Change-Id: I9222fa0512e1fcd55a012a9fa37853a9b66b0226
> > > > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3819763
> > > > Reviewed-by: Boris Sazonov <bsazonov@chromium.org>
> > > > Commit-Queue: Tanmoy Mollik <triploblastic@chromium.org>
> > > > Cr-Commit-Position: refs/heads/main@{#1035435}
> > >
> > > Bug: 1332527, 1353505
> > > Change-Id: I6e82abcd635da0999b0c7752c77d0ef08986f193
> > > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3834451
> > > Commit-Queue: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com>
> > > Auto-Submit: Boris Sazonov <bsazonov@chromium.org>
> > > Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com>
> > > Cr-Commit-Position: refs/heads/main@{#1035761}
> >
> > Bug: 1332527, 1353505
> > Change-Id: I387ba7a5c31e28a8f82726af53ac68c76ced6487
> > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3834251
> > Reviewed-by: Boris Sazonov <bsazonov@chromium.org>
> > Commit-Queue: Tanmoy Mollik <triploblastic@chromium.org>
> > Cr-Commit-Position: refs/heads/main@{#1036573}
>
> Bug: 1332527, 1353505
> Change-Id: Icd2b88b2d67bf945edc006e09629633a75680541
> No-Presubmit: true
> No-Tree-Checks: true
> No-Try: true
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3838475
> Commit-Queue: Lijin Shen <lazzzis@google.com>
> Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com>
> Owners-Override: Lijin Shen <lazzzis@google.com>
> Cr-Commit-Position: refs/heads/main@{#1036813}

(cherry picked from commit 0955f0f)

Bug: 1332527, 1353505
Change-Id: I0ba69c3047a91f01ac508d277a1c30797cd9a97e
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3845610
Reviewed-by: Boris Sazonov <bsazonov@chromium.org>
Commit-Queue: Tanmoy Mollik <triploblastic@chromium.org>
Cr-Original-Commit-Position: refs/heads/main@{#1037803}
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3851522
Auto-Submit: Boris Sazonov <bsazonov@chromium.org>
Commit-Queue: Boris Sazonov <bsazonov@chromium.org>
Reviewed-by: Tanmoy Mollik <triploblastic@chromium.org>
Cr-Commit-Position: refs/branch-heads/5249@{#52}
Cr-Branched-From: 4f7bea5-refs/heads/main@{#1036826}
blueboxd pushed a commit that referenced this issue Nov 30, 2022
Before: http://screenshot/38xrkoTxmajvjcC
After: http://screenshot/3dFk2MfMiMUQ8wH

(cherry picked from commit f5c576e)

Bug: 1372278
Change-Id: Iad55821aff686474db7f9e73b4538b4d94c2f9c5
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3951131
Reviewed-by: Tommy Li <tommycli@chromium.org>
Commit-Queue: Marlon Facey <mfacey@chromium.org>
Cr-Original-Commit-Position: refs/heads/main@{#1058988}
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3962268
Commit-Queue: Tommy Li <tommycli@chromium.org>
Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com>
Auto-Submit: Marlon Facey <mfacey@chromium.org>
Cr-Commit-Position: refs/branch-heads/5359@{#52}
Cr-Branched-From: 27d3765-refs/heads/main@{#1058933}
blueboxd pushed a commit that referenced this issue Feb 8, 2023
Change-Id: I8250a3709cb72b43e5ed48c876ba62b5f72688c9
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4119183
Bot-Commit: Chrome Release Bot (LUCI) <chrome-official-brancher@chops-service-accounts.iam.gserviceaccount.com>
Cr-Commit-Position: refs/branch-heads/5481@{#52}
Cr-Branched-From: 130f3e4-refs/heads/main@{#1084008}
blueboxd pushed a commit that referenced this issue Mar 7, 2023
…ark-111.0.5563.5-r1 to 111-5550.0-1675083099-benchmark-111.0.5563.7-r1

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/cros-afdo-bigcore-chromium-beta
Please CC c-compiler-chrome@google.com on the revert to ensure that a human
is aware of the problem.

To file a bug in Chromium Beta Branch: https://bugs.chromium.org/p/chromium/issues/entry

To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Tbr: c-compiler-chrome@google.com
Change-Id: Ie89fb110d227435a44ab84782f78f85ac414119e
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4209289
Commit-Queue: Chrome Release Autoroll <chromium-release-autoroll@skia-public.iam.gserviceaccount.com>
Bot-Commit: Chrome Release Autoroll <chromium-release-autoroll@skia-public.iam.gserviceaccount.com>
Cr-Commit-Position: refs/branch-heads/5563@{#52}
Cr-Branched-From: 3ac59a6-refs/heads/main@{#1097615}
blueboxd pushed a commit that referenced this issue Apr 14, 2023
Re-adds ripple effect to all touchable surfaces. Fixes the previous
issues of background re-usage by using a custom RippleDrawable that
prevents sharing.

(cherry picked from commit 03226c2)

Bug: 1334642,1419775
Change-Id: I7ed24e19566494aee0c1eacb6b9f035b2e0ccba7
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4288774
Commit-Queue: Wolfgang Billenstein <bwolfgang@google.com>
Reviewed-by: Norge Vizcay <vizcay@google.com>
Cr-Original-Commit-Position: refs/heads/main@{#1110200}
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4295278
Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com>
Cr-Commit-Position: refs/branch-heads/5615@{#52}
Cr-Branched-From: 9c6408e-refs/heads/main@{#1109224}
blueboxd pushed a commit that referenced this issue May 9, 2023
This changes the upper bound on items to show from 100 to 30. This
reduces the impact of possible O(n^2) behavior in showing the bubble,
which may be causing performance issues.

Because caching fewer items leads to backfilling more frequently,
which we want to avoid, the number of extra items to cache is
raised accordingly, so that we cache 60 items of each type
(previously 120). Caching more items is relatively cheap while
backfilling is expensive.

(cherry picked from commit a7cf2bd)

Bug: 1421426
Change-Id: I1b69f8971c3ed7c866b1219427fdd023a38459e7
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4370538
Commit-Queue: Lily Chen <chlily@chromium.org>
Reviewed-by: Xinghui Lu <xinghuilu@chromium.org>
Cr-Original-Commit-Position: refs/heads/main@{#1121772}
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4374859
Cr-Commit-Position: refs/branch-heads/5672@{#52}
Cr-Branched-From: 5f2a724-refs/heads/main@{#1121455}
blueboxd pushed a commit that referenced this issue May 27, 2023
Log the number of inactive tabs by arm definition and the number of
absolute inactive tab (older than 21 days) and the number of active tab
by arm definition.

(cherry picked from commit 31c2e8f)

Fixed: 1440123
Change-Id: I6c782535e22af450d366e8b422a736c26b2f8a01
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4459611
Reviewed-by: Olivier Robin <olivierrobin@chromium.org>
Commit-Queue: Aliona Dangla <alionadangla@chromium.org>
Reviewed-by: Sylvain Defresne <sdefresne@chromium.org>
Reviewed-by: Louis Romero <lpromero@google.com>
Reviewed-by: Dana Fried <dfried@chromium.org>
Code-Coverage: Findit <findit-for-me@appspot.gserviceaccount.com>
Cr-Original-Commit-Position: refs/heads/main@{#1135950}
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4482840
Reviewed-by: Mark Cogan <marq@chromium.org>
Commit-Queue: Mark Cogan <marq@chromium.org>
Auto-Submit: Aliona Dangla <alionadangla@chromium.org>
Cr-Commit-Position: refs/branch-heads/5735@{#52}
Cr-Branched-From: 2f562e4-refs/heads/main@{#1135570}
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

No branches or pull requests

3 participants