Thank you! As somebody who occasionally has had to go digging in that code, this cleanup is much appreciated.
On Thu., Mar. 14, 2019, 04:05 Masayuki Nakano, <masay...@d-toybox.com> wrote: > Hi, > > We had too big, complicated and important method, > `PresShell::HandleEvent()` (*1), and its helper method, > `PresShell::HandleEventInternal()` (*2). Those methods were too risky > to fix everything. Therefore, I redesigned them with a stack class > `PresShell::EventHandler` (*3). This new class is created in the stack > temporarily when `PresShell::HandleEvent()` or similar methods are > called, or handling event is retargeted to different `PresShell`. > > Note that I did not fix found bugs except trivial ones. So, if you need > to check the blame, it must be easier to do it starting from the first > link below since it may not be easy to check moved lines. > > 1. > > https://searchfox.org/mozilla-central/rev/9b614455de573ad1cac56fd1dbac73adde3425bc/layout/base/PresShell.cpp#6464-7122 > 2. > > https://searchfox.org/mozilla-central/rev/9b614455de573ad1cac56fd1dbac73adde3425bc/layout/base/PresShell.cpp#7190-7586 > 3. > > https://searchfox.org/mozilla-central/rev/8ff2cd0a27e3764d9540abdc5a66b2fb1e4e9644/layout/base/PresShell.h#499-1261 > > -- > Masayuki Nakano <masay...@d-toybox.com> > Software Engineer, Mozilla > _______________________________________________ > dev-platform mailing list > dev-platform@lists.mozilla.org > https://lists.mozilla.org/listinfo/dev-platform > _______________________________________________ dev-platform mailing list dev-platform@lists.mozilla.org https://lists.mozilla.org/listinfo/dev-platform
