Cleanup: Remove NativesCollection<.>::*Raw* methods.

src/natives-external.cc

 // Copyright 2014 the V8 project authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "src/natives.h"
 
 #include "src/base/logging.h"
 #include "src/list.h"
 #include "src/list-inl.h"
 #include "src/snapshot-source-sink.h"
@@ skipping 14 matching lines @@
  public:
   ~NativesStore() {
     for (int i = 0; i < native_names_.length(); i++) {
       native_names_[i].Dispose();
     }
   }
 
   int GetBuiltinsCount() { return native_ids_.length(); }
   int GetDebuggerCount() { return debugger_count_; }
 
-  Vector<const char> GetScriptName(int index) { return native_names_[index]; }
-
-  Vector<const char> GetRawScriptSource(int index) {
+  Vector<const char> GetScriptSource(int index) {
     return native_source_[index];
   }
 
+  Vector<const char> GetScriptName(int index) { return native_names_[index]; }
+
   int GetIndex(const char* id) {
     for (int i = 0; i < native_ids_.length(); ++i) {
       int native_id_length = native_ids_[i].length();
       if ((static_cast<int>(strlen(id)) == native_id_length) &&
           (strncmp(id, native_ids_[i].start(), native_id_length) == 0)) {
         return i;
       }
     }
     DCHECK(false);
     return -1;
   }
 
-  int GetRawScriptsSize() {
-    DCHECK(false);  // Used for compression. Doesn't really make sense here.
-    return 0;
-  }
-
-  Vector<const byte> GetScriptsSource() {
-    DCHECK(false);  // Used for compression. Doesn't really make sense here.
-    return Vector<const byte>();
+  Vector<const char> GetScriptsSource() {
+    DCHECK(false);  // Not implemented.
+    return Vector<const char>();
   }
 
   static NativesStore* MakeFromScriptsSource(SnapshotByteSource* source) {
     NativesStore* store = new NativesStore;
 
     // We expect the libraries in the following format:
     //   int: # of debugger sources.
     //   2N blobs: N pairs of source name + actual source.
     //   then, repeat for non-debugger sources.
     int debugger_count = source->GetInt();
@@ skipping 99 matching lines @@
 template<NativeType type>
 int NativesCollection<type>::GetDebuggerCount() {
   return NativesHolder<type>::get()->GetDebuggerCount();
 }
 
 template<NativeType type>
 int NativesCollection<type>::GetIndex(const char* name) {
   return NativesHolder<type>::get()->GetIndex(name);
 }
 
-template<NativeType type>
-int NativesCollection<type>::GetRawScriptsSize() {
-  return NativesHolder<type>::get()->GetRawScriptsSize();
+template <NativeType type>
+Vector<const char> NativesCollection<type>::GetScriptSource(int index) {
+  return NativesHolder<type>::get()->GetScriptSource(index);
 }
 
 template<NativeType type>
-Vector<const char> NativesCollection<type>::GetRawScriptSource(int index) {
-  return NativesHolder<type>::get()->GetRawScriptSource(index);
-}
-
-template<NativeType type>
 Vector<const char> NativesCollection<type>::GetScriptName(int index) {
   return NativesHolder<type>::get()->GetScriptName(index);
 }
 
-template<NativeType type>
-Vector<const byte> NativesCollection<type>::GetScriptsSource() {
+template <NativeType type>
+Vector<const char> NativesCollection<type>::GetScriptsSource() {
   return NativesHolder<type>::get()->GetScriptsSource();
 }
 
-template<NativeType type>
-void NativesCollection<type>::SetRawScriptsSource(
-    Vector<const char> raw_source) {
-  CHECK(false);  // Use SetNativesFromFile for this implementation.
-}
-
 
 // The compiler can't 'see' all uses of the static methods and hence
 // my chose to elide them. This we'll explicitly instantiate these.
 template class NativesCollection<CORE>;
 template class NativesCollection<EXPERIMENTAL>;
 
 }  // namespace v8::internal
 }  // namespace v8