Bug 1310101 - Remove nsILoadContext::IsAppOfType; r=baku

docshell/base/LoadContext.cpp

@@ -98,15 +98,6 @@ LoadContext::GetNestedFrameId(uint64_t* aId)
   return NS_OK;
 }
 
-NS_IMETHODIMP
-LoadContext::IsAppOfType(uint32_t, bool*)
-{
-  MOZ_ASSERT(mIsNotNull);
-
-  // don't expect we need this in parent (Thunderbird/SeaMonkey specific?)
-  return NS_ERROR_UNEXPECTED;
-}
-
 NS_IMETHODIMP
 LoadContext::GetIsContent(bool* aIsContent)
 {

docshell/base/nsDocShell.cpp

@@ -13556,24 +13556,6 @@ nsDocShell::GetNestedFrameId(uint64_t* aId)
   return NS_OK;
 }
 
-NS_IMETHODIMP
-nsDocShell::IsAppOfType(uint32_t aAppType, bool* aIsOfType)
-{
-  RefPtr<nsDocShell> shell = this;
-  while (shell) {
-    uint32_t type;
-    shell->GetAppType(&type);
-    if (type == aAppType) {
-      *aIsOfType = true;
-      return NS_OK;
-    }
-    shell = shell->GetParentDocshell();
-  }
-
-  *aIsOfType = false;
-  return NS_OK;
-}
-
 NS_IMETHODIMP
 nsDocShell::IsTrackingProtectionOn(bool* aIsTrackingProtectionOn)
 {

docshell/base/nsDocShell.h

@@ -224,7 +224,6 @@ public:
   NS_IMETHOD GetTopWindow(mozIDOMWindowProxy**) override;
   NS_IMETHOD GetTopFrameElement(nsIDOMElement**) override;
   NS_IMETHOD GetNestedFrameId(uint64_t*) override;
-  NS_IMETHOD IsAppOfType(uint32_t, bool*) override;
   NS_IMETHOD GetIsContent(bool*) override;
   NS_IMETHOD GetUsePrivateBrowsing(bool*) override;
   NS_IMETHOD SetUsePrivateBrowsing(bool) override;

docshell/base/nsILoadContext.idl

@@ -60,17 +60,6 @@ interface nsILoadContext : nsISupports
    */
   readonly attribute unsigned long long nestedFrameId;
 
-  /**
-   * Check whether the load is happening in a particular type of application.
-   *
-   * @param an application type.  For now, the constants to be passed here are
-   *        the nsIDocShell APP_TYPE_* constants.
-   *
-   * @return whether there is some ancestor of the associatedWindow that is of
-   *         the given app type.
-   */
-  boolean isAppOfType(in unsigned long appType);
-
   /**
    * True if the load context is content (as opposed to chrome).  This is
    * determined based on the type of window the load is performed in, NOT based

dom/apps/AppsUtils.jsm

@@ -157,10 +157,6 @@ this.AppsUtils = {
        usePrivateBrowsing: false,
        isContent: false,
 
-       isAppOfType: function(appType) {
-         throw Cr.NS_ERROR_NOT_IMPLEMENTED;
-       },
-
        QueryInterface: XPCOMUtils.generateQI([Ci.nsILoadContext,
                                               Ci.nsIInterfaceRequestor,
                                               Ci.nsISupports]),

dom/ipc/TabParent.cpp

@@ -3189,7 +3189,6 @@ public:
     return NS_OK;
   }
   NS_IMETHOD GetNestedFrameId(uint64_t*) NO_IMPL
-  NS_IMETHOD IsAppOfType(uint32_t, bool*) NO_IMPL
   NS_IMETHOD GetIsContent(bool*) NO_IMPL
   NS_IMETHOD GetUsePrivateBrowsing(bool*) NO_IMPL
   NS_IMETHOD SetUsePrivateBrowsing(bool) NO_IMPL

uriloader/prefetch/OfflineCacheUpdateParent.cpp

@@ -221,12 +221,6 @@ OfflineCacheUpdateParent::GetNestedFrameId(uint64_t* aId)
     return NS_ERROR_NOT_IMPLEMENTED;
 }
 
-NS_IMETHODIMP
-OfflineCacheUpdateParent::IsAppOfType(uint32_t appType, bool *_retval)
-{
-    return NS_ERROR_NOT_IMPLEMENTED;
-}
-
 NS_IMETHODIMP
 OfflineCacheUpdateParent::GetIsContent(bool *aIsContent)
 {