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

Unified Diff: printing/pdf_render_settings.h

Issue 821453003: Update legacy Tuple-using code. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: media Created 6 years 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
Index: printing/pdf_render_settings.h
diff --git a/printing/pdf_render_settings.h b/printing/pdf_render_settings.h
index 9e8f9301607488240b4d435baa89d15d15cb9317..1de192d1555f8a59e3f557bd24693fe1613f0eef 100644
--- a/printing/pdf_render_settings.h
+++ b/printing/pdf_render_settings.h
@@ -16,7 +16,7 @@ namespace printing {
// gfx::Rect - render area
// int - render dpi
// bool - autorotate pages to fit paper
-typedef Tuple3<gfx::Rect, int, bool> PdfRenderSettingsBase;
+typedef Tuple<gfx::Rect, int, bool> PdfRenderSettingsBase;
class PdfRenderSettings : public PdfRenderSettingsBase {
public:
@@ -25,9 +25,9 @@ class PdfRenderSettings : public PdfRenderSettingsBase {
: PdfRenderSettingsBase(area, dpi, autorotate) {}
~PdfRenderSettings() {}
- const gfx::Rect& area() const { return a; }
- int dpi() const { return b; }
- bool autorotate() const { return c; }
+ const gfx::Rect& area() const { return ::get<0>(*this); }
Nico 2014/12/22 17:03:12 Out of interest, why do you need the ::?
Avi (use Gerrit) 2014/12/22 18:03:02 Because "get" is used for the name of lots of memb
+ int dpi() const { return ::get<1>(*this); }
+ bool autorotate() const { return ::get<2>(*this); }
};
} // namespace printing

Powered by Google App Engine
This is Rietveld 408576698