Add missing initialization of RawPtr<> local var in r197835.

Add missing initialization of RawPtr<> local var in r197835.

Only insertCSSOMRuleInMediaRule()'s missing initialization is problematic.
Initialize the other occurrences here for consistency, aligning with
the convention we try to adhere to (but do not have tool support for
catching the failure to do so..)

R=pfeldman
BUG=178410
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=198128

[sof, 2015-07-01 09:48:14]

sigbjornf@opera.com changed reviewers:
+ oilpan-reviews@chromium.org, pfeldman@chromium.org

[sof, 2015-07-01 09:48:15]

please take a look.

a bunch of inspector tests are failing on the Oilpan bots,


https://storage.googleapis.com/chromium-layout-test-archives/WebKit_Mac_Oilpa...

A detail that's not very easy to spot by any stretch, and we rely on doing that
manually in reviews. That is,

  RefPtrWillBeRawPtr<T> local;

will null-initialize the RefPtr<T> non-Oilpan, but not do so for RawPtr<T> -- we
want RawPtr<T> as a minimal type wrapper around a bare pointer.

The reasoning was & still is that once we've transitioned to Oilpan the 'local'
will have to be null-initialized in any case, hence it doesn't make sense to
have a RawPtr<>() ctor (or a variant of RawPtr<> for stack allocations) that
null initializes.

Ideally we should have GC plugin support in place that catches out _potentially_
unsafe local variable annotations such as this.

[Hope that explains why we're here.]

[pfeldman, 2015-07-01 09:53:39]

thanks. lgtm.

[sof, 2015-07-01 09:59:09]

The CQ bit was checked by sigbjornf@opera.com

[commit-bot: I haz the power, 2015-07-01 09:59:40]

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

[commit-bot: I haz the power, 2015-07-01 10:29:39]

Committed patchset #1 (id:1) as
https://src.chromium.org/viewvc/blink?view=rev&revision=198128

[haraken, 2015-07-01 10:31:39]

LGTM