Backed out changeset 0036938db4fe (bug 1441598) - for accidental printf.
authorBob Owen <bobowencode@gmail.com>
Mon, 05 Mar 2018 08:41:00 +0000
changeset 406522 bb6db24a20dd8c081309b00c11d89f2b45aef6d7
parent 406521 0036938db4fe34793e96184998e5df543a6a4fec
child 406523 2219e0ec9d4f1e89dba54b3dfe213d2fc675ae75
push id100451
push userbobowencode@gmail.com
push dateMon, 05 Mar 2018 08:41:37 +0000
treeherdermozilla-inbound@2219e0ec9d4f [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
bugs1441598
milestone60.0a1
backs out0036938db4fe34793e96184998e5df543a6a4fec
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Backed out changeset 0036938db4fe (bug 1441598) - for accidental printf.
security/sandbox/chromium/base/win/scoped_handle.cc
toolkit/components/printingui/ipc/PrintProgressDialogParent.cpp
toolkit/components/printingui/ipc/PrintProgressDialogParent.h
--- a/security/sandbox/chromium/base/win/scoped_handle.cc
+++ b/security/sandbox/chromium/base/win/scoped_handle.cc
@@ -85,20 +85,17 @@ class ActiveVerifier {
                              const void* pc1, const void* pc2);
   virtual void StopTracking(HANDLE handle, const void* owner,
                             const void* pc1, const void* pc2);
   virtual void Disable();
   virtual void OnHandleBeingClosed(HANDLE handle);
   virtual HMODULE GetModule() const;
 
  private:
-   ~ActiveVerifier() 
-   {
-     printf("wibble: ~ActiveVerifier\n");
-   }  // Not implemented.
+  ~ActiveVerifier();  // Not implemented.
 
   static void InstallVerifier();
 
   base::debug::StackTrace creation_stack_;
   bool enabled_;
   base::ThreadLocalBoolean closing_;
   NativeLock* lock_;
   HandleMap map_;
--- a/toolkit/components/printingui/ipc/PrintProgressDialogParent.cpp
+++ b/toolkit/components/printingui/ipc/PrintProgressDialogParent.cpp
@@ -97,31 +97,22 @@ PrintProgressDialogParent::Recv__delete_
 NS_IMETHODIMP
 PrintProgressDialogParent::Observe(nsISupports *aSubject, const char *aTopic,
                                    const char16_t *aData)
 {
   if (mActive) {
     if (aTopic) {
       if (!strcmp(aTopic, "cancelled")) {
         Unused << SendCancelledCurrentJob();
-        if (!mDialogOpenedSent) {
-          // We haven't already called SendDialogOpened, so call it now or it
-          // might never get sent and block the child from new printing requests.
-          // Also set mActive to false because we don't want to send it twice
-          // and our PrintProgressDialogChild will get deleted anyway.
-          Unused << SendDialogOpened();
-          mActive = false;
-        }
       } else if (!strcmp(aTopic, "completed")) {
         // Once printing is complete don't send any messages to the child.
         mActive = false;
       }
     } else {
       Unused << SendDialogOpened();
-      mDialogOpenedSent = true;
     }
   } else {
     NS_WARNING("The print progress dialog finished opening, but communications "
                "with the child have been closed.");
   }
 
   return NS_OK;
 }
--- a/toolkit/components/printingui/ipc/PrintProgressDialogParent.h
+++ b/toolkit/components/printingui/ipc/PrintProgressDialogParent.h
@@ -51,15 +51,14 @@ public:
   Recv__delete__() override;
 
 private:
   virtual ~PrintProgressDialogParent();
 
   nsCOMPtr<nsIWebProgressListener> mWebProgressListener;
   nsCOMPtr<nsIPrintProgressParams> mPrintProgressParams;
   bool mActive;
-  bool mDialogOpenedSent = false;
 };
 
 } // namespace embedding
 } // namespace mozilla
 
 #endif