Some of the WKPreferences (API) property getters read like questions
authorandersca@apple.com <andersca@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 7 Jul 2014 19:52:26 +0000 (19:52 +0000)
committerandersca@apple.com <andersca@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 7 Jul 2014 19:52:26 +0000 (19:52 +0000)
https://bugs.webkit.org/show_bug.cgi?id=134678
<rdar://problem/17576847>

Reviewed by Dan Bernstein.

* UIProcess/API/Cocoa/WKPreferences.h:
* UIProcess/API/Cocoa/WKPreferences.mm:
(-[WKPreferences javaScriptIsEnabled]):
(-[WKPreferences javaIsEnabled]):
(-[WKPreferences plugInsAreEnabled]):
(-[WKPreferences isJavaScriptEnabled]): Deleted.
(-[WKPreferences isJavaEnabled]): Deleted.
(-[WKPreferences arePlugInsEnabled]): Deleted.

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

Source/WebKit2/ChangeLog
Source/WebKit2/UIProcess/API/Cocoa/WKPreferences.h
Source/WebKit2/UIProcess/API/Cocoa/WKPreferences.mm

index 516053d..cfa2376 100644 (file)
@@ -1,3 +1,20 @@
+2014-07-07  Anders Carlsson  <andersca@apple.com>
+
+        Some of the WKPreferences (API) property getters read like questions
+        https://bugs.webkit.org/show_bug.cgi?id=134678
+        <rdar://problem/17576847>
+
+        Reviewed by Dan Bernstein.
+
+        * UIProcess/API/Cocoa/WKPreferences.h:
+        * UIProcess/API/Cocoa/WKPreferences.mm:
+        (-[WKPreferences javaScriptIsEnabled]):
+        (-[WKPreferences javaIsEnabled]):
+        (-[WKPreferences plugInsAreEnabled]):
+        (-[WKPreferences isJavaScriptEnabled]): Deleted.
+        (-[WKPreferences isJavaEnabled]): Deleted.
+        (-[WKPreferences arePlugInsEnabled]): Deleted.
+
 2014-07-07  Brady Eidson  <beidson@apple.com>
 
         ServicesOverlayController menus show up in the wrong place.
index 58aabe9..6603a89 100644 (file)
@@ -45,7 +45,7 @@ WK_CLASS_AVAILABLE(10_10, 8_0)
 /*! @abstract A Boolean value indicating whether JavaScript is enabled.
  @discussion The default value is YES.
  */
-@property (nonatomic, getter=isJavaScriptEnabled) BOOL javaScriptEnabled;
+@property (nonatomic) BOOL javaScriptEnabled;
 
 /*! @abstract A Boolean value indicating whether JavaScript can open
  windows without user interaction.
@@ -55,14 +55,14 @@ WK_CLASS_AVAILABLE(10_10, 8_0)
 
 #if !TARGET_OS_IPHONE
 /*! @abstract A Boolean value indicating whether Java is enabled.
- @discussion The default value is YES.
+ @discussion The default value is NO.
  */
-@property (nonatomic, getter=isJavaEnabled) BOOL javaEnabled;
+@property (nonatomic) BOOL javaEnabled;
 
 /*! @abstract A Boolean value indicating whether plug-ins are enabled.
- @discussion The default value is YES.
+ @discussion The default value is NO.
  */
-@property (nonatomic, getter=arePlugInsEnabled) BOOL plugInsEnabled;
+@property (nonatomic) BOOL plugInsEnabled;
 #endif
 
 @end
index 441c54b..1d9248d 100644 (file)
@@ -53,7 +53,7 @@
     _preferences->setMinimumFontSize(minimumFontSize);
 }
 
-- (BOOL)isJavaScriptEnabled
+- (BOOL)javaScriptEnabled
 {
     return _preferences->javaScriptEnabled();
 }
@@ -77,7 +77,7 @@
 
 #if PLATFORM(MAC)
 
-- (BOOL)isJavaEnabled
+- (BOOL)javaEnabled
 {
     return _preferences->javaEnabled();
 }
@@ -87,7 +87,7 @@
     _preferences->setJavaEnabled(javaEnabled);
 }
 
-- (BOOL)arePlugInsEnabled
+- (BOOL)plugInsEnabled
 {
     return _preferences->pluginsEnabled();
 }