Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(345)

Unified Diff: runtime/vm/object.cc

Issue 246173002: Revert "Mark private functions in dart:* that are native or constructors as not visible (omitted fr… (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 6 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « runtime/vm/object.h ('k') | runtime/vm/object_test.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: runtime/vm/object.cc
diff --git a/runtime/vm/object.cc b/runtime/vm/object.cc
index 73e5851065732419b6a98ad92ca90b7d2b507f36..d1ec93ce97f0d11f0df9f1aca3a0e703db92d3c6 100644
--- a/runtime/vm/object.cc
+++ b/runtime/vm/object.cc
@@ -156,11 +156,47 @@ const double MegamorphicCache::kLoadFactor = 0.75;
// The following functions are marked as invisible, meaning they will be hidden
// in the stack trace and will be hidden from reflective access.
// (Library, class name, method name)
-// Additionally, private functions in dart:* that are native or constructors are
-// marked as invisible by the parser.
#define INVISIBLE_LIST(V) \
+ V(CoreLibrary, Object, _noSuchMethod) \
+ V(CoreLibrary, Object, _as) \
+ V(CoreLibrary, Object, _instanceOf) \
+ V(CoreLibrary, _List, _List.) \
+ V(CoreLibrary, AssertionError, _throwNew) \
+ V(CoreLibrary, TypeError, _throwNew) \
+ V(CoreLibrary, FallThroughError, _throwNew) \
+ V(CoreLibrary, AbstractClassInstantiationError, _throwNew) \
V(CoreLibrary, int, _throwFormatException) \
V(CoreLibrary, int, _parse) \
+ V(CoreLibrary, StackTrace, _setupFullStackTrace) \
+ V(CoreLibrary, _OneByteString, _setAt) \
+ V(CoreLibrary, _StringBase, _substringUncheckedNative) \
+ V(CoreLibrary, _OneByteString, _substringUncheckedNative) \
+ V(CoreLibrary, _GrowableList, _setData) \
+ V(CoreLibrary, _GrowableList, _setLength) \
+ V(TypedDataLibrary, _TypedList, _getInt8) \
+ V(TypedDataLibrary, _TypedList, _setInt8) \
+ V(TypedDataLibrary, _TypedList, _getUint8) \
+ V(TypedDataLibrary, _TypedList, _setUint8) \
+ V(TypedDataLibrary, _TypedList, _getInt16) \
+ V(TypedDataLibrary, _TypedList, _setInt16) \
+ V(TypedDataLibrary, _TypedList, _getUint16) \
+ V(TypedDataLibrary, _TypedList, _setUint16) \
+ V(TypedDataLibrary, _TypedList, _getInt32) \
+ V(TypedDataLibrary, _TypedList, _setInt32) \
+ V(TypedDataLibrary, _TypedList, _getUint32) \
+ V(TypedDataLibrary, _TypedList, _setUint32) \
+ V(TypedDataLibrary, _TypedList, _getInt64) \
+ V(TypedDataLibrary, _TypedList, _setInt64) \
+ V(TypedDataLibrary, _TypedList, _getUint64) \
+ V(TypedDataLibrary, _TypedList, _setUint64) \
+ V(TypedDataLibrary, _TypedList, _getFloat32) \
+ V(TypedDataLibrary, _TypedList, _setFloat32) \
+ V(TypedDataLibrary, _TypedList, _getFloat64) \
+ V(TypedDataLibrary, _TypedList, _setFloat64) \
+ V(TypedDataLibrary, _TypedList, _getFloat32x4) \
+ V(TypedDataLibrary, _TypedList, _setFloat32x4) \
+
+
static void MarkFunctionAsInvisible(const Library& lib,
const char* class_name,
@@ -8927,7 +8963,6 @@ RawLibrary* Library::NewLibraryHelper(const String& url,
result.set_native_entry_symbol_resolver(NULL);
result.raw_ptr()->corelib_imported_ = true;
result.set_debuggable(false);
- result.set_is_dart_scheme(url.StartsWith(Symbols::DartScheme()));
result.raw_ptr()->load_state_ = RawLibrary::kAllocated;
result.raw_ptr()->index_ = -1;
const intptr_t kInitialNameCacheSize = 64;
« no previous file with comments | « runtime/vm/object.h ('k') | runtime/vm/object_test.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698