commit bf2f289700bec61cb63e2e0ca0b0de79b8b2eace
author Lei Zhang <thestig@chromium.org> Wed Mar 27 17:00:02 2019 +0000
committer Chromium commit bot <commit-bot@chromium.org> Wed Mar 27 17:00:02 2019 +0000
tree 9926ad77b7dd462c38feca8ade74c753ecf81aff
parent 45835f1397012df427ff82ff8cd865a065542668

Undo a temporary workaround in UnownedPtr.

The workaround is no longer needed since Clang fixed the issue that made
ProbeForLowSeverityLifetimeIssue() fail.

BUG=chromium:929475

Change-Id: I71fbbc1aa86002b4533936a53858a112a38f7a12
Reviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/52470
Reviewed-by: Tom Sepez <tsepez@chromium.org>
Commit-Queue: Lei Zhang <thestig@chromium.org>

core/fxcrt/unowned_ptr.h

diff --git a/core/fxcrt/unowned_ptr.h b/core/fxcrt/unowned_ptr.h
index 8be3bd3..0a44f9d 100644
--- a/core/fxcrt/unowned_ptr.h
+++ b/core/fxcrt/unowned_ptr.h
@@ -107,7 +107,7 @@
   inline void ProbeForLowSeverityLifetimeIssue() {
 #if defined(MEMORY_TOOL_REPLACES_ALLOCATOR)
     if (m_pObj)
-      (void)reinterpret_cast<const volatile uint8_t*>(m_pObj)[0];
+      reinterpret_cast<const volatile uint8_t*>(m_pObj)[0];
 #endif
   }