LoadWebLocalizedStrings method should be moved in correct file
authoraakash_jain@apple.com <aakash_jain@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 25 Jan 2017 20:02:00 +0000 (20:02 +0000)
committeraakash_jain@apple.com <aakash_jain@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 25 Jan 2017 20:02:00 +0000 (20:02 +0000)
https://bugs.webkit.org/show_bug.cgi?id=167393

Reviewed by Alexey Proskuryakov.

Source/WebKit/ios:

* Misc/WebUIKitSupport.mm:
(LoadWebLocalizedStringsTimerCallback): Moved from WebLocalizableStrings.mm
(LoadWebLocalizedStrings): Moved from WebLocalizableStrings.mm

Source/WebKit/mac:

* Misc/WebLocalizableStrings.h: Deleted LoadWebLocalizedStrings method.
* Misc/WebLocalizableStrings.mm:
(LoadWebLocalizedStringsTimerCallback): Deleted, moved to WebUIKitSupport.mm
(LoadWebLocalizedStrings): Deleted, moved to WebUIKitSupport.mm

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@211156 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebKit/ios/ChangeLog
Source/WebKit/ios/Misc/WebUIKitSupport.mm
Source/WebKit/mac/ChangeLog
Source/WebKit/mac/Misc/WebLocalizableStrings.h
Source/WebKit/mac/Misc/WebLocalizableStrings.mm

index 6d24ae8..aa569a6 100644 (file)
@@ -1,3 +1,14 @@
+2017-01-25  Aakash Jain  <aakash_jain@apple.com>
+
+        LoadWebLocalizedStrings method should be moved in correct file
+        https://bugs.webkit.org/show_bug.cgi?id=167393
+
+        Reviewed by Alexey Proskuryakov.
+
+        * Misc/WebUIKitSupport.mm:
+        (LoadWebLocalizedStringsTimerCallback): Moved from WebLocalizableStrings.mm
+        (LoadWebLocalizedStrings): Moved from WebLocalizableStrings.mm
+
 2017-01-21  Darin Adler  <darin@apple.com>
 
         Remove PassRefPtr from "page" directory of WebCore, also deploy references
index c4efc2e..696949a 100644 (file)
@@ -29,7 +29,7 @@
 
 #import "WebDatabaseManagerInternal.h"
 #import "WebKitSystemInterface.h"
-#import "WebLocalizableStrings.h"
+#import "WebLocalizableStringsInternal.h"
 #import "WebPlatformStrategies.h"
 #import "WebSystemInterface.h"
 #import "WebViewPrivate.h"
@@ -51,6 +51,23 @@ static inline bool linkedOnOrAfterIOS5()
     return s_linkedOnOrAfterIOS5;
 }
 
+// See <rdar://problem/7902473> Optimize WebLocalizedString for why we do this on a background thread on a timer callback
+static void LoadWebLocalizedStringsTimerCallback(CFRunLoopTimerRef timer, void *info)
+{
+    dispatch_async(dispatch_get_global_queue(DISPATCH_QUEUE_PRIORITY_LOW, 0), ^ {
+        // We don't care if we find this string, but searching for it will load the plist and save the results.
+        // FIXME: It would be nicer to do this in a more direct way.
+        UI_STRING_KEY_INTERNAL("Typing", "Typing (Undo action name)", "Undo action name");
+    });
+}
+
+static void LoadWebLocalizedStrings()
+{
+    CFRunLoopTimerRef timer = CFRunLoopTimerCreate(kCFAllocatorDefault, CFAbsoluteTimeGetCurrent(), 0, 0, 0, &LoadWebLocalizedStringsTimerCallback, NULL);
+    CFRunLoopAddTimer(CFRunLoopGetCurrent(), timer, kCFRunLoopCommonModes);
+    CFRelease(timer);
+}
+
 void WebKitInitialize(void)
 {
     static bool webkitInitialized;
index 5ce060a..4f47c9e 100644 (file)
@@ -1,3 +1,15 @@
+2017-01-25  Aakash Jain  <aakash_jain@apple.com>
+
+        LoadWebLocalizedStrings method should be moved in correct file
+        https://bugs.webkit.org/show_bug.cgi?id=167393
+
+        Reviewed by Alexey Proskuryakov.
+
+        * Misc/WebLocalizableStrings.h: Deleted LoadWebLocalizedStrings method.
+        * Misc/WebLocalizableStrings.mm:
+        (LoadWebLocalizedStringsTimerCallback): Deleted, moved to WebUIKitSupport.mm
+        (LoadWebLocalizedStrings): Deleted, moved to WebUIKitSupport.mm
+
 2017-01-24  Zalan Bujtas  <zalan@apple.com>
 
         Add simple line layout toggle to MiniBrowser
index 81459b1..cc38c8a 100644 (file)
@@ -49,10 +49,6 @@ extern "C" {
 
 WebLocalizedStringType WebLocalizedString(WebLocalizableStringsBundle* bundle, const char* key);
 
-#if TARGET_OS_IPHONE
-void LoadWebLocalizedStrings(void); // The first WebLocalizedString call can take over 20ms unless this function is called beforehand.
-#endif
-
 #ifdef __cplusplus
 }
 #endif
index f749083..2a8a837 100644 (file)
 #import <wtf/Assertions.h>
 #import <wtf/MainThread.h>
 
-#if PLATFORM(IOS)
-#import "WebLocalizableStringsInternal.h"
-#import <dispatch/dispatch.h>
-#endif
-
 WebLocalizableStringsBundle WebKitLocalizableStringsBundle = { "com.apple.WebKit", 0 };
 
 NSString *WebLocalizedString(WebLocalizableStringsBundle *stringsBundle, const char *key)
@@ -72,24 +67,3 @@ NSString *WebLocalizedString(WebLocalizableStringsBundle *stringsBundle, const c
     ASSERT_WITH_MESSAGE(result != notFound, "could not find localizable string %s in bundle", key);
     return result;
 }
-
-#if PLATFORM(IOS)
-
-// See <rdar://problem/7902473> Optimize WebLocalizedString for why we do this on a background thread on a timer callback
-static void LoadWebLocalizedStringsTimerCallback(CFRunLoopTimerRef timer, void *info)
-{
-    dispatch_async(dispatch_get_global_queue(DISPATCH_QUEUE_PRIORITY_LOW, 0), ^{
-        // We don't care if we find this string, but searching for it will load the plist and save the results.
-        // FIXME: It would be nicer to do this in a more direct way.
-        UI_STRING_KEY_INTERNAL("Typing", "Typing (Undo action name)", "Undo action name");
-    });
-}
-
-void LoadWebLocalizedStrings(void)
-{
-    CFRunLoopTimerRef timer = CFRunLoopTimerCreate(kCFAllocatorDefault, CFAbsoluteTimeGetCurrent(), 0, 0, 0, &LoadWebLocalizedStringsTimerCallback, NULL);
-    CFRunLoopAddTimer(CFRunLoopGetCurrent(), timer, kCFRunLoopCommonModes);
-    CFRelease(timer);
-}
-
-#endif