Bug 1676760 Comment 0 Edit History

Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.

Original comment by
on 
Apparently some of the refresh driver vsync code that has "ParentProcess" in it is now used in the content process, as of bug 1641033.

See bug 1641033 comment 11:

(In reply to Olli Pettay [:smaug] from bug 1641033 comment #11)
> And don't now also child processes use code which is named like "ParentProcessVsyncNotifier", or am I missing something obvious?

Furthermore, bug 1661881 [added](https://hg.mozilla.org/integration/autoland/rev/c8ad715374b3) this particularly strange hunk of code:

```
      if (XRE_IsContentProcess()) {
        NotifyParentProcessVsync();
        return true;
      }
```

This should be cleaned up and the naming should be fixed.
Revision 1 by
on 
Apparently some of the refresh driver vsync code that has "ParentProcess" in its name is now used in the content process, as of bug 1641033.

See bug 1641033 comment 11:

(In reply to Olli Pettay [:smaug] from bug 1641033 comment #11)
> And don't now also child processes use code which is named like "ParentProcessVsyncNotifier", or am I missing something obvious?

Furthermore, bug 1661881 [added](https://hg.mozilla.org/integration/autoland/rev/c8ad715374b3) this particularly strange hunk of code:

```
      if (XRE_IsContentProcess()) {
        NotifyParentProcessVsync();
        return true;
      }
```

This should be cleaned up and the naming should be fixed.

Back to Bug 1676760 Comment 0