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

Unified Diff: runtime/vm/object.cc

Issue 247683002: Reapply "Mark private functions in dart:* that are native or constructors as not visible (omitted f… (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: regen snapshot test 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 820048d33a9e3b8e8ca5b53a2fbf6d69ae87d994..41533067419240b4b9b90555b494d12661d10b02 100644
--- a/runtime/vm/object.cc
+++ b/runtime/vm/object.cc
@@ -156,47 +156,11 @@ 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,
@@ -8963,6 +8927,7 @@ 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