Bug 231896 - Web Inspector: Remove Source/WebInspectorUI/UserInterface/Controllers/WebInspectorExtensionController.js.orig
Summary: Web Inspector: Remove Source/WebInspectorUI/UserInterface/Controllers/WebInsp...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Web Inspector (show other bugs)
Version: WebKit Nightly Build
Hardware: All All
: P2 Normal
Assignee: David Kilzer (:ddkilzer)
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2021-10-18 08:53 PDT by David Kilzer (:ddkilzer)
Modified: 2021-10-18 18:44 PDT (History)
6 users (show)

See Also:


Attachments
Differences between files (do not commit) (5.72 KB, patch)
2021-10-18 08:54 PDT, David Kilzer (:ddkilzer)
no flags Details | Formatted Diff | Diff
Patch v1 (10.77 KB, patch)
2021-10-18 08:56 PDT, David Kilzer (:ddkilzer)
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description David Kilzer (:ddkilzer) 2021-10-18 08:53:48 PDT
Source/WebInspectorUI/UserInterface/Controllers/WebInspectorExtensionController.js.orig was committed in r283857 and modified in r283859, but still has some differences from WebInspectorExtensionController.js.

It seems like it could be removed, but someone with knowledge of this source should check to see that it's okay to remove it without any further changes to WebInspectorExtensionController.js.

Found while running `find Source -name \*.orig` to look for local backup files left by patch(1) command.
Comment 1 David Kilzer (:ddkilzer) 2021-10-18 08:54:42 PDT
Created attachment 441609 [details]
Differences between files (do not commit)
Comment 2 David Kilzer (:ddkilzer) 2021-10-18 08:55:17 PDT
(In reply to David Kilzer (:ddkilzer) from comment #1)
> Created attachment 441609 [details]
> Differences between files (do not commit)

$ diff -u Source/WebInspectorUI/UserInterface/Controllers/WebInspectorExtensionController.js Source/WebInspectorUI/UserInterface/Controllers/WebInspectorExtensionController.js.orig > bug-231896-WebInspectorExtensionController.js.orig.diff
Comment 3 David Kilzer (:ddkilzer) 2021-10-18 08:56:49 PDT
Created attachment 441610 [details]
Patch v1
Comment 4 EWS 2021-10-18 18:43:09 PDT
Committed r284422 (243190@main): <https://commits.webkit.org/243190@main>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 441610 [details].
Comment 5 Radar WebKit Bug Importer 2021-10-18 18:44:18 PDT
<rdar://problem/84397260>