Issue 1109 - Support notifications

src/engine/NotificationWindow.cpp

+/*
+ * This file is part of Adblock Plus <https://adblockplus.org/>,
+ * Copyright (C) 2006-2015 Eyeo GmbH
+ *
+ * Adblock Plus is free software: you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License version 3 as
+ * published by the Free Software Foundation.
+ *
+ * Adblock Plus is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with Adblock Plus.  If not, see <http://www.gnu.org/licenses/>.
+ */
+
 #include <cassert>
 #include "NotificationWindow.h"
 #include "../shared/Utils.h"
 #include <algorithm>
 #include <fstream>
 #include "../shared/MsHTMLUtils.h"
 
 // it is taken from src/plugin/Resource.h
 #define IDI_ICON_ENABLED 301
 
@@ skipping 107 matching lines @@
   assert(linkIDCounter == m_links.size() && "The amount of links in the text is different from the amount of provided links");
   m_htmlPage = ReplaceMulti(m_htmlPage, L"<!--Title-->", ToUtf16String(notification.GetTitle()));
   m_htmlPage = ReplaceMulti(m_htmlPage, L"<!--Body-->", body);
 }
 
 NotificationWindow::~NotificationWindow()
 {
 }
 
 LRESULT NotificationWindow::OnCreate(const CREATESTRUCT* /*createStruct*/) {
+  {
+    DCHandle hdc(GetDC());
+    m_dpi = hdc.GetDeviceCaps(LOGPIXELSX);
+  }
   m_bgColor.CreateSolidBrush(RGB(255, 255, 255));
 
   CRect iconRect(CPoint(0, 0), CSize(kIconSize + 2 * kIconPadding, kIconSize + 2 * kIconPadding));
   m_icon.Create(m_hWnd, DPIAware(iconRect), nullptr, WS_CHILD | WS_VISIBLE | SS_BITMAP | SS_CENTERIMAGE);
   LoadABPIcon();
 
   m_axIE.Create(m_hWnd, DPIAware(CRect(CPoint(iconRect.right, 0), CSize(kWindowWidth - iconRect.right, kWindowHeight))),
     L"", WS_CHILD | WS_VISIBLE, 0, kHTMLDocumentCtrlID);
   m_axIE.CreateControl((L"mshtml:" + m_htmlPage).c_str());
   ATL::CComPtr<IAxWinAmbientDispatch> axWinAmbient;
@@ skipping 135 matching lines @@
   ::DrawIconEx(tmpDC, 0, 0, hIcon, tmpRect.Width(), tmpRect.Height(), 0, nullptr, DI_NORMAL);
   m_iconImg.Attach(bitmap.Detach());
   tmpDC.SelectBitmap(prevBitmap);
   m_icon.SetBitmap(m_iconImg);
 }
 
 POINT NotificationBorderWindow::GetWindowCoordinates() {
   HMONITOR primaryMonitor = MonitorFromWindow(m_hWnd, MONITOR_DEFAULTTOPRIMARY);
   {
     DCHandle hdc(GetDC());
     m_dpi = hdc.GetDeviceCaps(LOGPIXELSX);

[Oleksandr, 2015/08/18 11:47:40]

Is this still needed, seeing that we now get dpi in OnCreate?

[sergei, 2015/08/18 11:58:40]

It's a little bit odd. `GetWindowCoordinates` is a member of
`NotificationBorderWindow` and is being called from
`NotificationBorderWindow::OnCreate`. In the latest patch we are obtaining DPI
for `NotificationWindow`.

   }
   MONITORINFO monitorInfo = {};
   monitorInfo.cbSize = sizeof(monitorInfo);
   GetMonitorInfo(primaryMonitor, &monitorInfo);
   int windowX = monitorInfo.rcWork.right - DPIAware(kWindowWidth + kWindowMarginRight);
   int windowY = monitorInfo.rcWork.bottom - DPIAware(kWindowHeight + kWindowMarginBottom);
   POINT coords = {windowX, windowY};
   return coords;
 }
 
@@ skipping 47 matching lines @@
 {
   DestroyWindow();
   return 0;
 }
 
 void NotificationBorderWindow::OnFinalMessage(HWND) {
   if (!!m_onDestroyedCallback) {
     m_onDestroyedCallback();
   }
 }