Made PropertyType handling even more explicit.

Made PropertyType handling even more explicit.

Replaced FIRST_PHANTOM_PROPERTY_TYPE by a predicate. Removed the (hopefully)
last default cases for switches on PropertyType. Benchmarks show that both
changes are performace-neutral.

Now every value of PropertyType should either be handled by an explicit case in
a switch or by an equality operator. Therefore, the C++ compiler should finally
be able to tell us which places to touch when changing PropertyType.

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

[Kevin Millikin (Chromium), 2011-11-09 12:39:54]

LGTM.

http://codereview.chromium.org/8506004/diff/1/src/profile-generator.cc
File src/profile-generator.cc (right):

http://codereview.chromium.org/8506004/diff/1/src/profile-generator.cc#newcod...
src/profile-generator.cc:2123: // TODO(svenpanne): Should we really ignore
accessors here?
That's a good question.  They are really a reference to a function or pair of
functions, so we probably should not skip them.  It also looks like they are
counted in the dictionary case.  I'll ping Mikhail about it.