summaryrefslogtreecommitdiffstats
path: root/printing
diff options
context:
space:
mode:
authordcheng <dcheng@chromium.org>2016-03-02 15:20:34 -0800
committerCommit bot <commit-bot@chromium.org>2016-03-02 23:21:51 +0000
commit70c494273fa71d922405cba4dc39f5457652a2db (patch)
tree9285f262fe33d8a3d4d9f36f8cd8244caa568ef7 /printing
parent063be11909af7bd64a575259b5ff01eb0686e287 (diff)
downloadchromium_src-70c494273fa71d922405cba4dc39f5457652a2db.zip
chromium_src-70c494273fa71d922405cba4dc39f5457652a2db.tar.gz
chromium_src-70c494273fa71d922405cba4dc39f5457652a2db.tar.bz2
Convert Pass()→std::move() on Windows
Also cleaned up a few stragglers from grepping the source: it's unclear if these are just building with weird defines, don't build at all, only build on obscure platforms, or something else… either way, Pass() itself will be removed shortly after this. BUG=557422 Review URL: https://codereview.chromium.org/1752233002 Cr-Commit-Position: refs/heads/master@{#378872}
Diffstat (limited to 'printing')
-rw-r--r--printing/backend/win_helper.cc26
-rw-r--r--printing/emf_win.cc4
2 files changed, 15 insertions, 15 deletions
diff --git a/printing/backend/win_helper.cc b/printing/backend/win_helper.cc
index ef775e4..eed998e 100644
--- a/printing/backend/win_helper.cc
+++ b/printing/backend/win_helper.cc
@@ -395,17 +395,17 @@ scoped_ptr<DEVMODE, base::FreeDeleter> XpsTicketToDevMode(
printing::ScopedXPSInitializer xps_initializer;
if (!xps_initializer.initialized()) {
// TODO(sanjeevr): Handle legacy proxy case (with no prntvpt.dll)
- return dev_mode.Pass();
+ return dev_mode;
}
printing::ScopedPrinterHandle printer;
if (!printer.OpenPrinter(printer_name.c_str()))
- return dev_mode.Pass();
+ return dev_mode;
base::win::ScopedComPtr<IStream> pt_stream;
HRESULT hr = StreamFromPrintTicket(print_ticket, pt_stream.Receive());
if (FAILED(hr))
- return dev_mode.Pass();
+ return dev_mode;
HPTPROVIDER provider = NULL;
hr = printing::XPSModule::OpenProvider(printer_name, 1, &provider);
@@ -424,7 +424,7 @@ scoped_ptr<DEVMODE, base::FreeDeleter> XpsTicketToDevMode(
}
printing::XPSModule::CloseProvider(provider);
}
- return dev_mode.Pass();
+ return dev_mode;
}
scoped_ptr<DEVMODE, base::FreeDeleter> CreateDevModeWithColor(
@@ -434,11 +434,11 @@ scoped_ptr<DEVMODE, base::FreeDeleter> CreateDevModeWithColor(
scoped_ptr<DEVMODE, base::FreeDeleter> default_ticket =
CreateDevMode(printer, NULL);
if (!default_ticket)
- return default_ticket.Pass();
+ return default_ticket;
if ((default_ticket->dmFields & DM_COLOR) &&
((default_ticket->dmColor == DMCOLOR_COLOR) == color)) {
- return default_ticket.Pass();
+ return default_ticket;
}
default_ticket->dmFields |= DM_COLOR;
@@ -446,27 +446,27 @@ scoped_ptr<DEVMODE, base::FreeDeleter> CreateDevModeWithColor(
DriverInfo6 info_6;
if (!info_6.Init(printer))
- return default_ticket.Pass();
+ return default_ticket;
const DRIVER_INFO_6* p = info_6.get();
// Only HP known to have issues.
if (!p->pszMfgName || wcscmp(p->pszMfgName, L"HP") != 0)
- return default_ticket.Pass();
+ return default_ticket;
// Need XPS for this workaround.
printing::ScopedXPSInitializer xps_initializer;
if (!xps_initializer.initialized())
- return default_ticket.Pass();
+ return default_ticket;
const char* xps_color = color ? kXpsTicketColor : kXpsTicketMonochrome;
std::string xps_ticket = base::StringPrintf(kXpsTicketTemplate, xps_color);
scoped_ptr<DEVMODE, base::FreeDeleter> ticket =
printing::XpsTicketToDevMode(printer_name, xps_ticket);
if (!ticket)
- return default_ticket.Pass();
+ return default_ticket;
- return ticket.Pass();
+ return ticket;
}
scoped_ptr<DEVMODE, base::FreeDeleter> CreateDevMode(HANDLE printer,
@@ -491,7 +491,7 @@ scoped_ptr<DEVMODE, base::FreeDeleter> CreateDevMode(HANDLE printer,
int size = out->dmSize;
int extra_size = out->dmDriverExtra;
CHECK_GE(buffer_size, size + extra_size);
- return out.Pass();
+ return out;
}
scoped_ptr<DEVMODE, base::FreeDeleter> PromptDevMode(
@@ -530,7 +530,7 @@ scoped_ptr<DEVMODE, base::FreeDeleter> PromptDevMode(
int size = out->dmSize;
int extra_size = out->dmDriverExtra;
CHECK_GE(buffer_size, size + extra_size);
- return out.Pass();
+ return out;
}
} // namespace printing
diff --git a/printing/emf_win.cc b/printing/emf_win.cc
index 3d54384..2e8df76 100644
--- a/printing/emf_win.cc
+++ b/printing/emf_win.cc
@@ -569,7 +569,7 @@ scoped_ptr<Emf> Emf::RasterizeMetafile(int raster_area_in_pixels) const {
result->FinishPage();
result->FinishDocument();
- return result.Pass();
+ return result;
}
scoped_ptr<Emf> Emf::RasterizeAlphaBlend() const {
@@ -602,7 +602,7 @@ scoped_ptr<Emf> Emf::RasterizeAlphaBlend() const {
result->FinishDocument();
- return result.Pass();
+ return result;
}