Updating ASAN stub with new API

Updating ASAN stub with new API

BUG=chromium-os:34523
TEST=manual


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=167656

[glotov, 2012-11-13 16:40:50]

hi!

[Alexander Potapenko, 2012-11-13 16:47:19]

https://codereview.chromium.org/11275285/diff/1/chrome/test/pyautolib/asan_st...
File chrome/test/pyautolib/asan_stub.c (right):

https://codereview.chromium.org/11275285/diff/1/chrome/test/pyautolib/asan_st...
chrome/test/pyautolib/asan_stub.c:76: void __asan_set_error_report_callback();
I guess there should also be the implementation, i.e. "{ }"

[glotov, 2012-11-13 16:52:15]

https://codereview.chromium.org/11275285/diff/1/chrome/test/pyautolib/asan_st...
File chrome/test/pyautolib/asan_stub.c (right):

https://codereview.chromium.org/11275285/diff/1/chrome/test/pyautolib/asan_st...
chrome/test/pyautolib/asan_stub.c:76: void __asan_set_error_report_callback();
Yes, right!

On 2012/11/13 16:47:19, Alexander Potapenko wrote:
> I guess there should also be the implementation, i.e. "{ }"

[Alexander Potapenko, 2012-11-13 16:53:35]

LGTM

[commit-bot: I haz the power, 2012-11-13 16:58:15]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/glotov@chromium.org/11275285/4001

[commit-bot: I haz the power, 2012-11-13 16:58:18]

Presubmit check for 11275285-4001 failed and returned exit status 1.


Running presubmit commit checks ...

** Presubmit ERRORS **
Missing LGTM from an OWNER for files in these directories:
    chrome/test/pyautolib

Was the presubmit check useful? Please send feedback & hate mail to
maruel@chromium.org!

[glotov, 2012-11-13 17:00:31]

Hi Nirnimesh, could you please approve.

[dennis_jeffrey, 2012-11-13 17:24:33]

LGTM

Nirnimesh has moved on to another project.

[commit-bot: I haz the power, 2012-11-13 17:35:36]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/glotov@chromium.org/11275285/4001

[commit-bot: I haz the power, 2012-11-13 20:14:57]

Retried try job too often for step(s) browser_tests

[commit-bot: I haz the power, 2012-11-14 11:01:58]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/glotov@chromium.org/11275285/4001

[commit-bot: I haz the power, 2012-11-14 12:06:23]

Change committed as 167656