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
-~----------~----~----~----~------~----~------~--~---

Reply via email to