Address a couple of review comments
authorandersca@apple.com <andersca@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 18 Apr 2014 20:56:13 +0000 (20:56 +0000)
committerandersca@apple.com <andersca@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 18 Apr 2014 20:56:13 +0000 (20:56 +0000)
https://bugs.webkit.org/show_bug.cgi?id=131856

Reviewed by Dan Bernstein.

* UIProcess/API/Cocoa/WKNavigationAction.h:
* UIProcess/API/Cocoa/WKNavigationDelegate.h:
* UIProcess/API/Cocoa/WKPreferences.h:
* UIProcess/API/Cocoa/WKProcessPoolPrivate.h:
* UIProcess/API/Cocoa/WKWebView.h:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@167507 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebKit2/ChangeLog
Source/WebKit2/UIProcess/API/Cocoa/WKNavigationAction.h
Source/WebKit2/UIProcess/API/Cocoa/WKNavigationDelegate.h
Source/WebKit2/UIProcess/API/Cocoa/WKPreferences.h
Source/WebKit2/UIProcess/API/Cocoa/WKProcessPoolPrivate.h
Source/WebKit2/UIProcess/API/Cocoa/WKWebView.h

index 9733e72..a4c7e6f 100644 (file)
@@ -1,5 +1,18 @@
 2014-04-18  Anders Carlsson  <andersca@apple.com>
 
+        Address a couple of review comments
+        https://bugs.webkit.org/show_bug.cgi?id=131856
+
+        Reviewed by Dan Bernstein.
+
+        * UIProcess/API/Cocoa/WKNavigationAction.h:
+        * UIProcess/API/Cocoa/WKNavigationDelegate.h:
+        * UIProcess/API/Cocoa/WKPreferences.h:
+        * UIProcess/API/Cocoa/WKProcessPoolPrivate.h:
+        * UIProcess/API/Cocoa/WKWebView.h:
+
+2014-04-18  Anders Carlsson  <andersca@apple.com>
+
         Handle pluginOrigin being null
         https://bugs.webkit.org/show_bug.cgi?id=131849
         <rdar://problem/16655898>
index a5facea..d745fb4 100644 (file)
@@ -34,9 +34,9 @@
  @abstract the type of action that triggered a possible navigation.
  @constant WKNavigationTypeLinkActivated    A link with an href activated by the user.
  @constant WKNavigationTypeFormSubmitted    A form was submitted.
