Issue 630349 link

Starred by 2 users

Issue metadata

Status:	Fixed
Owner:	drott@chromium.org
Closed:	Sep 2016
Cc:	e...@chromium.org
EstimatedDays:	----
NextAction:	----
OS:	All
Pri:	3
Type:	Bug

Consider splitting getGlyphWidthAndExtentsForHarfBuzz into two functions

Project Member Reported by drott@chromium.org, Jul 21 2016

Issue description

eae@ suggests in https://codereview.chromium.org/1980913002/ to split this function since it's never called with both arguments, allowing us to save an extra branching in this rather hot function during shaping.

Comment 1 by drott@chromium.org, Sep 8 2016

Status: Fixed (was: Assigned)

This was done in https://chromium.googlesource.com/chromium/src/+/994c65ef57ec7c84b18fd66d7b8b85e5a2dc29bd

►

Issue 630349 link

Issue metadata

Consider splitting getGlyphWidthAndExtentsForHarfBuzz into two functions

Issue description

Comment 1 by drott@chromium.org, Sep 8 2016

Comment 1 Deleted

Sign in to add a comment