platform/testing/{URL|Unit}TestHelpers improvements

third_party/WebKit/Source/web/tests/ListenerLeakTest.cpp

Index: third_party/WebKit/Source/web/tests/ListenerLeakTest.cpp
diff --git a/third_party/WebKit/Source/web/tests/ListenerLeakTest.cpp b/third_party/WebKit/Source/web/tests/ListenerLeakTest.cpp
index a30220e0b71ff87db9ff118ea20497519a2ab26e..d5892d7692cbf94eff3eeba60e10e0b8ceb807e0 100644
--- a/third_party/WebKit/Source/web/tests/ListenerLeakTest.cpp
+++ b/third_party/WebKit/Source/web/tests/ListenerLeakTest.cpp
@@ -28,15 +28,16 @@
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
+#include <v8/include/v8-profiler.h>

[Takashi Toyoshima, 2017/02/06 09:13:42]

it seems we do not have "v8/include/" prefix usually for v8 headers.

+#include <v8/include/v8.h>
 #include "platform/testing/URLTestHelpers.h"
+#include "platform/testing/UnitTestHelpers.h"
 #include "public/platform/Platform.h"
 #include "public/platform/WebURLLoaderMockFactory.h"
 #include "public/web/WebCache.h"
 #include "public/web/WebView.h"
 #include "testing/gtest/include/gtest/gtest.h"
 #include "web/tests/FrameTestHelpers.h"
-#include <v8/include/v8-profiler.h>
-#include <v8/include/v8.h>
 
 namespace blink {
 
@@ -82,15 +83,15 @@ int GetNumObjects(const char* constructor) {
   return count;
 }
 
-class ListenerLeakTest : public testing::Test {
+class ListenerLeakTest : public ::testing::Test {
  public:
   void RunTest(const std::string& filename) {
     std::string baseURL("http://www.example.com/");
     std::string fileName(filename);
     bool executeScript = true;
-    URLTestHelpers::registerMockedURLFromBaseURL(
-        WebString::fromUTF8(baseURL.c_str()),
-        WebString::fromUTF8(fileName.c_str()));
+    URLTestHelpers::registerMockedURLLoadFromBase(
+        WebString::fromUTF8(baseURL), blink::testing::webTestDataPath(),
+        WebString::fromUTF8(fileName));
     webViewHelper.initializeAndLoad(baseURL + fileName, executeScript);
   }