Closed Bug 1661281 Opened 4 years ago Closed 4 years ago

Format the body of an if statement on a different line than the condition in nsFocusManager

Categories

(Core :: DOM: UI Events & Focus Handling, task)

task

Tracking

()

RESOLVED FIXED
82 Branch
Tracking Status
firefox82 --- fixed

People

(Reporter: hsivonen, Assigned: hsivonen)

Details

Attachments

(1 file)

If you look at

  if (clearFirstFocusEvent) mFirstFocusEvent = nullptr;

in Pernosco, you don't see from the margin if the branch was taken.

However, if you look at

  if (clearFirstFocusEvent) {
    mFirstFocusEvent = nullptr;
  }

you do.

Pushed by hsivonen@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/deda0fae36aa Use standard if formatting in nsFocusManager.cpp. r=smaug
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → 82 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: