[icu] Support loading data file from default location

include/v8.h

Index: include/v8.h
diff --git a/include/v8.h b/include/v8.h
index 1c50d9b473b916ac115e58cd2ae6f9c92990beb4..17bd0b884400be528f156a82fce34b798628c6fd 100644
--- a/include/v8.h
+++ b/include/v8.h
@@ -6710,6 +6710,19 @@ class V8_EXPORT V8 {
   static bool InitializeICU(const char* icu_data_file = NULL);
 
   /**
+   * Initialize the ICU library bundled with V8. The embedder should only
+   * invoke this method when V8 was compiled with the ICU data in an external
+   * file and when the default location of that file should be used. The
+   * default is a file called icudtl.dat side-by-side with the executable.
+   * Returns true on success.

[jochen (gone - plz use gerrit), 2016/06/07 19:05:44]

or icudtl.dll on windows

[Michael Achenbach, 2016/06/08 07:08:18]

Don't understand. This is only about the data file not the dynamic library. The
data file is the same on windows, no? E.g. I see it here being packed:
https://build.chromium.org/p/chromium.win/builders/Win%20Builder/builds/25121...

I don't see the dll there.

[Michael Achenbach, 2016/06/08 08:00:25]

I rewrote the docu a bit.

+   *
+   * A path to the executable must be provided. Optionally, the location of the
+   * data file can be provided to override the default.
+   */
+  static bool InitializeICUDefaultLocation(const char* exec_path,

[jochen (gone - plz use gerrit), 2016/06/07 19:05:44]

what about just replacing the old API? Or at least mark it as deprecate_soon?

[Michael Achenbach, 2016/06/08 07:08:18]

Is chromium at all using the old API? They must have their own mechanism for
initializing ICU, right?

[Michael Achenbach, 2016/06/08 08:00:25]

Marked.

+                                           const char* icu_data_file = NULL);

[jochen (gone - plz use gerrit), 2016/06/07 19:05:44]

nullptr

[Michael Achenbach, 2016/06/08 08:00:25]

Done. Also in the old api.

+
+  /**
    * Initialize the external startup data. The embedder only needs to
    * invoke this method when external startup data was enabled in a build.
    *