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

Unified Diff: third_party/ocmock/README.chromium

Issue 316873003: Add a workaround to compare types in mocked method (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Add note in README.chromium and sync with upstream pull request Created 6 years, 6 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 | « third_party/ocmock/OCMock/OCMBoxedReturnValueProvider.mm ('k') | third_party/ocmock/ocmock.gyp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: third_party/ocmock/README.chromium
diff --git a/third_party/ocmock/README.chromium b/third_party/ocmock/README.chromium
index 666072a86b2760f4ba0eb0efbbd8f1d39731985f..dec968a3fa81a1af0701c3f3bad5aed4dc03b04b 100644
--- a/third_party/ocmock/README.chromium
+++ b/third_party/ocmock/README.chromium
@@ -31,3 +31,9 @@ Chromium also patches in 77400c22b5bafd8aaaa75f90a1d997838204577a that fixes
the comparison of types between invocation return type description (which
changed from "c" to "B" on arm64) and NSValue objCType (which didn't change)
on arm64 for BOOL values.
+
+Chromium does patch OCMBoxedReturnValueProvider.m to handle comparison of
+return type and value type for C++ objects when compiling with clang and
+using libc++ STL library. The file is renamed OCMBoxedReturnValueProvider.mm
+to be compiled as an Objective-C++ class to allow detection of the C++ STL
+library used. Pull request: https://github.com/erikdoe/ocmock/pull/97
« no previous file with comments | « third_party/ocmock/OCMock/OCMBoxedReturnValueProvider.mm ('k') | third_party/ocmock/ocmock.gyp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698