summaryrefslogtreecommitdiffstats
path: root/printing
Commit message (Collapse)AuthorAgeFilesLines
* Cleanup: Remove unused PrintedDocument::RenderPrintedPageNumber().thestig@chromium.org2010-12-172-15/+0
| | | | | | | | BUG=none TEST=none Review URL: http://codereview.chromium.org/5948002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@69522 0039d316-1c4b-4281-b951-d872f2087c98
* Making CUPS calls non-blocking.gene@google.com2010-12-163-7/+20
| | | | | | | | | BUG=none TEST=Verify proxy is not blocked on CUPS calls. Review URL: http://codereview.chromium.org/5846001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@69357 0039d316-1c4b-4281-b951-d872f2087c98
* Make members of Singleton<T> private and only visible to the singleton type. ↵satish@chromium.org2010-12-142-9/+13
| | | | | | | | | | | | | | | This enforces that the Singleton<T> pattern can only be used within classes which want singleton-ness. As part of this CL I have also fixed up files which got missed in my previous CLs to use a GetInstance() method and use Singleton<T> from the source file. There are a small number of places where I have also switched to LazyInstance as that was more appropriate for types used in a single source file. BUG=65298 TEST=all existing tests should continue to pass. Review URL: http://codereview.chromium.org/5682008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@69107 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 68932 - Make members of Singleton<T> private and only visible to the ↵satish@chromium.org2010-12-112-13/+9
| | | | | | | | | | | | | | | | | | singleton type. This enforces that the Singleton<T> pattern can only be used within classes which want singleton-ness. As part of this CL I have also fixed up files which got missed in my previous CLs to use a GetInstance() method and use Singleton<T> from the source file. There are a small number of places where I have also switched to LazyInstance as that was more appropriate for types used in a single source file. BUG=65298 TEST=all existing tests should continue to pass. Review URL: http://codereview.chromium.org/5682008 TBR=satish@chromium.org Review URL: http://codereview.chromium.org/5721005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68936 0039d316-1c4b-4281-b951-d872f2087c98
* Make members of Singleton<T> private and only visible to the singleton type. ↵satish@chromium.org2010-12-112-9/+13
| | | | | | | | | | | | | | | This enforces that the Singleton<T> pattern can only be used within classes which want singleton-ness. As part of this CL I have also fixed up files which got missed in my previous CLs to use a GetInstance() method and use Singleton<T> from the source file. There are a small number of places where I have also switched to LazyInstance as that was more appropriate for types used in a single source file. BUG=65298 TEST=all existing tests should continue to pass. Review URL: http://codereview.chromium.org/5682008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68932 0039d316-1c4b-4281-b951-d872f2087c98
* cupsGetPPD got stuck sometime for an infinite time due to network ↵gene@google.com2010-12-063-2/+37
| | | | | | | | | | | | | configuration/issues. Make http connection for cupsGetPPD non-blocking to prevent that. BUG=none TEST=Make sure GettPPD does not get stuck. Review URL: http://codereview.chromium.org/5609004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68356 0039d316-1c4b-4281-b951-d872f2087c98
* On Chrome OS, use the application locale to determine the default paper size ↵sanjeevr@chromium.org2010-12-0310-22/+76
| | | | | | | | | | | rather than rely on GTK. BUG=None. TEST=Web pages printed from Chrome OS should use locale-specific default paper size (specifically, for US, it should use 8.5 x 11). Review URL: http://codereview.chromium.org/5636001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68233 0039d316-1c4b-4281-b951-d872f2087c98
* Add a check to help prevent PrintingContextTest flakiness.thestig@chromium.org2010-12-011-5/+10
| | | | | | | | BUG=61509 TEST=Less flaky printing tests. Review URL: http://codereview.chromium.org/5406005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@67802 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 67662 - FBTF: Remove unneeded headers from base/ (part 10)thestig@chromium.org2010-11-304-15/+9
| | | | | | | | | | | BUG=none TEST=none Review URL: http://codereview.chromium.org/5139006 TBR=thestig@chromium.org Review URL: http://codereview.chromium.org/5270010 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@67674 0039d316-1c4b-4281-b951-d872f2087c98
* FBTF: Remove unneeded headers from base/ (part 10)thestig@chromium.org2010-11-304-9/+15
| | | | | | | | BUG=none TEST=none Review URL: http://codereview.chromium.org/5139006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@67662 0039d316-1c4b-4281-b951-d872f2087c98
* Reset AssertHandler in IPCMessageTest.Metafile.sanjeevr@chromium.org2010-11-121-1/+0
| | | | | | | | | BUG=55177 TEST=Unit tests Review URL: http://codereview.chromium.org/4660011 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@65972 0039d316-1c4b-4281-b951-d872f2087c98
* Used GetProcAddress to load exported functions from prntvpt.dll instead of ↵sanjeevr@chromium.org2010-11-103-15/+224
| | | | | | | | | | | using DELAYLOAD. BUG=62421 TEST=ui_tests on XP without the XPS Framework installed should run. Review URL: http://codereview.chromium.org/4637004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@65678 0039d316-1c4b-4281-b951-d872f2087c98
* Fix DCHECK failure in page_setup.cc on Linux.hamaji@chromium.org2010-11-101-4/+8
| | | | | | | | | | | | | | I think this issue was introduced in http://codereview.chromium.org/3619002 We need to subtract margins from page size to calculate the size of printable areas. BUG=none TEST=none Review URL: http://codereview.chromium.org/4732002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@65635 0039d316-1c4b-4281-b951-d872f2087c98
* Disable PrintingContextTest.Base on winnirnimesh@chromium.org2010-11-011-1/+2
| | | | | | | | | | | | PrintingContextTest.Base fails on win full release builder BUG=61509 TEST= Review URL: http://codereview.chromium.org/4264001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@64684 0039d316-1c4b-4281-b951-d872f2087c98
* Mark PrintingContextTest.PrintAll as flaky as it fails on Win builder.amit@chromium.org2010-11-011-1/+2
| | | | | | | | | | BUG=61499 TEST=PrintingContextTest.PrintAll TBR=maruel Review URL: http://codereview.chromium.org/4239001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@64667 0039d316-1c4b-4281-b951-d872f2087c98
* Change LOG(INFO) to VLOG(1).gene@google.com2010-10-291-2/+2
| | | | | | | | | BUG=none TEST=none Review URL: http://codereview.chromium.org/4105007 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@64491 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 64454 - Revert 64441 - Mac cloud print proxy work.scottbyer@google.com2010-10-292-6/+22
| | | | | | | | | | | | | | | | | | | Turn on use_cups for the Mac, add in the required library, and tweak the code to get it linking. There is still more work to do; this is just a gyp-oriented change to get things started. BUG=none TEST=All targets still build on all platforms. Review URL: http://codereview.chromium.org/4167006 TBR=scottbyer@chromium.org Review URL: http://codereview.chromium.org/4136011 TBR=scottbyer@google.com Review URL: http://codereview.chromium.org/4137013 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@64477 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 64441 - Mac cloud print proxy work.scottbyer@google.com2010-10-292-22/+6
| | | | | | | | | | | | | | | | Turn on use_cups for the Mac, add in the required library, and tweak the code to get it linking. There is still more work to do; this is just a gyp-oriented change to get things started. BUG=none TEST=All targets still build on all platforms. Review URL: http://codereview.chromium.org/4167006 TBR=scottbyer@chromium.org Review URL: http://codereview.chromium.org/4136011 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@64454 0039d316-1c4b-4281-b951-d872f2087c98
* Mac cloud print proxy work.scottbyer@chromium.org2010-10-292-6/+22
| | | | | | | | | | | | | Turn on use_cups for the Mac, add in the required library, and tweak the code to get it linking. There is still more work to do; this is just a gyp-oriented change to get things started. BUG=none TEST=All targets still build on all platforms. Review URL: http://codereview.chromium.org/4167006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@64441 0039d316-1c4b-4281-b951-d872f2087c98
* Convert ElideUrl to return a string16 instead of a wstring.tony@chromium.org2010-10-271-1/+2
| | | | | | | | BUG=23581 Review URL: http://codereview.chromium.org/4115003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@64156 0039d316-1c4b-4281-b951-d872f2087c98
* Printing: Refactor PrintSettings::Init into a PrintSettingsFactory classjhawkins@chromium.org2010-10-2614-191/+339
| | | | | | | | | | | per-platform. BUG=none TEST=none Review URL: http://codereview.chromium.org/4040001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@63907 0039d316-1c4b-4281-b951-d872f2087c98
* Move useful printing backend code from chrome/service/cloud_print to ↵thestig@chromium.org2010-10-2512-11/+687
| | | | | | | | | | printing/backend. BUG=none TEST=none Review URL: http://codereview.chromium.org/3945003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@63772 0039d316-1c4b-4281-b951-d872f2087c98
* Convert gfx::ElideText from using wstrings to string16.tony@chromium.org2010-10-211-1/+2
| | | | | | | | BUG=23581 Review URL: http://codereview.chromium.org/3885003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@63433 0039d316-1c4b-4281-b951-d872f2087c98
* Printing: Split off Win/Mac implementations of the Image class into their ownjhawkins@chromium.org2010-10-215-114/+151
| | | | | | | | | | | files. BUG=none TEST=none Review URL: http://codereview.chromium.org/3966002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@63316 0039d316-1c4b-4281-b951-d872f2087c98
* Move scoped_cftyperef from base to base/mac, use the new namespace, and name itbrettw@chromium.org2010-10-175-14/+15
| | | | | | | | | | properly (scoped_cftyperef -> ScopedCFTypeRef). TEST=it compiles BUG=none Review URL: http://codereview.chromium.org/3855001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62887 0039d316-1c4b-4281-b951-d872f2087c98
* Move Stats, histograms, and field trial into a metrics subdirectory of base andbrettw@chromium.org2010-10-141-1/+1
| | | | | | | | | put them in the base namespace. TEST=it compiles BUG=none git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62510 0039d316-1c4b-4281-b951-d872f2087c98
* Used ScopedTempDir dir for the FileBackedDC unit test.sanjeevr@chromium.org2010-10-081-1/+6
| | | | | | | | | BUG=None TEST=Printing unittests. Review URL: http://codereview.chromium.org/3535011 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62013 0039d316-1c4b-4281-b951-d872f2087c98
* Printing: Refactor a few helper methods implemented per-platform to clean upjhawkins@chromium.org2010-10-077-40/+78
| | | | | | | | | | | ifdef mess in PrintedDocument implementation. BUG=none TEST=none Review URL: http://codereview.chromium.org/3564016 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@61865 0039d316-1c4b-4281-b951-d872f2087c98
* Printing: Standardize on Metafile::Init, removing Emf::CreateFromData in thejhawkins@chromium.org2010-10-076-22/+15
| | | | | | | | | | | process and any ifdefs needed for this. BUG=none TEST=none Review URL: http://codereview.chromium.org/3595016 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@61834 0039d316-1c4b-4281-b951-d872f2087c98
* Printing: Convert PrintingContext into an interface implemented by the separatejhawkins@chromium.org2010-10-0612-246/+400
| | | | | | | | | | | platforms. BUG=none TEST=none Review URL: http://codereview.chromium.org/3610013 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@61714 0039d316-1c4b-4281-b951-d872f2087c98
* Printing: Temporary fix on Linux using dummy print settings until we get the ↵thestig@chromium.org2010-10-052-17/+41
| | | | | | | | | | entire Linux printing pipeline to match Win/Mac. BUG=none TEST=printing works. Review URL: http://codereview.chromium.org/3619002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@61556 0039d316-1c4b-4281-b951-d872f2087c98
* Reapply r61237: FBTF: Another big ctor/dtor cleanup found by automated tools.erg@google.com2010-10-042-0/+4
| | | | | | | | | | | | | | | Removes changes to code in webkit/ that broke chrome_frame. Will debug that portion later. (Shaves ~2MB off Linux Debug .a files) BUG=none TEST=compiles First Review URL: http://codereview.chromium.org/3563004 Review URL: http://codereview.chromium.org/3621003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@61435 0039d316-1c4b-4281-b951-d872f2087c98
* Revert "FBTF: Another big ctor/dtor cleanup found by automated tools."erg@google.com2010-10-012-4/+0
| | | | | | | | | | | This reverts commit 27ea47d65cf8767f350113d5ad9e25170efde811 (r61237). BUG=none TEST=compiles Review URL: http://codereview.chromium.org/3609005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@61240 0039d316-1c4b-4281-b951-d872f2087c98
* FBTF: Another big ctor/dtor cleanup found by automated tools.erg@google.com2010-10-012-0/+4
| | | | | | | | | | | (Shaves ~2MB off Linux Debug .a files) BUG=none TEST=compiles Review URL: http://codereview.chromium.org/3563004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@61237 0039d316-1c4b-4281-b951-d872f2087c98
* FBTF: Move ctors/dtors into implementation files. Adds ctors/dtors to ↵erg@google.com2010-09-304-1/+13
| | | | | | | | | | | | | | non-POD structs. Cuts ~2MB off our .a files (Debug, Linux). Also added the "virtual" keyword on a whole bunch of virtual dtors that were missing it. BUG=none TEST=compiles Review URL: http://codereview.chromium.org/3522004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@61100 0039d316-1c4b-4281-b951-d872f2087c98
* Added methods to create and read from file-backed metafiles to the Windows ↵sanjeevr@chromium.org2010-09-293-2/+62
| | | | | | | | | | | Emf class. BUG=None TEST=Unittests Review URL: http://codereview.chromium.org/3538003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@60994 0039d316-1c4b-4281-b951-d872f2087c98
* Fix Mac print crashgfeher@chromium.org2010-09-291-4/+4
| | | | | | | | | | | Stop executing the "OK branch" when the user cancels a print on Mac. BUG=57125 TEST=manual, see bug description Review URL: http://codereview.chromium.org/3509009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@60914 0039d316-1c4b-4281-b951-d872f2087c98
* Fix a c_str() use-after-free bug in the printing code.nick@chromium.org2010-09-281-1/+2
| | | | | | | | | BUG= TEST= Review URL: http://codereview.chromium.org/3454022 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@60735 0039d316-1c4b-4281-b951-d872f2087c98
* Printing: Remove the debug-only |page_number_| from PrintingContext, which hasjhawkins@chromium.org2010-09-284-47/+0
| | | | | | | | | | | outlived its usefulness. BUG=none TEST=none Review URL: http://codereview.chromium.org/3417025 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@60731 0039d316-1c4b-4281-b951-d872f2087c98
* Printing: Push checking of |context| one level lower, as the differentjhawkins@chromium.org2010-09-271-0/+1
| | | | | | | | | | | platforms have different expectations at this level. BUG=none TEST=none Review URL: http://codereview.chromium.org/3418034 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@60728 0039d316-1c4b-4281-b951-d872f2087c98
* Printing: Push checking of |context| one level lower, as the differentjhawkins@chromium.org2010-09-274-0/+8
| | | | | | | | | | | platforms have different expectations at this level. BUG=none TEST=none Review URL: http://codereview.chromium.org/3475021 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@60704 0039d316-1c4b-4281-b951-d872f2087c98
* Printing: Remove OS_WIN ifdef in a win-only file. Remove an unused debuggingjhawkins@chromium.org2010-09-271-28/+0
| | | | | | | | | | | block. BUG=none TEST=none Review URL: http://codereview.chromium.org/3474014 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@60694 0039d316-1c4b-4281-b951-d872f2087c98
* Printing: Implement PrintingContext::UseDefaultSettings for Linux/Cairo.jhawkins@chromium.org2010-09-218-24/+115
| | | | | | | | | BUG=none TEST=none Review URL: http://codereview.chromium.org/3389005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@60087 0039d316-1c4b-4281-b951-d872f2087c98
* Printing: Convert several wstrings to string16.jhawkins@chromium.org2010-09-139-48/+49
| | | | | | | | | BUG=none TEST=none Review URL: http://codereview.chromium.org/3292019 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@59262 0039d316-1c4b-4281-b951-d872f2087c98
* Removing emf record constructor that is not used.finnur@chromium.org2010-08-312-7/+2
| | | | | | | | | | BUG=None TEST=None CID=5008 Review URL: http://codereview.chromium.org/3106005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@57997 0039d316-1c4b-4281-b951-d872f2087c98
* FBTF: Remove unneeded headers from base/ (part 7)thestig@chromium.org2010-08-271-1/+4
| | | | | | | | BUG=none TEST=none Review URL: http://codereview.chromium.org/3176026 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@57737 0039d316-1c4b-4281-b951-d872f2087c98
* Re-land part of r55416, minus actually moving the test_suite.h contents to a ↵brettw@chromium.org2010-08-171-1/+1
| | | | | | | | | | | | | | | | | | | | | | | .cc file. Cleanup in base. This moves the implementation (and a bunch of header file dependencies) from the multiprocess test and the test_suite headers to .cc files. Moves multiprocess_test to the test directory, and all of this stuff to the existing base_test_support project. I also used the base namespace. Previously other projects included this functionality just by #include because it was all inline, so I had to add dependencies on base_test_support in a few places. Moves and renames the command line switch this was using to base_switches. Move the base switch for process type to chrome switches. TEST=none BUG=none Review URL: http://codereview.chromium.org/3026055 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@56399 0039d316-1c4b-4281-b951-d872f2087c98
* Reapplies all the IPC system work (reverts the revert r56272).erg@google.com2010-08-172-0/+7
| | | | | | | | | | | That patch wasn't what caused the regression in the page cycler. BUG=51411,52103 TEST=still compiles Review URL: http://codereview.chromium.org/3106018 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@56390 0039d316-1c4b-4281-b951-d872f2087c98
* Completely revert all my IPC work to see if this was what regressed the page ↵erg@google.com2010-08-172-7/+0
| | | | | | | | | | | cycler. BUG=51411,52103 TEST=page cycler Review URL: http://codereview.chromium.org/3170020 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@56272 0039d316-1c4b-4281-b951-d872f2087c98
* Re-land just the project file changes from r55416. This makes projects that usebrettw@chromium.org2010-08-151-0/+1
| | | | | | | | | base test files dependent on test_support_base. TEST=none BUG=none git-svn-id: svn://svn.chromium.org/chrome/trunk/src@56172 0039d316-1c4b-4281-b951-d872f2087c98