Workaround for http://crbug.com/9746:...

Workaround for http://crbug.com/9746:
- Added special cutouts if a Vector has NULL data, which will now happen
  if an external string's resource has been deleted.
- Added an verification phase before old gen GC to verify that all real
  entries in the SymbolTable are valid symbols.
- Added test that verifies the correct behaviour of the workaround.


Committed: http://code.google.com/p/v8/source/detail?r=1689

[Mike Belshe, 2009-04-09 22:55:56]

lgtm

http://codereview.chromium.org/66011/diff/1/2
File src/objects.cc (right):

http://codereview.chromium.org/66011/diff/1/2#newcode4149
Line 4149: Vector<const uc16> vecb = b->ToUC16Vector();
Is 'vecb' a good name?  (not sure what it is?)

[iposva, 2009-04-09 23:02:09]

Not sure if this helps readability, but at least it is "consistent" now.

-Ivan

http://codereview.chromium.org/66011/diff/1/2
File src/objects.cc (right):

http://codereview.chromium.org/66011/diff/1/2#newcode4149
Line 4149: Vector<const uc16> vecb = b->ToUC16Vector();
On 2009/04/09 22:55:56, Mike Belshe wrote:
> Is 'vecb' a good name?  (not sure what it is?)

Renamed to vb to match the existing convention naming the VectorIterator for the
string b: ib.