Bug 1524214 - Grab caching capturing content with local variable r=smaug
authorMasayuki Nakano <masayuki@d-toybox.com>
Fri, 01 Feb 2019 20:44:28 +0000
changeset 456528 e53f607940cb5e1db1bb7eeb3c5c0bc6f4b0d608
parent 456527 e2780a098404114db5aea796c87e577ce2c54f05
child 456529 026c691e29c66aa0c3f01c8198b331e9afc26405
push id111656
push userdvarga@mozilla.com
push dateSat, 02 Feb 2019 09:51:54 +0000
treeherdermozilla-inbound@d8cebb3b46cf [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewerssmaug
bugs1524214
milestone67.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 1524214 - Grab caching capturing content with local variable r=smaug Differential Revision: https://phabricator.services.mozilla.com/D18209
layout/base/PresShell.cpp
--- a/layout/base/PresShell.cpp
+++ b/layout/base/PresShell.cpp
@@ -6553,17 +6553,17 @@ nsresult PresShell::EventHandler::Handle
   }
 
   nsIFrame* frame = aFrame;
 
   if (aGUIEvent->IsUsingCoordinates()) {
     // XXX Retrieving capturing content here.  However, some of the following
     //     methods allow to run script.  So, isn't it possible the capturing
     //     content outdated?
-    nsIContent* capturingContent =
+    nsCOMPtr<nsIContent> capturingContent =
         EventHandler::GetCapturingContentFor(aGUIEvent);
     if (GetDocument()) {
       if (aGUIEvent->mClass == eTouchEventClass) {
         Document::UnlockPointer();
       }
 
       AutoWeakFrame weakFrame(frame);
       {  // scope for scriptBlocker.