Remove RawPtr from core/css

Remove RawPtr from core/css

BUG=585328
Committed: https://crrev.com/323d61ead93eeb077566ea4d014e6c038221ebaa
Cr-Commit-Position: refs/heads/master@{#385662}

[keishi, 2016-04-05 04:24:47]

keishi@chromium.org changed reviewers:
+ haraken@chromium.org, oilpan-reviews@chromium.org

[keishi, 2016-04-05 04:24:50]

PTAL

https://codereview.chromium.org/1858753003/diff/20001/third_party/WebKit/Sour...
File third_party/WebKit/Source/core/css/CSSDefaultStyleSheets.cpp (right):

https://codereview.chromium.org/1858753003/diff/20001/third_party/WebKit/Sour...
third_party/WebKit/Source/core/css/CSSDefaultStyleSheets.cpp:70:
LEAK_SANITIZER_IGNORE_OBJECT(sheet);
This line caused "error: expression result unused" so I modified
LeakAnnotations.h

[haraken, 2016-04-05 04:29:10]

LGTM

[Timothy Loh, 2016-04-05 04:31:56]

timloh@chromium.org changed reviewers:
+ timloh@chromium.org

[Timothy Loh, 2016-04-05 04:31:57]

lgtm

[keishi, 2016-04-05 04:33:12]

The CQ bit was checked by keishi@chromium.org

[commit-bot: I haz the power, 2016-04-05 04:33:38]

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

[commit-bot: I haz the power, 2016-04-05 04:37:40]

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

[commit-bot: I haz the power, 2016-04-05 04:37:42]

Try jobs failed on following builders:
  ios_dbg_simulator_gn on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_dbg_simulator...)
  mac_chromium_compile_dbg_ng on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)
  mac_chromium_gn_rel on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_gn_r...)
  mac_chromium_rel_ng on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[keishi, 2016-04-05 08:04:22]

The CQ bit was checked by keishi@chromium.org

[keishi, 2016-04-05 08:04:23]

The patchset sent to the CQ was uploaded after l-g-t-m from
haraken@chromium.org, timloh@chromium.org
Link to the patchset: https://codereview.chromium.org/1858753003/#ps40001
(title: "rebased")

[commit-bot: I haz the power, 2016-04-05 08:04:39]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1858753003/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1858753003/40001

[commit-bot: I haz the power, 2016-04-05 08:53:33]

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

[commit-bot: I haz the power, 2016-04-05 08:53:35]

Try jobs failed on following builders:
  win_chromium_compile_dbg_ng on tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_comp...)

[keishi, 2016-04-06 11:58:32]

The CQ bit was checked by keishi@chromium.org

[keishi, 2016-04-06 11:58:33]

The patchset sent to the CQ was uploaded after l-g-t-m from
haraken@chromium.org, timloh@chromium.org
Link to the patchset: https://codereview.chromium.org/1858753003/#ps60001
(title: " ")

[commit-bot: I haz the power, 2016-04-06 11:58:50]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1858753003/60001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1858753003/60001

[commit-bot: I haz the power, 2016-04-06 12:02:29]

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

[commit-bot: I haz the power, 2016-04-06 12:02:30]

Try jobs failed on following builders:
  mac_chromium_compile_dbg_ng on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)
  mac_chromium_gn_rel on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_gn_r...)

[keishi, 2016-04-06 13:48:01]

The CQ bit was checked by keishi@chromium.org

[keishi, 2016-04-06 13:48:02]

The patchset sent to the CQ was uploaded after l-g-t-m from
haraken@chromium.org, timloh@chromium.org
Link to the patchset: https://codereview.chromium.org/1858753003/#ps80001
(title: "rebased")

[commit-bot: I haz the power, 2016-04-06 13:48:21]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1858753003/80001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1858753003/80001

[commit-bot: I haz the power, 2016-04-06 14:51:22]

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

[commit-bot: I haz the power, 2016-04-06 14:51:24]

Try jobs failed on following builders:
  win_chromium_rel_ng on tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[keishi, 2016-04-06 15:22:58]

The CQ bit was checked by keishi@chromium.org

[keishi, 2016-04-06 15:22:59]

The patchset sent to the CQ was uploaded after l-g-t-m from
haraken@chromium.org, timloh@chromium.org
Link to the patchset: https://codereview.chromium.org/1858753003/#ps100001
(title: " ")

[commit-bot: I haz the power, 2016-04-06 15:23:23]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1858753003/100001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1858753003/100001

[commit-bot: I haz the power, 2016-04-06 16:36:11]

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

[commit-bot: I haz the power, 2016-04-06 16:36:13]

Try jobs failed on following builders:
  win_chromium_rel_ng on tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[keishi, 2016-04-07 03:16:46]

The CQ bit was checked by keishi@chromium.org

[keishi, 2016-04-07 03:16:48]

The patchset sent to the CQ was uploaded after l-g-t-m from
haraken@chromium.org, timloh@chromium.org
Link to the patchset: https://codereview.chromium.org/1858753003/#ps120001
(title: " ")

[commit-bot: I haz the power, 2016-04-07 03:17:11]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1858753003/120001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1858753003/120001

[commit-bot: I haz the power, 2016-04-07 04:47:02]

Committed patchset #7 (id:120001)

[commit-bot: I haz the power, 2016-04-07 04:48:03]

Description was changed from

==========
Remove RawPtr from core/css

BUG=585328
==========

to

==========
Remove RawPtr from core/css

BUG=585328

Committed: https://crrev.com/323d61ead93eeb077566ea4d014e6c038221ebaa
Cr-Commit-Position: refs/heads/master@{#385662}
==========

[commit-bot: I haz the power, 2016-04-07 04:48:05]

Patchset 7 (id:??) landed as
https://crrev.com/323d61ead93eeb077566ea4d014e6c038221ebaa
Cr-Commit-Position: refs/heads/master@{#385662}

[keishi, 2016-04-07 11:21:16]

A revert of this CL (patchset #7 id:120001) has been created in
https://codereview.chromium.org/1866973002/ by keishi@chromium.org.

The reason for reverting is: Broke Linux Tests (valgrind).