New issue
Advanced search Search tips

Issue 850144 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Jan 15
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

[EventTiming] Clean up the event pointers in EventTiming

Project Member Reported by maxlg@chromium.org, Jun 6 2018

Issue description

Clean up the event pointers in EventTiming.
 
Ping any updates?
Status: Started (was: Assigned)
Project Member

Comment 3 by bugdroid1@chromium.org, Sep 25

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/914f0969a7737aa14faa6c496650e2c434715ceb

commit 914f0969a7737aa14faa6c496650e2c434715ceb
Author: Liquan(Max) Gu <maxlg@chromium.org>
Date: Tue Sep 25 20:24:33 2018

[EventTiming] Change pointers to references

Initially the events in EventTiming class are pointers, this CL is to change
them to references, since reference is safer to use.

Bug:  850144 
Change-Id: Ia13ef944c143a7bed8096c1faf370163bfa95aaf
Reviewed-on: https://chromium-review.googlesource.com/1244102
Reviewed-by: Dave Tapuska <dtapuska@chromium.org>
Commit-Queue: Liquan (Max) Gǔ <maxlg@chromium.org>
Cr-Commit-Position: refs/heads/master@{#594081}
[modify] https://crrev.com/914f0969a7737aa14faa6c496650e2c434715ceb/third_party/blink/renderer/core/dom/events/event_dispatcher.cc
[modify] https://crrev.com/914f0969a7737aa14faa6c496650e2c434715ceb/third_party/blink/renderer/core/timing/event_timing.cc
[modify] https://crrev.com/914f0969a7737aa14faa6c496650e2c434715ceb/third_party/blink/renderer/core/timing/event_timing.h

Status: Fixed (was: Started)

Sign in to add a comment