Closed
Bug 1284422
Opened 8 years ago
Closed 8 years ago
Remove unnecessary prefix from MOZ_LOG() in IME related code since current MOZ_LOG appends log module name to each line
Categories
(Core :: Widget, defect)
Core
Widget
Tracking
()
RESOLVED
FIXED
mozilla50
Tracking | Status | |
---|---|---|
firefox50 | --- | fixed |
People
(Reporter: masayuki, Assigned: masayuki)
Details
(Keywords: inputmethod)
Attachments
(11 files)
58 bytes,
text/x-review-board-request
|
m_kato
:
review+
|
Details |
58 bytes,
text/x-review-board-request
|
m_kato
:
review+
|
Details |
58 bytes,
text/x-review-board-request
|
m_kato
:
review+
|
Details |
58 bytes,
text/x-review-board-request
|
m_kato
:
review+
|
Details |
58 bytes,
text/x-review-board-request
|
m_kato
:
review+
|
Details |
58 bytes,
text/x-review-board-request
|
m_kato
:
review+
|
Details |
58 bytes,
text/x-review-board-request
|
m_kato
:
review+
|
Details |
58 bytes,
text/x-review-board-request
|
m_kato
:
review+
|
Details |
58 bytes,
text/x-review-board-request
|
m_kato
:
review+
|
Details |
58 bytes,
text/x-review-board-request
|
m_kato
:
review+
|
Details |
58 bytes,
text/x-review-board-request
|
m_kato
:
review+
|
Details |
Now, all IME related logs use LazyLogModule. In this case, each line starts with the log module name. Therefore, now, each log's prefix like "TSF: " isn't necessary. We should remove them.
Assignee | ||
Comment 1•8 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=try&revision=c59396417c17
Assignee | ||
Comment 2•8 years ago
|
||
Review commit: https://reviewboard.mozilla.org/r/62316/diff/#index_header See other reviews: https://reviewboard.mozilla.org/r/62316/
Attachment #8767927 -
Flags: review?(m_kato)
Attachment #8767928 -
Flags: review?(m_kato)
Attachment #8767929 -
Flags: review?(m_kato)
Attachment #8767930 -
Flags: review?(m_kato)
Attachment #8767931 -
Flags: review?(m_kato)
Attachment #8767932 -
Flags: review?(m_kato)
Attachment #8767933 -
Flags: review?(m_kato)
Attachment #8767934 -
Flags: review?(m_kato)
Attachment #8767936 -
Flags: review?(m_kato)
Attachment #8767937 -
Flags: review?(m_kato)
Assignee | ||
Comment 3•8 years ago
|
||
Review commit: https://reviewboard.mozilla.org/r/62318/diff/#index_header See other reviews: https://reviewboard.mozilla.org/r/62318/
Assignee | ||
Comment 4•8 years ago
|
||
Review commit: https://reviewboard.mozilla.org/r/62320/diff/#index_header See other reviews: https://reviewboard.mozilla.org/r/62320/
Assignee | ||
Comment 5•8 years ago
|
||
Review commit: https://reviewboard.mozilla.org/r/62322/diff/#index_header See other reviews: https://reviewboard.mozilla.org/r/62322/
Assignee | ||
Comment 6•8 years ago
|
||
Review commit: https://reviewboard.mozilla.org/r/62324/diff/#index_header See other reviews: https://reviewboard.mozilla.org/r/62324/
Assignee | ||
Comment 7•8 years ago
|
||
Review commit: https://reviewboard.mozilla.org/r/62326/diff/#index_header See other reviews: https://reviewboard.mozilla.org/r/62326/
Assignee | ||
Comment 8•8 years ago
|
||
Review commit: https://reviewboard.mozilla.org/r/62328/diff/#index_header See other reviews: https://reviewboard.mozilla.org/r/62328/
Assignee | ||
Comment 9•8 years ago
|
||
Review commit: https://reviewboard.mozilla.org/r/62330/diff/#index_header See other reviews: https://reviewboard.mozilla.org/r/62330/
Assignee | ||
Comment 10•8 years ago
|
||
Review commit: https://reviewboard.mozilla.org/r/62332/diff/#index_header See other reviews: https://reviewboard.mozilla.org/r/62332/
Assignee | ||
Comment 11•8 years ago
|
||
Review commit: https://reviewboard.mozilla.org/r/62334/diff/#index_header See other reviews: https://reviewboard.mozilla.org/r/62334/
Assignee | ||
Comment 12•8 years ago
|
||
Review commit: https://reviewboard.mozilla.org/r/62336/diff/#index_header See other reviews: https://reviewboard.mozilla.org/r/62336/
Assignee | ||
Updated•8 years ago
|
Attachment #8767935 -
Flags: review?(m_kato)
Updated•8 years ago
|
Attachment #8767927 -
Flags: review?(m_kato) → review+
Comment 13•8 years ago
|
||
Comment on attachment 8767927 [details] Bug 1284422 part.1 Remove "TSF: " from log of TSFTextStore.cpp https://reviewboard.mozilla.org/r/62316/#review59278
Updated•8 years ago
|
Attachment #8767928 -
Flags: review?(m_kato) → review+
Comment 14•8 years ago
|
||
Comment on attachment 8767928 [details] Bug 1284422 part.2 Fix odd indent of MOZ_LOG() in TSFTextStore.cpp (mainly caused by replacing from PR_LOG()) https://reviewboard.mozilla.org/r/62318/#review59304
Comment 15•8 years ago
|
||
Comment on attachment 8767929 [details] Bug 1284422 part.3 Remove "IMM: " from log of IMMHandler.cpp https://reviewboard.mozilla.org/r/62320/#review59310
Attachment #8767929 -
Flags: review?(m_kato) → review+
Updated•8 years ago
|
Attachment #8767930 -
Flags: review?(m_kato) → review+
Comment 16•8 years ago
|
||
Comment on attachment 8767930 [details] Bug 1284422 part.4 Fix odd indent of MOZ_LOG() in TextInputHandler.mm https://reviewboard.mozilla.org/r/62322/#review59312
Comment 17•8 years ago
|
||
Comment on attachment 8767931 [details] Bug 1284422 part.5 Remove "GTKIM: " from log of IMContextWrapper.cpp https://reviewboard.mozilla.org/r/62324/#review59324
Attachment #8767931 -
Flags: review?(m_kato) → review+
Comment 18•8 years ago
|
||
Comment on attachment 8767932 [details] Bug 1284422 part.6 Reduce indent of MOZ_LOG() in IMContextWrapper.cpp https://reviewboard.mozilla.org/r/62326/#review59326
Attachment #8767932 -
Flags: review?(m_kato) → review+
Comment 19•8 years ago
|
||
Comment on attachment 8767933 [details] Bug 1284422 part.7 Replace "KeymapWrapper(%p): " with "0x%p " and "%p" with "0x%p" in nsGtkKeyUtils.cpp https://reviewboard.mozilla.org/r/62328/#review59328
Attachment #8767933 -
Flags: review?(m_kato) → review+
Comment 20•8 years ago
|
||
Comment on attachment 8767934 [details] Bug 1284422 part.8 Fix odd indent of MOZ_LOG() in nsGtkKeyUtils.cpp (mainly caused by replacing from PR_LOG()) https://reviewboard.mozilla.org/r/62330/#review59330
Attachment #8767934 -
Flags: review?(m_kato) → review+
Comment 21•8 years ago
|
||
Comment on attachment 8767935 [details] Bug 1284422 part.9 Remove "ContentCache(InParent|InChild)?: " from log of ContentCache.cpp since the classes are not in same process under current design https://reviewboard.mozilla.org/r/62332/#review59334
Attachment #8767935 -
Flags: review?(m_kato) → review+
Comment 22•8 years ago
|
||
https://reviewboard.mozilla.org/r/62328/#review59336 nit: replace %p with 0x%p ::: widget/gtk/nsGtkKeyUtils.cpp:168 (Diff revision 1) > mInitialized(false), mGdkKeymap(gdk_keymap_get_default()), > mXKBBaseEventCode(0) > { > MOZ_LOG(gKeymapWrapperLog, LogLevel::Info, > - ("KeymapWrapper(%p): Constructor, mGdkKeymap=%p", > + ("%p Constructor, mGdkKeymap=%p", > this, mGdkKeymap)); other pathes use 0x%p, but these fixes use %p. I think that you should replace all %p with 0x%p
Updated•8 years ago
|
Attachment #8767936 -
Flags: review?(m_kato) → review+
Comment 23•8 years ago
|
||
Comment on attachment 8767936 [details] Bug 1284422 part.10 Remove "IMCO: " from log of IMEContentObserver.cpp https://reviewboard.mozilla.org/r/62334/#review59338
Updated•8 years ago
|
Attachment #8767937 -
Flags: review?(m_kato) → review+
Comment 24•8 years ago
|
||
Comment on attachment 8767937 [details] Bug 1284422 part.11 Remove "IMS: " and "IMEStateManager::" from log of IMEStateManager.cpp https://reviewboard.mozilla.org/r/62336/#review59340
Assignee | ||
Comment 25•8 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=try&revision=7d42fb1d8c74
Assignee | ||
Comment 26•8 years ago
|
||
Comment on attachment 8767927 [details] Bug 1284422 part.1 Remove "TSF: " from log of TSFTextStore.cpp Review request updated; see interdiff: https://reviewboard.mozilla.org/r/62316/diff/1-2/
Attachment #8767933 -
Attachment description: Bug 1284422 part.7 Replace "KeymapWrapper(%p): " with "%p " in nsGtkKeyUtils.cpp → Bug 1284422 part.7 Replace "KeymapWrapper(%p): " with "0x%p " and "%p" with "0x%p" in nsGtkKeyUtils.cpp
Attachment #8767935 -
Attachment description: Bug 1284422 part.9 Remove "ContentCache(InParent|InChild)?: " from log of ContentCache.cpp since the classes are not in same process under current designr?m_kato → Bug 1284422 part.9 Remove "ContentCache(InParent|InChild)?: " from log of ContentCache.cpp since the classes are not in same process under current design
Assignee | ||
Comment 27•8 years ago
|
||
Comment on attachment 8767928 [details] Bug 1284422 part.2 Fix odd indent of MOZ_LOG() in TSFTextStore.cpp (mainly caused by replacing from PR_LOG()) Review request updated; see interdiff: https://reviewboard.mozilla.org/r/62318/diff/1-2/
Assignee | ||
Comment 28•8 years ago
|
||
Comment on attachment 8767929 [details] Bug 1284422 part.3 Remove "IMM: " from log of IMMHandler.cpp Review request updated; see interdiff: https://reviewboard.mozilla.org/r/62320/diff/1-2/
Assignee | ||
Comment 29•8 years ago
|
||
Comment on attachment 8767930 [details] Bug 1284422 part.4 Fix odd indent of MOZ_LOG() in TextInputHandler.mm Review request updated; see interdiff: https://reviewboard.mozilla.org/r/62322/diff/1-2/
Assignee | ||
Comment 30•8 years ago
|
||
Comment on attachment 8767931 [details] Bug 1284422 part.5 Remove "GTKIM: " from log of IMContextWrapper.cpp Review request updated; see interdiff: https://reviewboard.mozilla.org/r/62324/diff/1-2/
Assignee | ||
Comment 31•8 years ago
|
||
Comment on attachment 8767932 [details] Bug 1284422 part.6 Reduce indent of MOZ_LOG() in IMContextWrapper.cpp Review request updated; see interdiff: https://reviewboard.mozilla.org/r/62326/diff/1-2/
Assignee | ||
Comment 32•8 years ago
|
||
Comment on attachment 8767933 [details] Bug 1284422 part.7 Replace "KeymapWrapper(%p): " with "0x%p " and "%p" with "0x%p" in nsGtkKeyUtils.cpp Review request updated; see interdiff: https://reviewboard.mozilla.org/r/62328/diff/1-2/
Assignee | ||
Comment 33•8 years ago
|
||
Comment on attachment 8767934 [details] Bug 1284422 part.8 Fix odd indent of MOZ_LOG() in nsGtkKeyUtils.cpp (mainly caused by replacing from PR_LOG()) Review request updated; see interdiff: https://reviewboard.mozilla.org/r/62330/diff/1-2/
Assignee | ||
Comment 34•8 years ago
|
||
Comment on attachment 8767935 [details] Bug 1284422 part.9 Remove "ContentCache(InParent|InChild)?: " from log of ContentCache.cpp since the classes are not in same process under current design Review request updated; see interdiff: https://reviewboard.mozilla.org/r/62332/diff/1-2/
Assignee | ||
Comment 35•8 years ago
|
||
Comment on attachment 8767936 [details] Bug 1284422 part.10 Remove "IMCO: " from log of IMEContentObserver.cpp Review request updated; see interdiff: https://reviewboard.mozilla.org/r/62334/diff/1-2/
Assignee | ||
Comment 36•8 years ago
|
||
Comment on attachment 8767937 [details] Bug 1284422 part.11 Remove "IMS: " and "IMEStateManager::" from log of IMEStateManager.cpp Review request updated; see interdiff: https://reviewboard.mozilla.org/r/62336/diff/1-2/
Assignee | ||
Comment 37•8 years ago
|
||
https://reviewboard.mozilla.org/r/62328/#review59336 > other pathes use 0x%p, but these fixes use %p. I think that you should replace all %p with 0x%p Done.
Comment 38•8 years ago
|
||
Pushed by masayuki@d-toybox.com: https://hg.mozilla.org/integration/autoland/rev/09d03ee4fd06 part.1 Remove "TSF: " from log of TSFTextStore.cpp r=m_kato https://hg.mozilla.org/integration/autoland/rev/08e730cb01f0 part.2 Fix odd indent of MOZ_LOG() in TSFTextStore.cpp (mainly caused by replacing from PR_LOG()) r=m_kato https://hg.mozilla.org/integration/autoland/rev/ecde910f10a0 part.3 Remove "IMM: " from log of IMMHandler.cpp r=m_kato https://hg.mozilla.org/integration/autoland/rev/532398e5766f part.4 Fix odd indent of MOZ_LOG() in TextInputHandler.mm r=m_kato https://hg.mozilla.org/integration/autoland/rev/20d68ac5d2b9 part.5 Remove "GTKIM: " from log of IMContextWrapper.cpp r=m_kato https://hg.mozilla.org/integration/autoland/rev/924b9f0a3bb0 part.6 Reduce indent of MOZ_LOG() in IMContextWrapper.cpp r=m_kato https://hg.mozilla.org/integration/autoland/rev/34a07555c050 part.7 Replace "KeymapWrapper(%p): " with "0x%p " and "%p" with "0x%p" in nsGtkKeyUtils.cpp r=m_kato https://hg.mozilla.org/integration/autoland/rev/cf238281b89f part.8 Fix odd indent of MOZ_LOG() in nsGtkKeyUtils.cpp (mainly caused by replacing from PR_LOG()) r=m_kato https://hg.mozilla.org/integration/autoland/rev/9a7b0cdc1761 part.9 Remove "ContentCache(InParent|InChild)?: " from log of ContentCache.cpp since the classes are not in same process under current design r=m_kato https://hg.mozilla.org/integration/autoland/rev/b12eb7caa712 part.10 Remove "IMCO: " from log of IMEContentObserver.cpp r=m_kato https://hg.mozilla.org/integration/autoland/rev/70852530cdb5 part.11 Remove "IMS: " and "IMEStateManager::" from log of IMEStateManager.cpp r=m_kato
Comment 39•8 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/09d03ee4fd06 https://hg.mozilla.org/mozilla-central/rev/08e730cb01f0 https://hg.mozilla.org/mozilla-central/rev/ecde910f10a0 https://hg.mozilla.org/mozilla-central/rev/532398e5766f https://hg.mozilla.org/mozilla-central/rev/20d68ac5d2b9 https://hg.mozilla.org/mozilla-central/rev/924b9f0a3bb0 https://hg.mozilla.org/mozilla-central/rev/34a07555c050 https://hg.mozilla.org/mozilla-central/rev/cf238281b89f https://hg.mozilla.org/mozilla-central/rev/9a7b0cdc1761 https://hg.mozilla.org/mozilla-central/rev/b12eb7caa712 https://hg.mozilla.org/mozilla-central/rev/70852530cdb5
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla50
You need to log in
before you can comment on or make changes to this bug.
Description
•