- @constant WKNavigationTypeBackForward      The user requested an item from the back-forward list.
- @constant WKNavigationTypeReload           The user reloaded a page.
- @constant WKNavigationTypeFormResubmitted  A form as resubmitted (for example by going back, forward or reloading).
+ @constant WKNavigationTypeBackForward      An item from the back-forward list was requested.
+ @constant WKNavigationTypeReload           The page was reloaded.
+ @constant WKNavigationTypeFormResubmitted  A form was resubmitted (for example by going back, forward or reloading).
  @constant WKNavigationTypeOther            Navigation is taking place for some other reason.
  */
 typedef NS_ENUM(NSInteger, WKNavigationType) {
index bc9c59b..6d32a04 100644 (file)
@@ -70,7 +70,7 @@ typedef NS_ENUM(NSInteger, WKNavigationResponsePolicy) {
 /*! @abstract Decides whether a navigation should be allowed or cancelled once its response is known.
  @param webView The WKWebView invoking the delegate method.
  @param navigationResponse A description of the navigation response.
- @param decisionHandler The decision handler that should be call to allow or cancel the load.
+ @param decisionHandler The decision handler that should be called to allow or cancel the load.
  */
 - (void)webView:(WKWebView *)webView decidePolicyForNavigationResponse:(WKNavigationResponse *)navigationResponse decisionHandler:(void (^)(WKNavigationResponsePolicy))decisionHandler;
 
index 849534d..bf9fe7a 100644 (file)
@@ -40,52 +40,52 @@ WK_API_CLASS
 
 /*! @abstract Returns an initialized WKPreferences object.
  @param userDefaultsKeyPrefix The user defaults key prefix.
- @discussion If the userDefaultsKeyPrefix argument is non-nil, it is is prepended to the keys used to store preferences 
+ @discussion If the userDefaultsKeyPrefix argument is non-nil, it is prepended to the keys used to store preferences
  in the user defaults database. If the argument is nil, the preferences object won't save anything to the user defaults database.
  */
 - (instancetype)initWithUserDefaultsKeyPrefix:(NSString *)userDefaultsKeyPrefix WK_DESIGNATED_INITIALIZER;
 
-/*! @abstract The user defaults key prefix
+/*! @abstract The user defaults key prefix.
  */
 @property (nonatomic, readonly) NSString *userDefaultsKeyPrefix;
 
-/*! @abstract The minimum font size in points.
+/*! @abstract The minimum font size in points. Defaults to 0.
  */
 @property (nonatomic) CGFloat minimumFontSize;
 
-/*! @abstract Whether JavaScript is enabled.
+/*! @abstract Whether JavaScript is enabled. Defaults to YES.
  */
 @property (nonatomic, getter=isJavaScriptEnabled) BOOL javaScriptEnabled;
 
-/*! @abstract Whether JavaScript can open windows without user interaction.
+/*! @abstract Whether JavaScript can open windows without user interaction. Defaults to NO on iOS and YES on OS X.
  */
 @property (nonatomic) BOOL javaScriptCanOpenWindowsAutomatically;
 
-/*! @abstract Whether the WKWebView suppresses content rendering until it is fully loaded into memory.
+/*! @abstract Whether the WKWebView suppresses content rendering until it is fully loaded into memory. Defaults to NO.
  */
 @property (nonatomic) BOOL suppressesIncrementalRendering;
 
 #if TARGET_OS_IPHONE
-/*! @abstract Whether HTML5 videos play inline or use the native full-screen controller.
+/*! @abstract Whether HTML5 videos play inline or use the native full-screen controller. Defaults to NO.
  */
 @property (nonatomic) BOOL allowsInlineMediaPlayback;
 
-/*! @abstract Whether HTML5 videos can play automatically or require the user to start playing them.
+/*! @abstract Whether HTML5 videos can play automatically or require the user to start playing them. Defaults to YES.
  */
 @property (nonatomic) BOOL mediaPlaybackRequiresUserAction;
 
-/*! @abstract Whether AirPlay is allowed.
+/*! @abstract Whether AirPlay is allowed. Defaults to YES.
  */
 @property (nonatomic) BOOL mediaPlaybackAllowsAirPlay;
 
 #endif
 
 #if !TARGET_OS_IPHONE
-/*! @abstract Whether Java is enabled.
+/*! @abstract Whether Java is enabled. Defaults to YES.
  */
 @property (nonatomic, getter=isJavaEnabled) BOOL javaEnabled;
 
-/*! abstract Whether plug-ins are enabled.
+/*! abstract Whether plug-ins are enabled. Defaults to YES.
  */
 @property (nonatomic, getter=arePlugInsEnabled) BOOL plugInsEnabled;
 #endif
index 509464a..63bc657 100644 (file)
@@ -32,7 +32,7 @@
 
 @interface WKProcessPool (WKPrivate)
 
-- (instancetype)_initWithConfiguration:(_WKProcessPoolConfiguration *)configuration WK_DESIGNATED_INITIALIZER;
+- (instancetype)_initWithConfiguration:(_WKProcessPoolConfiguration *)configuration __attribute__((objc_method_family(init))) WK_DESIGNATED_INITIALIZER;
 
 @property (nonatomic, readonly) _WKProcessPoolConfiguration *_configuration;
 
index e888d89..45cc16c 100644 (file)
@@ -123,12 +123,12 @@ WK_API_CLASS
 /*! @abstract Whether there's a back item in the back-forward list that can be navigated to.
  @seealso backForwardList.
  */
-@property (readonly) BOOL canGoBack;
+@property (nonatomic, readonly) BOOL canGoBack;
 
 /*! @abstract Whether there's a forward item in the back-forward list that can be navigated to.
  @seealso backForwardList.
  */
-@property (readonly) BOOL canGoForward;
+@property (nonatomic, readonly) BOOL canGoForward;
 
 /*! @abstract Navigates to the back item in the back-forward list.
  @result A new navigation to the requested item, or nil if there is no back item in the back-forward list.
@@ -154,7 +154,7 @@ WK_API_CLASS
  */
 - (void)stopLoading;
 
-/*! @abstract Whether horizontal swipe gestures will trigger back-forward list navigations.
+/*! @abstract Whether horizontal swipe gestures will trigger back-forward list navigations. Defaults to NO.
  */
 @property (nonatomic) BOOL allowsBackForwardNavigationGestures;
 
@@ -165,12 +165,12 @@ WK_API_CLASS
 #endif
 
 #if !TARGET_OS_IPHONE
-/* @abstract Whether magnify gestures will change the WKWebView magnification.
+/* @abstract Whether magnify gestures will change the WKWebView magnification. Defaults to NO.
  @discussion It is possible to set the magnification property even if allowsMagnify is set to NO.
  */
 @property (nonatomic) BOOL allowsMagnification;
 
-/* @abstract The amount by which the page content is currently scaled. The default value is 1.0.
+/* @abstract The amount by which the page content is currently scaled. Defaults to 1.0.
  */
 @property (nonatomic) CGFloat magnification;