Remove unused sourceData variable in InspectorStyleSheet.

Remove unused sourceData variable in InspectorStyleSheet.

Breaks build for targets that treat unused variables as errors.

R=dgozman@chromium.org, vsevik@chromium.org

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

[dgozman, 2014-09-17 09:26:11]

lgtm

[Mads Ager (chromium), 2014-09-17 09:29:28]

The CQ bit was checked by ager@chromium.org

[commit-bot: I haz the power, 2014-09-17 09:30:25]

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

[dgozman, 2014-09-17 09:32:18]

How does original patch did it through CQ? Should we file a bug for that?

[Mads Ager (chromium), 2014-09-17 09:34:37]

On 2014/09/17 09:32:18, dgozman wrote:
> How does original patch did it through CQ? Should we file a bug for that?

This is only showing up on the Oilpan bots. It looks like the Oilpan bots are
currently the only bots treating unused warnings as errors. That is a little
odd.

[Mads Ager (chromium), 2014-09-17 09:39:58]

Committed patchset #1 (id:1) manually as 182145 (presubmit successful).

[vsevik, 2014-09-30 12:17:53]

I think this change is wrong.
extractSourceData() call has important side effects hence the resulting code
still works by accident only and has become quite fragile.
The ASSERT was also meaningful so I don't think we should remove it, it should
be replaced with ASSERT_UNUSED create for this exact case instead.
I will revert this CL and adjust the code to avoid unused warnings.

[vsevik, 2014-09-30 12:18:17]

A revert of this CL (patchset #1 id:1) has been created in
https://codereview.chromium.org/611333002/ by vsevik@chromium.org.

The reason for reverting is: This code is still needed.