[blink-in-js] Migrate resources required for blink-in-js to grd - part 3

Source/platform/PlatformResourceLoader.cpp

Index: Source/platform/PlatformResourceLoader.cpp
diff --git a/Source/platform/PlatformResourceLoader.cpp b/Source/platform/PlatformResourceLoader.cpp
new file mode 100644
index 0000000000000000000000000000000000000000..13b21aa69598fd37a345cffc1c80e3f8f273a376
--- /dev/null
+++ b/Source/platform/PlatformResourceLoader.cpp
@@ -0,0 +1,21 @@
+// Copyright 2014 The Chromium 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 "config.h"
+#include "PlatformResourceLoader.h"
+
+#include "public/platform/Platform.h"
+#include "public/platform/WebData.h"
+
+namespace blink {
+
+String loadResourceAsASCIIString(const char* resource)
+{
+    const WebData& resourceData = Platform::current()->loadResource(resource);
+    String dataString(resourceData.data(), resourceData.size());
+    ASSERT(!dataString.isEmpty() && dataString.containsOnlyASCII());

[haraken, 2014/09/10 07:30:33]

What is this check for?

Other call sites of Platform::current()->loadResource are not checking the
condition.

e.g.,
https://code.google.com/p/chromium/codesearch#chromium/src/third_party/WebKit...

If the check is not a worth, you can just directly write
Platform::current()->loadResource in PrivateScriptRunner.cpp and remove
PlatformResourceLoader.{h,cpp}.

[vivekg, 2014/09/10 07:42:45]

This is to make sure the script/stylesheets are indeed in ascii format. @abarth
suggested it here https://codereview.chromium.org/438843004/#msg2
This will be required here: https://codereview.chromium.org/438843004/ as there
are many call sites doing the same. Its a helper to minimize that effort.
Actually this was supposed to go along with that patch. As thats getting
delayed, I am pushing it via this patch which anyway will be used.

+    return dataString;
+}
+
+} // namespace blink