Fixed regression http://code.google.com/p/v8/issues/detail?id=236.

test/cctest/test-compiler.cc

 // Copyright 2006-2008 the V8 project authors. All rights reserved.
 // Redistribution and use in source and binary forms, with or without
 // modification, are permitted provided that the following conditions are
 // met:
 //
 //     * Redistributions of source code must retain the above copyright
 //       notice, this list of conditions and the following disclaimer.
 //     * Redistributions in binary form must reproduce the above
 //       copyright notice, this list of conditions and the following
 //       disclaimer in the documentation and/or other materials provided
@@ skipping 284 matching lines @@
               *Factory::LookupAsciiSymbol("foo")));
   CHECK(fun1->IsJSFunction());
 
   Object** argv[1] = {
     Handle<Object>::cast(Factory::LookupAsciiSymbol("hello")).location()
   };
   Execution::Call(Handle<JSFunction>::cast(fun1), global, 1, argv,
                   &has_pending_exception);
   CHECK(!has_pending_exception);
 }
+
+
+// Regression 236. Calling InitLineEnds on a Script with undefined
+// source resulted in crash.
+TEST(Regression236) {
+  InitializeVM();
+  v8::HandleScope scope;
+
+  Handle<Script> script = Factory::NewScript(Factory::empty_string());
+  script->set_source(Heap::undefined_value());
+  CHECK_EQ(-1, script->GetLineNumber(0));
+  CHECK_EQ(-1, script->GetLineNumber(100));
+  CHECK_EQ(-1, script->GetLineNumber(-1));

[Søren Thygesen Gjesse, 2009/02/24 13:23:52]

As this bug originated from Chrome where source is external strings using an
empty external string in the test case as well might be a good idea.

[Mikhail Naganov, 2009/02/24 13:41:01]

Ok, will add such test. But it will be a new CL since I already commited this
one immediately after your LGTM ;)

+}