Remove obsolete for_web_contents parameter in FontRenderParamsQuery.

Remove obsolete for_web_contents parameter in FontRenderParamsQuery.

BUG=143619
Committed: https://crrev.com/575097ce8654423b32a15a6cd630398beb0734d3
Cr-Commit-Position: refs/heads/master@{#323920}

[stapelberg, 2015-04-05 19:55:41]

The CQ bit was checked by stapelberg@google.com to run a CQ dry run

[commit-bot: I haz the power, 2015-04-05 19:55:45]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1063603003/1

[commit-bot: I haz the power, 2015-04-05 19:55:46]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-04-05 19:55:47]

No LGTM from a valid reviewer yet. Only full committers are accepted.
Even if an LGTM may have been provided, it was from a non-committer,
_not_ a full super star committer.
See http://www.chromium.org/getting-involved/become-a-committer
Note that this has nothing to do with OWNERS files.

[stapelberg, 2015-04-05 19:59:56]

stapelberg@google.com changed reviewers:
+ jam@chromium.org, sadrul@chromium.org

[stapelberg, 2015-04-05 20:00:25]

Removing this now-unused flag as requested in
https://codereview.chromium.org/1020263004/

[Daniel Erat, 2015-04-06 12:23:28]

derat@chromium.org changed reviewers:
+ derat@chromium.org, msw@chromium.org

[Daniel Erat, 2015-04-06 12:23:30]

lgtm with nits

https://codereview.chromium.org/1063603003/diff/1/ui/gfx/font_render_params.h
File ui/gfx/font_render_params.h (right):

https://codereview.chromium.org/1063603003/diff/1/ui/gfx/font_render_params.h...
ui/gfx/font_render_params.h:70: explicit FontRenderParamsQuery();
nit: remove 'explicit'

https://codereview.chromium.org/1063603003/diff/1/ui/gfx/font_render_params_l...
File ui/gfx/font_render_params_linux_unittest.cc (right):

https://codereview.chromium.org/1063603003/diff/1/ui/gfx/font_render_params_l...
ui/gfx/font_render_params_linux_unittest.cc:143: FontRenderParamsQuery query();
nit: remove parentheses here and below

https://codereview.chromium.org/1063603003/diff/1/ui/gfx/platform_font_mac.mm
File ui/gfx/platform_font_mac.mm (right):

https://codereview.chromium.org/1063603003/diff/1/ui/gfx/platform_font_mac.mm...
ui/gfx/platform_font_mac.mm:173: FontRenderParamsQuery query();
nit: remove parentheses

[Daniel Erat, 2015-04-06 12:24:26]

oh, and please add a BUG= line to the change description with the ID of the bug
that's driving this change.

[stapelberg, 2015-04-06 13:14:48]

https://codereview.chromium.org/1063603003/diff/1/ui/gfx/font_render_params.h
File ui/gfx/font_render_params.h (right):

https://codereview.chromium.org/1063603003/diff/1/ui/gfx/font_render_params.h...
ui/gfx/font_render_params.h:70: explicit FontRenderParamsQuery();
On 2015/04/06 12:23:29, Daniel Erat wrote:
> nit: remove 'explicit'

Done.

https://codereview.chromium.org/1063603003/diff/1/ui/gfx/font_render_params_l...
File ui/gfx/font_render_params_linux_unittest.cc (right):

https://codereview.chromium.org/1063603003/diff/1/ui/gfx/font_render_params_l...
ui/gfx/font_render_params_linux_unittest.cc:143: FontRenderParamsQuery query();
On 2015/04/06 12:23:29, Daniel Erat wrote:
> nit: remove parentheses here and below

Done.

https://codereview.chromium.org/1063603003/diff/1/ui/gfx/platform_font_mac.mm
File ui/gfx/platform_font_mac.mm (right):

https://codereview.chromium.org/1063603003/diff/1/ui/gfx/platform_font_mac.mm...
ui/gfx/platform_font_mac.mm:173: FontRenderParamsQuery query();
On 2015/04/06 12:23:29, Daniel Erat wrote:
> nit: remove parentheses

Done.

[stapelberg, 2015-04-06 13:14:56]

The CQ bit was checked by stapelberg@google.com to run a CQ dry run

[stapelberg, 2015-04-06 13:14:57]

The patchset sent to the CQ was uploaded after l-g-t-m from derat@chromium.org
Link to the patchset: https://codereview.chromium.org/1063603003/#ps20001
(title: "Remove obsolete for_web_contents parameter in FontRenderParamsQuery.")

[commit-bot: I haz the power, 2015-04-06 13:15:04]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1063603003/20001

[commit-bot: I haz the power, 2015-04-06 13:24:13]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-04-06 13:24:14]

Try jobs failed on following builders:
  chromium_presubmit on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[jam, 2015-04-06 16:00:42]

lgtm

[stapelberg, 2015-04-06 18:02:26]

The CQ bit was checked by stapelberg@google.com

[commit-bot: I haz the power, 2015-04-06 18:03:21]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1063603003/20001

[commit-bot: I haz the power, 2015-04-06 18:31:55]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-04-06 18:36:51]

Patchset 2 (id:??) landed as
https://crrev.com/575097ce8654423b32a15a6cd630398beb0734d3
Cr-Commit-Position: refs/heads/master@{#323920}