Oilpan: account for DevToolsEmulator reference from WebSettingsImpl.

third_party/WebKit/Source/web/WebSettingsImpl.h

Index: third_party/WebKit/Source/web/WebSettingsImpl.h
diff --git a/third_party/WebKit/Source/web/WebSettingsImpl.h b/third_party/WebKit/Source/web/WebSettingsImpl.h
index 4524b3dbfdc7f5e194c277a58c354171dd6e10db..6835ec3697a613298d20af32d2d083c8e796ddb0 100644
--- a/third_party/WebKit/Source/web/WebSettingsImpl.h
+++ b/third_party/WebKit/Source/web/WebSettingsImpl.h
@@ -31,6 +31,7 @@
 #ifndef WebSettingsImpl_h
 #define WebSettingsImpl_h
 
+#include "platform/heap/Handle.h"
 #include "public/web/WebSettings.h"
 
 namespace blink {
@@ -207,7 +208,7 @@ public:
 
 private:
     Settings* m_settings;
-    DevToolsEmulator* m_devToolsEmulator;
+    RawPtrWillBeUntracedMember<DevToolsEmulator> m_devToolsEmulator;

[haraken, 2015/11/13 11:47:51]

As commented in another CL, I think our plan is to use WeakPersistent (instead
of UntracedMember) for this kind of raw pointer. (Correct me if I'm
misunderstanding the previous discussion.)

[sof, 2015/11/13 13:31:48]

See the description for why I consider that is excessive in this particular
case.

[haraken, 2015/11/13 13:39:41]

LGTM.

It's still not quite clear to me when we should prefer UntracedMember though. I
was assuming that UntracedMembers should be avoided as much as possible (i.e.,
UntracedMembers are allowed only if using WeakPersistent has a risk of hitting
performance issues).

     bool m_showFPSCounter;
     bool m_showPaintRects;
     bool m_renderVSyncNotificationEnabled;