Add bailout checks to Message::Get{Start,End}Column()

Also add an initial ON_BAILOUT() check to these two 'bailout enabled'
accessors.

(The need for these was encountered via http://crbug.com/393273)

R=dcarney@chromium.org

Review URL: https://codereview.chromium.org/402173003

Patch from Sigbjorn Finne <sof@opera.com>.

git-svn-id: https://v8.googlecode.com/svn/branches/bleeding_edge@22553 ce2b1a6d-e550-0410-aec6-3dcde31c8c00

diff --git a/src/api.cc b/src/api.cc
index fb367de4914ca5321613f17cc3319dfaef1d0fb1..ddc57f26d1b12c39e4ea0d0f139d39f94d4c183a 100644 (file)
--- a/src/api.cc
+++ b/src/api.cc
@@ -2068,6 +2068,7 @@ int Message::GetEndPosition() const {
 
 int Message::GetStartColumn() const {
   i::Isolate* isolate = Utils::OpenHandle(this)->GetIsolate();
+  ON_BAILOUT(isolate, "v8::Message::GetStartColumn()", return kNoColumnInfo);
   ENTER_V8(isolate);
   i::HandleScope scope(isolate);
   i::Handle<i::JSObject> data_obj = Utils::OpenHandle(this);
@@ -2082,6 +2083,7 @@ int Message::GetStartColumn() const {
 
 int Message::GetEndColumn() const {
   i::Isolate* isolate = Utils::OpenHandle(this)->GetIsolate();
+  ON_BAILOUT(isolate, "v8::Message::GetEndColumn()", return kNoColumnInfo);
   ENTER_V8(isolate);
   i::HandleScope scope(isolate);
   i::Handle<i::JSObject> data_obj = Utils::OpenHandle(this);