Bug 1281391 IMContextWrapper and nsGtkKeyUtils should use LazyLogModule instead of PR_NewLogModule() for making MOZ_LOG* aware r=m_kato

MozReview-Commit-ID: 5xqIfFmakvq

--HG--
extra : rebase_source : 825b0b667d5200ba8d8c07ddbb3ea98f499ebd79

widget/gtk/IMContextWrapper.cpp

@@ -21,7 +21,7 @@
 namespace mozilla {
 namespace widget {
 
-PRLogModuleInfo* gGtkIMLog = nullptr;
+LazyLogModule gGtkIMLog("nsGtkIMModuleWidgets");
 
 static inline const char*
 ToChar(bool aBool)
@@ -180,9 +180,6 @@ IMContextWrapper::IMContextWrapper(nsWindow* aOwnerWindow)
     , mSetCursorPositionOnKeyEvent(true)
     , mPendingResettingIMContext(false)
 {
-    if (!gGtkIMLog) {
-        gGtkIMLog = PR_NewLogModule("nsGtkIMModuleWidgets");
-    }
     static bool sFirstInstance = true;
     if (sFirstInstance) {
         sFirstInstance = false;

widget/gtk/nsGtkKeyUtils.cpp

@@ -23,8 +23,6 @@
 #include "nsIBidiKeyboard.h"
 #include "nsServiceManagerUtils.h"
 
-PRLogModuleInfo* gKeymapWrapperLog = nullptr;
-
 #include "mozilla/ArrayUtils.h"
 #include "mozilla/MouseEvents.h"
 #include "mozilla/TextEvents.h"
@@ -32,6 +30,8 @@ PRLogModuleInfo* gKeymapWrapperLog = nullptr;
 namespace mozilla {
 namespace widget {
 
+LazyLogModule gKeymapWrapperLog("KeymapWrapperWidgets");
+
 #define IS_ASCII_ALPHABETICAL(key) \
     ((('a' <= key) && (key <= 'z')) || (('A' <= key) && (key <= 'Z')))
 
@@ -163,9 +163,6 @@ KeymapWrapper::KeymapWrapper() :
     mInitialized(false), mGdkKeymap(gdk_keymap_get_default()),
     mXKBBaseEventCode(0)
 {
-    if (!gKeymapWrapperLog) {
-        gKeymapWrapperLog = PR_NewLogModule("KeymapWrapperWidgets");
-    }
     MOZ_LOG(gKeymapWrapperLog, LogLevel::Info,
         ("KeymapWrapper(%p): Constructor, mGdkKeymap=%p",
          this, mGdkKeymap));