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

Issue 14876015: Add wrapper library for the nacl_irt_exception_handling IRT interface (Closed)

Created:
7 years, 7 months ago by Mark Seaborn
Modified:
7 years, 7 months ago
Reviewers:
noelallen1, bradn, binji
CC:
native-client-reviews_googlegroups.com, noelallen_use_chromium
Visibility:
Public.

Description

Add wrapper library for the nacl_irt_exception_handling IRT interface Switch src/untrusted/minidump_generator and src/untrusted/crash_dump over to using the new wrapper library. Add the wrapper library to the Gyp build. Minor cleanup to use perror() to report errno status in minidump_generator.cc. BUG=https://code.google.com/p/nativeclient/issues/detail?id=2401 TEST=run_minidump_test_irt run_untrusted_crash_dump_test_irt Committed: https://src.chromium.org/viewvc/native_client?view=rev&revision=11345

Patch Set 1 #

Patch Set 2 : Rebase #

Patch Set 3 : Remove changes to nacl_core_sdk.gyp since this file is unused #

Patch Set 4 : Cleanup + remove minidump_generator.gyp for now because of pthread.h problem in Gyp build #

Unified diffs Side-by-side diffs Delta from patch set Stats (+132 lines, -31 lines) Patch
M site_scons/site_tools/library_deps.py View 1 1 chunk +4 lines, -0 lines 0 comments Download
M src/include/nacl/nacl_exception.h View 1 2 3 2 chunks +46 lines, -0 lines 0 comments Download
M src/untrusted/crash_dump/untrusted_crash_dump.c View 4 chunks +2 lines, -10 lines 0 comments Download
M src/untrusted/minidump_generator/minidump_generator.cc View 1 4 chunks +6 lines, -12 lines 0 comments Download
M src/untrusted/nacl/nacl.gyp View 1 chunk +15 lines, -0 lines 0 comments Download
M src/untrusted/nacl/nacl.scons View 1 chunk +4 lines, -0 lines 0 comments Download
A src/untrusted/nacl/nacl_exception.c View 1 2 3 1 chunk +50 lines, -0 lines 0 comments Download
M tests/exception_test/exception_crash_test.c View 1 2 3 2 chunks +1 line, -3 lines 0 comments Download
M tests/exception_test/exception_test.c View 1 2 3 4 chunks +4 lines, -6 lines 0 comments Download

Messages

Total messages: 6 (0 generated)
Mark Seaborn
Questions: * nacl_code_sdk.gyp doesn't seem to install libraries for ARM or PNaCl. Does it actually ...
7 years, 7 months ago (2013-05-10 16:50:46 UTC) #1
bradn
* nacl_code_sdk.gyp doesn't seem to install libraries for ARM or PNaCl. Does it actually work? ...
7 years, 7 months ago (2013-05-10 17:34:46 UTC) #2
binji
On 2013/05/10 17:34:46, bradn wrote: > * nacl_code_sdk.gyp doesn't seem to install libraries for ARM ...
7 years, 7 months ago (2013-05-10 17:41:26 UTC) #3
Mark Seaborn
Please take a look. > * nacl_code_sdk.gyp doesn't seem to install libraries for ARM or ...
7 years, 7 months ago (2013-05-14 18:05:34 UTC) #4
bradn
lgtm
7 years, 7 months ago (2013-05-14 18:22:34 UTC) #5
Mark Seaborn
7 years, 7 months ago (2013-05-14 18:29:53 UTC) #6
Message was sent while issue was closed.
Committed patchset #4 manually as r11345 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698