IppParserService: ConvertIppToMojo usage fix

ipp_converter::ConvertIppToMojo does not take ownership of the passed in
ipp_t*; this change reflects that.

Bug: None
Test: unitests still pass
Change-Id: I19cbdb084175a57ab533cd63099d54acae624c2b
Reviewed-on: https://chromium-review.googlesource.com/c/1493206Reviewed-by: Sean Kau <skau@chromium.org>
Commit-Queue: Luum Habtemariam <luum@chromium.org>
Cr-Commit-Position: refs/heads/master@{#636478}

chrome/services/cups_ipp_parser/ipp_parser.cc

@@ -127,7 +127,7 @@ mojom::IppMessagePtr ExtractIppMessage(base::StringPiece request) {
     return nullptr;
   }
 
-  return ipp_converter::ConvertIppToMojo(ipp.release());
+  return ipp_converter::ConvertIppToMojo(ipp.get());
 }
 
 // Parse IPP request's |ipp_data|

chrome/services/cups_ipp_parser/public/cpp/ipp_converter.h

@@ -85,6 +85,7 @@ base::Optional<std::vector<uint8_t>> BuildIppRequest(
     ipp_t* ipp);
 
 // Mojom converter for ipp_t objects, return nullptr on failure.
+// Note: This function does not take ownership of |ipp|.
 chrome::mojom::IppMessagePtr ConvertIppToMojo(ipp_t* ipp);
 
 // Common converters for working with arbitrary byte buffers.