Bug 80308 - Privately expose WKWebInspectorProxyObjCAdapter so Safari can add categories
Summary: Privately expose WKWebInspectorProxyObjCAdapter so Safari can add categories
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit2 (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Timothy Hatcher
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-03-05 11:44 PST by Timothy Hatcher
Modified: 2012-03-05 13:48 PST (History)
6 users (show)

See Also:

Attachments
Proposed Change (11.26 KB, patch)
2012-03-05 11:48 PST, Timothy Hatcher 		no flags Details | Formatted Diff | Diff
Proposed Change (round 2) (11.56 KB, patch)
2012-03-05 13:16 PST, Timothy Hatcher 		sullivan: review+
timothy: commit-queue-
Details | Formatted Diff | Diff
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Timothy Hatcher 2012-03-05 11:44:07 PST 
The WKWebInspectorProxyObjCAdapter class impelments actions that are used by the Develop menu. This is hard to support and makes changing the Develop menu difficult. We shoudl expose WKWebInspectorProxyObjCAdapter so Safari can add the methods it needs.
Comment 1 Timothy Hatcher 2012-03-05 11:48:36 PST 
Created attachment 130172 [details]
Proposed Change
Comment 2 WebKit Review Bot 2012-03-05 11:52:04 PST 
Attachment 130172 [details] did not pass style-queue:

Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Source/WebKit2/ChangeLog', u'Source/WebKit..." exit_code: 1
Source/WebKit2/UIProcess/API/C/mac/WKInspectorPrivateMac.h:26:  #ifndef header guard has wrong style, please use: WKInspectorPrivateMac_h  [build/header_guard] [5]
Total errors found: 1 in 4 files


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 3 Timothy Hatcher 2012-03-05 11:53:55 PST 
(In reply to comment #2)
> Attachment 130172 [details] did not pass style-queue:
> 
> Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Source/WebKit2/ChangeLog', u'Source/WebKit..." exit_code: 1
> Source/WebKit2/UIProcess/API/C/mac/WKInspectorPrivateMac.h:26:  #ifndef header guard has wrong style, please use: WKInspectorPrivateMac_h  [build/header_guard] [5]
> Total errors found: 1 in 4 files
> 
> 
> If any of these errors are false positives, please file a bug against check-webkit-style.

Fixed locally.
Comment 4 Timothy Hatcher 2012-03-05 13:16:00 PST 
Created attachment 130192 [details]
Proposed Change (round 2)

With header guard fixed and inspectorRef as a @property.
Comment 5 WebKit Review Bot 2012-03-05 13:18:07 PST 
Attachment 130192 [details] did not pass style-queue:

Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Source/WebKit2/ChangeLog', u'Source/WebKit..." exit_code: 1
Source/WebKit2/UIProcess/API/C/mac/WKInspectorPrivateMac.h:38:  Extra space before ( in function call  [whitespace/parens] [4]
Total errors found: 1 in 4 files


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 6 Timothy Hatcher 2012-03-05 13:47:47 PST 
r109795