Commit fb895939 authored by alokp@chromium.org's avatar alokp@chromium.org

Made result_filefaceid output optional in FontConfigIPC::Match. There are...

Made result_filefaceid output optional in FontConfigIPC::Match. There are cases when we are only interested in result_family.
BUG=142421

Review URL: https://chromiumcodereview.appspot.com/10836242

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@152247 0039d316-1c4b-4281-b951-d872f2087c98
parent 1ce81502
...@@ -70,7 +70,8 @@ bool FontConfigIPC::Match(std::string* result_family, ...@@ -70,7 +70,8 @@ bool FontConfigIPC::Match(std::string* result_family,
return false; return false;
} }
*result_filefaceid = reply_filefaceid; if (result_filefaceid)
*result_filefaceid = reply_filefaceid;
if (result_family) if (result_family)
*result_family = reply_family; *result_family = reply_family;
......
...@@ -32,8 +32,10 @@ class FontConfigInterface { ...@@ -32,8 +32,10 @@ class FontConfigInterface {
/** Performs config match /** Performs config match
* *
* @param result_family (output) on success, the resulting family name. * @param result_family (optional, set to NULL to ignore, output)
* @param result_filefaceid (output) on success, the resulting fileface id. * on success, set to the resulting family name.
* @param result_filefaceid (optional, set to NULL to ignore, output)
* on success, set to the resulting fileface id.
* @param filefaceid_valid if true, then |filefaceid| is valid * @param filefaceid_valid if true, then |filefaceid| is valid
* @param filefaceid the filefaceid (as returned by this function) * @param filefaceid the filefaceid (as returned by this function)
* which we are trying to match. * which we are trying to match.
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment