Fix link errors with Skia in libui_touch_selection.so

Fix link errors with Skia in libui_touch_selection.so

When building a debug version of Chromium in Linux with
component=shared_library build flag, there are link errors as follows:

error: undefined reference to 'SkDebugf_FileLine'
error: undefined reference to 'SkToS32'

BUG=None

Committed: https://crrev.com/64b0e3fa8022ac6932b72e2f86784b83ab03139b
Cr-Commit-Position: refs/heads/master@{#329745}

[joone, 2015-05-13 20:19:50]

joone.hur@intel.com changed reviewers:
+ jdduke@chromium.org

[joone, 2015-05-13 20:19:51]

please take a look at this CL.

[jdduke (slow), 2015-05-13 20:26:31]

Can you also update BUILD.gn?

https://codereview.chromium.org/1128203009/diff/1/ui/touch_selection/ui_touch...
File ui/touch_selection/ui_touch_selection.gyp (right):

https://codereview.chromium.org/1128203009/diff/1/ui/touch_selection/ui_touch...
ui/touch_selection/ui_touch_selection.gyp:45: '../aura/aura.gyp:aura',
Can you also exclude the skia dep here?

[joone, 2015-05-13 20:38:03]

On 2015/05/13 20:26:31, jdduke wrote:
> Can you also update BUILD.gn?
> 
>
https://codereview.chromium.org/1128203009/diff/1/ui/touch_selection/ui_touch...
> File ui/touch_selection/ui_touch_selection.gyp (right):
> 
>
https://codereview.chromium.org/1128203009/diff/1/ui/touch_selection/ui_touch...
> ui/touch_selection/ui_touch_selection.gyp:45: '../aura/aura.gyp:aura',
> Can you also exclude the skia dep here?

OK, I will do.

[joone, 2015-05-13 21:19:10]

On 2015/05/13 20:38:03, joone wrote:
> On 2015/05/13 20:26:31, jdduke wrote:
> > Can you also update BUILD.gn?
> > 
> >
>
https://codereview.chromium.org/1128203009/diff/1/ui/touch_selection/ui_touch...
> > File ui/touch_selection/ui_touch_selection.gyp (right):
> > 
> >
>
https://codereview.chromium.org/1128203009/diff/1/ui/touch_selection/ui_touch...
> > ui/touch_selection/ui_touch_selection.gyp:45: '../aura/aura.gyp:aura',
> > Can you also exclude the skia dep here?
> 
> OK, I will do.

I've updated the CL. Please review it.

[jdduke (slow), 2015-05-13 21:30:49]

lgtm

[joone, 2015-05-13 21:46:29]

The CQ bit was checked by joone.hur@intel.com

[commit-bot: I haz the power, 2015-05-13 21:47:26]

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

[commit-bot: I haz the power, 2015-05-14 00:01:52]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-05-14 00:03:37]

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

[davve, 2015-05-18 08:03:06]

Thanks for fixing this, joone!

Have we come any closer to finding out what the culprit was? Is it a clang vs.
gcc issue somehow? I suppose if all component=shared_library builds were affect,
it would have been caught earlier?