Remove Scope::scope_contains_with_ bit

test/mjsunit/debug-function-scopes.js

 // Copyright 2012 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 55 matching lines @@
   }
   return F2(17);
 })(5);
 
 var mirror = Debug.MakeMirror(f1);
 
 assertEquals(6, mirror.scopeCount());
 
 CheckScope(mirror.scope(0), { a: 4, b: 5 }, ScopeType.Closure);
 CheckScope(mirror.scope(1), { w: 5, v: "Capybara" }, ScopeType.With);
-CheckScope(mirror.scope(2), { y: 17, z: 22 }, ScopeType.Closure);

[adamk, 2016/03/14 22:22:11]

'y' is now in the same state as 'p', as it's not referred to by any inner
closure.

+CheckScope(mirror.scope(2), { z: 22 }, ScopeType.Closure);
 CheckScope(mirror.scope(3), { x: 5 }, ScopeType.Closure);
 CheckScope(mirror.scope(4), {}, ScopeType.Script);
 CheckScope(mirror.scope(5), {}, ScopeType.Global);
 
 var f2 = function() { return 5; }
 
 var mirror = Debug.MakeMirror(f2);
 
 assertEquals(2, mirror.scopeCount());
 
@@ skipping 71 matching lines @@
   var mirror = Debug.MakeMirror(f);
   assertEquals(0, mirror.scopeCount());
 }
 
 CheckNoScopeVisible(Number);
 
 CheckNoScopeVisible(Function.toString);
 
 // This getter is known to be implemented as closure.
 CheckNoScopeVisible(new Error().__lookupGetter__("stack"));