Move declaration of SerializedScopeInfo from variables.h to objects.h

Move declaration of SerializedScopeInfo from variables.h to objects.h

This eliminates compile-errors when assigning Handle<SerializedScopeInfo> to
Handle<Object> in a place where the declaration was not available because
variables.h was not included.

As a result I had to also move the enum Variable::Mode to v8globals.h and
rename it to VariableMode.
Committed: http://code.google.com/p/v8/source/detail?r=9575

[Yang, 2011-10-11 07:56:21]

On 2011/10/10 14:17:21, fschneider wrote:

LGTM with comments.

[Yang, 2011-10-11 07:56:35]

LGTM with comments.

http://codereview.chromium.org/8221004/diff/1/src/parser.cc
File src/parser.cc (left):

http://codereview.chromium.org/8221004/diff/1/src/parser.cc#oldcode1691
src/parser.cc:1691: Scope* declaration_scope = mode == Variable::LET
Consider adding brackets here for better readability.

http://codereview.chromium.org/8221004/diff/1/src/parser.cc#oldcode1882
src/parser.cc:1882: bool in_with = mode == Variable::VAR ? inside_with() :
false;
Consider brackets.

[fschneider, 2011-10-11 11:04:22]

http://codereview.chromium.org/8221004/diff/1/src/parser.cc
File src/parser.cc (left):

http://codereview.chromium.org/8221004/diff/1/src/parser.cc#oldcode1691
src/parser.cc:1691: Scope* declaration_scope = mode == Variable::LET
On 2011/10/11 07:56:35, Yang wrote:
> Consider adding brackets here for better readability.

Done.

http://codereview.chromium.org/8221004/diff/1/src/parser.cc#oldcode1882
src/parser.cc:1882: bool in_with = mode == Variable::VAR ? inside_with() :
false;
On 2011/10/11 07:56:35, Yang wrote:
> Consider brackets.

Done.