Bug 1322576 - [4.0] Fix some coding style issues. r=smaug
authorEugen Sawin <esawin@mozilla.com>
Wed, 15 Feb 2017 23:06:25 +0100
changeset 373330 fb8a931adf8e834ded30116a5bf193c333f33405
parent 373329 162492fcf4a9ecad41d71ff55c8a9341f44411bc
child 373331 6cf3a4f98a622d3c1b40a24d3ae9f486339f8b1c
push id10863
push userjlorenzo@mozilla.com
push dateMon, 06 Mar 2017 23:02:23 +0000
treeherdermozilla-aurora@0931190cd725 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewerssmaug
bugs1322576
milestone54.0a1
Bug 1322576 - [4.0] Fix some coding style issues. r=smaug
docshell/base/nsILoadContext.idl
uriloader/prefetch/OfflineCacheUpdateParent.cpp
--- a/docshell/base/nsILoadContext.idl
+++ b/docshell/base/nsILoadContext.idl
@@ -83,30 +83,33 @@ interface nsILoadContext : nsISupports
    * changed after a document has been loaded in this context.
    */
   attribute boolean useTrackingProtection;
 
 %{C++
   /**
    * De-XPCOMed getter to make call-sites cleaner.
    */
-  bool UsePrivateBrowsing() {
-    bool usingPB;
+  bool UsePrivateBrowsing()
+  {
+    bool usingPB = false;
     GetUsePrivateBrowsing(&usingPB);
     return usingPB;
   }
 
-  bool UseRemoteTabs() {
-    bool usingRT;
+  bool UseRemoteTabs()
+  {
+    bool usingRT = false;
     GetUseRemoteTabs(&usingRT);
     return usingRT;
   }
 
-  bool UseTrackingProtection() {
-    bool usingTP;
+  bool UseTrackingProtection()
+  {
+    bool usingTP = false;
     GetUseTrackingProtection(&usingTP);
     return usingTP;
   }
 %}
 
   /**
    * Set the private browsing state of the load context, meant to be used internally.
    */
--- a/uriloader/prefetch/OfflineCacheUpdateParent.cpp
+++ b/uriloader/prefetch/OfflineCacheUpdateParent.cpp
@@ -144,17 +144,17 @@ OfflineCacheUpdateParent::Schedule(const
         documentURI->Clone(getter_AddRefs(stickURI));
         update->StickDocument(stickURI);
     }
 
     return NS_OK;
 }
 
 NS_IMETHODIMP
-OfflineCacheUpdateParent::UpdateStateChanged(nsIOfflineCacheUpdate *aUpdate, uint32_t state)
+OfflineCacheUpdateParent::UpdateStateChanged(nsIOfflineCacheUpdate* aUpdate, uint32_t state)
 {
     if (mIPCClosed)
         return NS_ERROR_UNEXPECTED;
 
     LOG(("OfflineCacheUpdateParent::StateEvent [%p]", this));
 
     uint64_t byteProgress;
     aUpdate->GetByteProgress(&byteProgress);
@@ -171,17 +171,17 @@ OfflineCacheUpdateParent::UpdateStateCha
 
         Unused << SendFinish(succeeded, isUpgrade);
     }
 
     return NS_OK;
 }
 
 NS_IMETHODIMP
-OfflineCacheUpdateParent::ApplicationCacheAvailable(nsIApplicationCache *aApplicationCache)
+OfflineCacheUpdateParent::ApplicationCacheAvailable(nsIApplicationCache* aApplicationCache)
 {
     if (mIPCClosed)
         return NS_ERROR_UNEXPECTED;
 
     NS_ENSURE_ARG(aApplicationCache);
 
     nsCString cacheClientId;
     aApplicationCache->GetClientID(cacheClientId);
@@ -216,52 +216,52 @@ OfflineCacheUpdateParent::GetTopFrameEle
 
 NS_IMETHODIMP
 OfflineCacheUpdateParent::GetNestedFrameId(uint64_t* aId)
 {
     return NS_ERROR_NOT_IMPLEMENTED;
 }
 
 NS_IMETHODIMP
-OfflineCacheUpdateParent::GetIsContent(bool *aIsContent)
+OfflineCacheUpdateParent::GetIsContent(bool* aIsContent)
 {
     return NS_ERROR_NOT_IMPLEMENTED;
 }
 
 NS_IMETHODIMP
-OfflineCacheUpdateParent::GetUsePrivateBrowsing(bool *aUsePrivateBrowsing)
+OfflineCacheUpdateParent::GetUsePrivateBrowsing(bool* aUsePrivateBrowsing)
 {
     return NS_ERROR_NOT_IMPLEMENTED;
 }
 NS_IMETHODIMP
 OfflineCacheUpdateParent::SetUsePrivateBrowsing(bool aUsePrivateBrowsing)
 {
     return NS_ERROR_NOT_IMPLEMENTED;
 }
 
 NS_IMETHODIMP
 OfflineCacheUpdateParent::SetPrivateBrowsing(bool aUsePrivateBrowsing)
 {
     return NS_ERROR_NOT_IMPLEMENTED;
 }
 
 NS_IMETHODIMP
-OfflineCacheUpdateParent::GetUseRemoteTabs(bool *aUseRemoteTabs)
+OfflineCacheUpdateParent::GetUseRemoteTabs(bool* aUseRemoteTabs)
 {
     return NS_ERROR_NOT_IMPLEMENTED;
 }
 
 NS_IMETHODIMP
 OfflineCacheUpdateParent::SetRemoteTabs(bool aUseRemoteTabs)
 {
     return NS_ERROR_NOT_IMPLEMENTED;
 }
 
 NS_IMETHODIMP
-OfflineCacheUpdateParent::GetIsInIsolatedMozBrowserElement(bool *aIsInIsolatedMozBrowserElement)
+OfflineCacheUpdateParent::GetIsInIsolatedMozBrowserElement(bool* aIsInIsolatedMozBrowserElement)
 {
     NS_ENSURE_TRUE(mLoadingPrincipal, NS_ERROR_UNEXPECTED);
     return mLoadingPrincipal->GetIsInIsolatedMozBrowserElement(aIsInIsolatedMozBrowserElement);
 }
 
 NS_IMETHODIMP
 OfflineCacheUpdateParent::GetOriginAttributes(JS::MutableHandleValue aAttrs)
 {
@@ -272,17 +272,17 @@ OfflineCacheUpdateParent::GetOriginAttri
 
     nsresult rv = mLoadingPrincipal->GetOriginAttributes(cx, aAttrs);
     NS_ENSURE_SUCCESS(rv, rv);
 
     return NS_OK;
 }
 
 NS_IMETHODIMP
-OfflineCacheUpdateParent::GetUseTrackingProtection(bool *aUseTrackingProtection)
+OfflineCacheUpdateParent::GetUseTrackingProtection(bool* aUseTrackingProtection)
 {
     return NS_ERROR_NOT_IMPLEMENTED;
 }
 
 NS_IMETHODIMP
 OfflineCacheUpdateParent::SetUseTrackingProtection(bool aUseTrackingProtection)
 {
     return NS_ERROR_NOT_IMPLEMENTED;