Bug 1330496 - Followup patch to remove redundant parenthses. r=aklotz
authorMasatoshi Kimura <VYV03354@nifty.ne.jp>
Wed, 08 Feb 2017 20:33:38 +0900
changeset 342273 87c98844baf6157db5a6b8172ab01bf9a485de71
parent 342272 5ae8504e59996ff6d46a2b0e0d797f4c9ac8cdb4
child 342274 7aa83a5ee9847bafe41ff10c979ee39f53c533d3
push id31347
push userkwierso@gmail.com
push dateFri, 10 Feb 2017 23:23:41 +0000
treeherdermozilla-central@855e6b2f6199 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersaklotz
bugs1330496
milestone54.0a1
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
Bug 1330496 - Followup patch to remove redundant parenthses. r=aklotz MozReview-Commit-ID: DYGbu2S20Ev
ipc/glue/WindowsMessageLoop.cpp
--- a/ipc/glue/WindowsMessageLoop.cpp
+++ b/ipc/glue/WindowsMessageLoop.cpp
@@ -392,17 +392,17 @@ ProcessOrDeferMessage(HWND hwnd,
       return 0;
 
     // We only support a query for our IAccessible or UIA pointers.
     // This should be safe, and needs to be sync.
 #if defined(ACCESSIBILITY)
    case WM_GETOBJECT: {
       if (!::GetPropW(hwnd, k3rdPartyWindowProp)) {
         DWORD objId = static_cast<DWORD>(lParam);
-        if ((objId == OBJID_CLIENT || objId == MOZOBJID_UIAROOT)) {
+        if (objId == OBJID_CLIENT || objId == MOZOBJID_UIAROOT) {
           WNDPROC oldWndProc = (WNDPROC)GetProp(hwnd, kOldWndProcProp);
           if (oldWndProc) {
             return CallWindowProcW(oldWndProc, hwnd, uMsg, wParam, lParam);
           }
         }
       }
       return DefWindowProc(hwnd, uMsg, wParam, lParam);
    }
@@ -1016,17 +1016,17 @@ MessageChannel::WaitForSyncNotifyWithA11
 bool
 MessageChannel::WaitForSyncNotify(bool aHandleWindowsMessages)
 {
   mMonitor->AssertCurrentThreadOwns();
 
   MOZ_ASSERT(gUIThreadId, "InitUIThread was not called!");
 
 #if defined(ACCESSIBILITY)
-  if ((mFlags & REQUIRE_A11Y_REENTRY)) {
+  if (mFlags & REQUIRE_A11Y_REENTRY) {
     MOZ_ASSERT(!(mFlags & REQUIRE_DEFERRED_MESSAGE_PROTECTION));
     return WaitForSyncNotifyWithA11yReentry();
   }
 #endif
 
   // Use a blocking wait if this channel does not require
   // Windows message deferral behavior.
   if (!(mFlags & REQUIRE_DEFERRED_MESSAGE_PROTECTION) || !aHandleWindowsMessages) {