WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
185100
Add logging when SpringBoard enables WebThread
https://bugs.webkit.org/show_bug.cgi?id=185100
Summary
Add logging when SpringBoard enables WebThread
David Kilzer (:ddkilzer)
Reported
2018-04-27 17:13:18 PDT
Add logging when SpringBoard enables WebThread. <
rdar://problem/39746542
>
Attachments
Patch v1
(5.57 KB, patch)
2018-04-27 17:19 PDT
,
David Kilzer (:ddkilzer)
no flags
Details
Formatted Diff
Diff
Patch v2
(5.82 KB, patch)
2018-04-27 20:30 PDT
,
David Kilzer (:ddkilzer)
no flags
Details
Formatted Diff
Diff
Show Obsolete
(1)
View All
Add attachment
proposed patch, testcase, etc.
David Kilzer (:ddkilzer)
Comment 1
2018-04-27 17:19:08 PDT
Created
attachment 339043
[details]
Patch v1
Daniel Bates
Comment 2
2018-04-27 19:34:59 PDT
Comment on
attachment 339043
[details]
Patch v1 View in context:
https://bugs.webkit.org/attachment.cgi?id=339043&action=review
> Source/WTF/ChangeLog:5 > + <
rdar://problem/39746542
>
I assume if I were to look at this bug that it would explain why it is beneficial to add this logging.
> Source/WTF/wtf/Platform.h:-1266 > -/* FIXME: Enable USE_OS_LOG when building with the public iOS 10 SDK once we fix <
rdar://problem/27758343
>. */
Not at my computer. I am assuming we fixed the referenced radar and hence we can enable this. The ChangeLog didn’t mention this fact and EWS failed to compile this patch for iOS due to another issue below.
> Source/WebCore/platform/ios/wak/WebCoreThread.mm:880 > + RELEASE_LOG_FAULT(Threading, "SpringBoard enabled WebThread.");
According to the EWS we don’t have a logging channel called Threading.
David Kilzer (:ddkilzer)
Comment 3
2018-04-27 20:14:16 PDT
Comment on
attachment 339043
[details]
Patch v1 View in context:
https://bugs.webkit.org/attachment.cgi?id=339043&action=review
>> Source/WTF/ChangeLog:5 >> + <
rdar://problem/39746542
> > > I assume if I were to look at this bug that it would explain why it is beneficial to add this logging.
Correct. Added more detail just now as well.
>> Source/WTF/wtf/Platform.h:-1266 >> -/* FIXME: Enable USE_OS_LOG when building with the public iOS 10 SDK once we fix <
rdar://problem/27758343
>. */ > > Not at my computer. I am assuming we fixed the referenced radar and hence we can enable this. The ChangeLog didn’t mention this fact and EWS failed to compile this patch for iOS due to another issue below.
Yes, it's fixed. I'll add a comment to the ChangeLog.
>> Source/WebCore/platform/ios/wak/WebCoreThread.mm:880 >> + RELEASE_LOG_FAULT(Threading, "SpringBoard enabled WebThread."); > > According to the EWS we don’t have a logging channel called Threading.
Weird. I thought it was defined here: Source/WebCore/platform/Logging.h Maybe I forgot to #include that.
David Kilzer (:ddkilzer)
Comment 4
2018-04-27 20:30:15 PDT
Created
attachment 339053
[details]
Patch v2
David Kilzer (:ddkilzer)
Comment 5
2018-04-27 20:38:37 PDT
(In reply to David Kilzer (:ddkilzer) from
comment #4
)
> Created
attachment 339053
[details]
> Patch v2
Added missing #import "Logging.h" to WebCoreThread.mm to fix the iOS build failures. Also updated Source/WTF/ChangeLog with info about the fixed radar.
WebKit Commit Bot
Comment 6
2018-04-27 22:53:09 PDT
Comment on
attachment 339053
[details]
Patch v2 Clearing flags on attachment: 339053 Committed
r231130
: <
https://trac.webkit.org/changeset/231130
>
WebKit Commit Bot
Comment 7
2018-04-27 22:53:10 PDT
All reviewed patches have been landed. Closing bug.
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug