Comment #17 on issue 947 by bugdro...@chromium.org: Printer Setup missing http://code.google.com/p/chromium/issues/detail?id=947
The following revision refers to this bug: http://src.chromium.org/viewvc/chrome?view=rev&revision=13426 ------------------------------------------------------------------------ r13426 | mar...@chromium.org | 2009-04-09 09:36:44 -0700 (Thu, 09 Apr 2009) | 52 lines Changed paths: M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/printing/page_overlays.cc?r1=13426&r2=13425 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/printing/page_overlays.h?r1=13426&r2=13425 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/printing/page_overlays_unittest.cc?r1=13426&r2=13425 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/common/pref_names.cc?r1=13426&r2=13425 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/common/pref_names.h?r1=13426&r2=13425 Implement Printer Preference Persistence (Step 1/3) Introduce new "Preferences" for printer overlays were added in PrefNames: - printing.page.header.left - printing.page.header.center - printing.page.header.right - printing.page.footer.left - printing.page.footer.center - printing.page.footer.right The prefs file will look like this: "printing": { "page": { "header": { "left": " ", "center": " ", "right": " " }, "footer": { "left": "{date}", "center": " ", "right": " " } } } The steps for Printer Preferences: Step 1: Introduce some sort of persistence for header/footer which is located in the Preference file. Step 2: Allow PrinterQuery to read those stored preferences and setup PrinterSettings so that it will apply it when printing. Step 3: Create a UI for the print preference, so that the users will easily change those printing settings. NOTE: I tested the following approach while doing a simple Hack with PrintJob (if you see the previous patch), and it works great, with some bad after effect. It was done for testing purposes. So I hope we are going in the right direction. BUG=947 (http://crbug.com/947) Patch contributed by Mohamed Mansour <m0.interact...@gmail.com> Review: http://codereview.chromium.org/60118/show ------------------------------------------------------------------------ -- You received this message because you are listed in the owner or CC fields of this issue, or because you starred this issue. You may adjust your issue notification preferences at: http://code.google.com/hosting/settings --~--~---------~--~----~------------~-------~--~----~ Automated mail from issue updates at http://crbug.com/ Subscription options: http://groups.google.com/group/chromium-bugs -~----------~----~----~----~------~----~------~--~---