Remove unnecessary updateGraphicsContext calls.

Remove unnecessary updateGraphicsContext calls.

These don't do anything, since all it does is update the fill color, which is
not even used in drawHighlightForText (it takes its own color argument).

After this, nearly all updateGraphicsContext calls correspond to a
paintTextWithShadows call, which opens some refactoring options.

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=180376

[jbroman, 2014-08-15 14:23:34]

This makes it easier to change from doing:

updateGraphicsContext(...)
paintText(...)
context->clearDrawLooper()

to having one method both set up and clean up state, calling through to an
underlying method, as you suggested in:
https://codereview.chromium.org/472083002/

[f(malita), 2014-08-15 14:32:01]

lgtm!

[jbroman, 2014-08-15 14:36:44]

The CQ bit was checked by jbroman@chromium.org

[commit-bot: I haz the power, 2014-08-15 14:38:55]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/jbroman@chromium.org/477143002/1

[commit-bot: I haz the power, 2014-08-15 15:31:40]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  win_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/win_blink_rel/builds/22947)

[commit-bot: I haz the power, 2014-08-15 16:11:42]

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

[commit-bot: I haz the power, 2014-08-15 16:11:43]

Try jobs failed on following builders:
  win_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/win_blink_rel/builds/22960)

[jbroman, 2014-08-15 16:46:03]

The CQ bit was checked by jbroman@chromium.org

[commit-bot: I haz the power, 2014-08-15 16:46:30]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/jbroman@chromium.org/477143002/1

[commit-bot: I haz the power, 2014-08-15 17:20:41]

Committed patchset #1 (1) as 180376