Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1289)

Unified Diff: printing/printed_document.h

Issue 7585015: Printing: Remove unused PageOverlay since header/footers are drawn in PrintWebViewHelper. (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: fix win Created 9 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « printing/print_settings.cc ('k') | printing/printed_document.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: printing/printed_document.h
===================================================================
--- printing/printed_document.h (revision 95638)
+++ printing/printed_document.h (working copy)
@@ -98,8 +98,6 @@
return immutable_.name_;
}
const GURL& url() const { return immutable_.url_; }
- const string16& date() const { return immutable_.date_; }
- const string16& time() const { return immutable_.time_; }
int cookie() const { return immutable_.cookie_; }
// Sets a path where to dump printing output files for debugging. If never set
@@ -154,9 +152,6 @@
int cookie);
~Immutable();
- // Sets the document's |date_| and |time_|.
- void SetDocumentDate();
-
// Print settings used to generate this document. Immutable.
PrintSettings settings_;
@@ -169,12 +164,6 @@
// URL that generated this document. Immutable.
GURL url_;
- // The date on which this job started. Immutable.
- string16 date_;
-
- // The time at which this job started. Immutable.
- string16 time_;
-
// Cookie to uniquely identify this document. It is used to make sure that a
// PrintedPage is correctly belonging to the PrintedDocument. Since
// PrintedPage generation is completely asynchronous, it could be easy to
@@ -184,23 +173,6 @@
int cookie_;
};
- // Prints the headers and footers for one page in the specified context
- // according to the current settings.
- void PrintHeaderFooter(gfx::NativeDrawingContext context,
- const PrintedPage& page,
- PageOverlays::HorizontalPosition x,
- PageOverlays::VerticalPosition y,
- const gfx::Font& font) const;
-
- // Draws the computed |text| into |context| taking into account the bounding
- // region |bounds|. |bounds| is the position in which to draw |text| and
- // the minimum area needed to contain |text| which may not be larger than the
- // header or footer itself.
- // TODO(jhawkins): string16.
- void DrawHeaderFooter(gfx::NativeDrawingContext context,
- std::wstring text,
- gfx::Rect bounds) const;
-
void DebugDump(const PrintedPage& page);
// All writable data member access must be guarded by this lock. Needs to be
« no previous file with comments | « printing/print_settings.cc ('k') | printing/printed_document.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698