Remove RawPtr from Source/web/

Remove RawPtr from Source/web/

BUG=585328
Committed: https://crrev.com/36eee5a33fff7b046946db6d0f8f80a60f3e2057
Cr-Commit-Position: refs/heads/master@{#386075}

[keishi, 2016-04-06 06:46:14]

The CQ bit was checked by keishi@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-04-06 06:46:47]

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

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

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

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

Dry run: Try jobs failed on following builders:
  ios_rel_device_gn on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_rel_device_gn...)
  ios_rel_device_ninja on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_rel_device_ni...)
  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 08:26:25]

The CQ bit was checked by keishi@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-04-06 08:26:41]

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

[commit-bot: I haz the power, 2016-04-06 09:34:33]

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

[commit-bot: I haz the power, 2016-04-06 09:34:34]

Dry run: This issue passed the CQ dry run.

[keishi, 2016-04-06 09:37:02]

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

[keishi, 2016-04-06 09:37:04]

PTAL

[haraken, 2016-04-06 09:38:15]

LGTM

[keishi, 2016-04-06 10:35:50]

keishi@chromium.org changed reviewers:
+ dcheng@chromium.org

[keishi, 2016-04-06 10:35:51]

+dcheng Could you review the public/ files? Thanks.

[keishi, 2016-04-07 11:35:08]

Ping dcheng, PTAL

[dcheng, 2016-04-07 16:14:16]

lgtm

[keishi, 2016-04-08 04:44:47]

The CQ bit was checked by keishi@chromium.org

[keishi, 2016-04-08 04:44:48]

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

[commit-bot: I haz the power, 2016-04-08 04:44:59]

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

[commit-bot: I haz the power, 2016-04-08 04:48:30]

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

[commit-bot: I haz the power, 2016-04-08 04:48:32]

Try jobs failed on following builders:
  android_compile_dbg on tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_comp...)
  linux_chromium_asan_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_compile_dbg_32_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  ios_dbg_simulator_gn on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_dbg_simulator...)
  ios_dbg_simulator_ninja on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_dbg_simulator...)
  ios_rel_device_gn on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_rel_device_gn...)
  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-08 06:59:28]

The CQ bit was checked by keishi@chromium.org

[keishi, 2016-04-08 06:59:29]

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

[commit-bot: I haz the power, 2016-04-08 06:59:38]

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

[commit-bot: I haz the power, 2016-04-08 08:46:01]

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

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

Failed to apply patch for third_party/WebKit/Source/web/WebLocalFrameImpl.cpp:
While running git apply --index -3 -p1;
  error: patch failed: third_party/WebKit/Source/web/WebLocalFrameImpl.cpp:1589
  Falling back to three-way merge...
  Applied patch to 'third_party/WebKit/Source/web/WebLocalFrameImpl.cpp' with
conflicts.
  U third_party/WebKit/Source/web/WebLocalFrameImpl.cpp

Patch:       third_party/WebKit/Source/web/WebLocalFrameImpl.cpp
Index: third_party/WebKit/Source/web/WebLocalFrameImpl.cpp
diff --git a/third_party/WebKit/Source/web/WebLocalFrameImpl.cpp
b/third_party/WebKit/Source/web/WebLocalFrameImpl.cpp
index
821bc348a9a786c3396d161d58fdbbf872702b9c..3f82e14dd4f7dd8a1939fba0192389bd6666eff9
100644
--- a/third_party/WebKit/Source/web/WebLocalFrameImpl.cpp
+++ b/third_party/WebKit/Source/web/WebLocalFrameImpl.cpp
@@ -543,7 +543,7 @@ WebLocalFrame*
WebLocalFrame::frameForContext(v8::Local<v8::Context> context)
 
 WebLocalFrame* WebLocalFrame::fromFrameOwnerElement(const WebElement& element)
 {
-    return
WebLocalFrameImpl::fromFrameOwnerElement(RawPtr<Element>(element).get());
+    return WebLocalFrameImpl::fromFrameOwnerElement(element);
 }
 
 bool WebLocalFrameImpl::isWebLocalFrame() const
@@ -996,7 +996,7 @@ bool WebLocalFrameImpl::hasMarkedText() const
 
 WebRange WebLocalFrameImpl::markedRange() const
 {
-    return frame()->inputMethodController().compositionRange();
+    return frame()->inputMethodController().compositionRange().get();
 }
 
 bool WebLocalFrameImpl::firstRectForCharacterRange(unsigned location, unsigned
length, WebRect& rectInViewport) const
@@ -1117,7 +1117,7 @@ bool WebLocalFrameImpl::hasSelection() const
 
 WebRange WebLocalFrameImpl::selectionRange() const
 {
-    return
createRange(frame()->selection().selection().toNormalizedEphemeralRange());
+    return
createRange(frame()->selection().selection().toNormalizedEphemeralRange()).get();
 }
 
 WebString WebLocalFrameImpl::selectionAsText() const
@@ -1166,8 +1166,8 @@ void WebLocalFrameImpl::selectRange(const WebPoint&
baseInViewport, const WebPoi
 void WebLocalFrameImpl::selectRange(const WebRange& webRange)
 {
     TRACE_EVENT0("blink", "WebLocalFrameImpl::selectRange");
-    if (RawPtr<Range> range = static_cast<RawPtr<Range>>(webRange))
-        frame()->selection().setSelectedRange(range.get(), VP_DEFAULT_AFFINITY,
SelectionDirectionalMode::NonDirectional, NotUserTriggered);
+    if (Range* range = static_cast<Range*>(webRange))
+        frame()->selection().setSelectedRange(range, VP_DEFAULT_AFFINITY,
SelectionDirectionalMode::NonDirectional, NotUserTriggered);
 }
 
 void WebLocalFrameImpl::moveRangeSelectionExtent(const WebPoint& point)
@@ -1418,7 +1418,7 @@ WebLocalFrameImpl*
WebLocalFrameImpl::create(WebTreeScopeType scope, WebFrameCli
 
 WebLocalFrameImpl* WebLocalFrameImpl::createProvisional(WebFrameClient* client,
WebRemoteFrame* oldWebFrame, WebSandboxFlags flags, const
WebFrameOwnerProperties& frameOwnerProperties)
 {
-    RawPtr<WebLocalFrameImpl> webFrame = new WebLocalFrameImpl(oldWebFrame,
client);
+    WebLocalFrameImpl* webFrame = new WebLocalFrameImpl(oldWebFrame, client);
     Frame* oldFrame = oldWebFrame->toImplBase()->frame();
     webFrame->setParent(oldWebFrame->parent());
     webFrame->setOpener(oldWebFrame->opener());
@@ -1426,9 +1426,9 @@ WebLocalFrameImpl*
WebLocalFrameImpl::createProvisional(WebFrameClient* client,
     // When a core Frame is created with no owner, it attempts to set itself as
     // the main frame of the Page. However, this is a provisional frame, and
may
     // disappear, so Page::m_mainFrame can't be updated just yet.
-    RawPtr<FrameOwner> tempOwner = DummyFrameOwner::create();
+    FrameOwner* tempOwner = DummyFrameOwner::create();
     // TODO(dcheng): This block is very similar to initializeCoreFrame. Try to
reuse it here.
-    RawPtr<LocalFrame> frame =
LocalFrame::create(webFrame->m_frameLoaderClientImpl.get(), oldFrame->host(),
tempOwner.get(), client ? client->serviceRegistry() : nullptr);
+    LocalFrame* frame =
LocalFrame::create(webFrame->m_frameLoaderClientImpl.get(), oldFrame->host(),
tempOwner, client ? client->serviceRegistry() : nullptr);
     // Set the name and unique name directly, bypassing any of the normal logic
     // to calculate unique name.
    
frame->tree().setPrecalculatedName(toWebRemoteFrameImpl(oldWebFrame)->frame()->tree().name(),
toWebRemoteFrameImpl(oldWebFrame)->frame()->tree().uniqueName());
@@ -1447,11 +1447,7 @@ WebLocalFrameImpl*
WebLocalFrameImpl::createProvisional(WebFrameClient* client,
     // during init(). Note that this may dispatch JS events; the frame may be
     // detached after init() returns.
     frame->init();
-#if ENABLE(OILPAN)
-    return webFrame.get();
-#else
-    return webFrame.release().leakRef();
-#endif
+    return webFrame;
 }
 
 
@@ -1505,7 +1501,7 @@ DEFINE_TRACE(WebLocalFrameImpl)
 }
 #endif
 
-void WebLocalFrameImpl::setCoreFrame(RawPtr<LocalFrame> frame)
+void WebLocalFrameImpl::setCoreFrame(LocalFrame* frame)
 {
     m_frame = frame;
 
@@ -1562,7 +1558,7 @@ void WebLocalFrameImpl::initializeCoreFrame(FrameHost*
host, FrameOwner* owner,
         frame()->document()->getSecurityOrigin()->grantUniversalAccess();
 }
 
-RawPtr<LocalFrame> WebLocalFrameImpl::createChildFrame(const FrameLoadRequest&
request,
+LocalFrame* WebLocalFrameImpl::createChildFrame(const FrameLoadRequest&
request,
     const AtomicString& name, HTMLFrameOwnerElement* ownerElement)
 {
     DCHECK(m_client);
@@ -1577,7 +1573,7 @@ RawPtr<LocalFrame>
WebLocalFrameImpl::createChildFrame(const FrameLoadRequest& r
     // which can identify the element.
     AtomicString uniqueName =
frame()->tree().calculateUniqueNameForNewChildFrame(
         name,
ownerElement->getAttribute(ownerElement->subResourceAttributeName()));
-    RawPtr<WebLocalFrameImpl> webframeChild =
toWebLocalFrameImpl(m_client->createChildFrame(this, scope, name, uniqueName,
static_cast<WebSandboxFlags>(ownerElement->getSandboxFlags()),
ownerProperties));
+    WebLocalFrameImpl* webframeChild =
toWebLocalFrameImpl(m_client->createChildFrame(this, scope, name, uniqueName,
static_cast<WebSandboxFlags>(ownerElement->getSandboxFlags()),
ownerProperties));
     if (!webframeChild)
         return nullptr;
 
@@ -1589,18 +1585,18 @@ RawPtr<LocalFrame>
WebLocalFrameImpl::createChildFrame(const FrameLoadRequest& r
 
     // If we're moving in the back/forward list, we might want to replace the
content
     // of this child frame with whatever was there at that point.
-    RawPtr<HistoryItem> childItem = nullptr;
+    HistoryItem* childItem = nullptr;
     if (isBackForwardLoadType(frame()->loader().loadType()) &&
!frame()->document()->loadEventFinished())
-        childItem =
RawPtr<HistoryItem>(webframeChild->client()->historyItemForNewChildFrame());
+        childItem = webframeChild->client()->historyItemForNewChildFrame();
 
     FrameLoadRequest newRequest = request;
     FrameLoadType loadType = FrameLoadTypeStandard;
     if (childItem) {
         newRequest = FrameLoadRequest(request.originDocument(),
-            FrameLoader::resourceRequestFromHistoryItem(childItem.get(),
UseProtocolCachePolicy));
+            FrameLoader::resourceRequestFromHistoryItem(childItem,
UseProtocolCachePolicy));
         loadType = FrameLoadTypeInitialHistoryLoad;
     }
-    webframeChild->frame()->loader().load(newRequest, loadType,
childItem.get());
+    webframeChild->frame()->loader().load(newRequest, loadType, childItem);
 
     // Note a synchronous navigation (about:blank) would have already processed
     // onload, so it is possible for the child frame to have already been
@@ -1802,7 +1798,7 @@ void WebLocalFrameImpl::loadJavaScriptURL(const KURL& url)
     if (!frame()->document() || !frame()->page())
         return;
 
-    RawPtr<Document> ownerDocument(frame()->document());
+    Document* ownerDocument(frame()->document());
 
     // Protect privileged pages against bookmarklets and other javascript
manipulations.
     if
(SchemeRegistry::shouldTreatURLSchemeAsNotAllowingJavascriptURLs(frame()->document()->url().protocol()))
@@ -1816,7 +1812,7 @@ void WebLocalFrameImpl::loadJavaScriptURL(const KURL& url)
         return;
     String scriptResult = toCoreString(v8::Local<v8::String>::Cast(result));
     if (!frame()->navigationScheduler().locationChangePending())
-       
frame()->loader().replaceDocumentWhileExecutingJavaScriptURL(scriptResult,
ownerDocument.get());
+       
frame()->loader().replaceDocumentWhileExecutingJavaScriptURL(scriptResult,
ownerDocument);
 }
 
 static void ensureFrameLoaderHasCommitted(FrameLoader& frameLoader)
@@ -1899,9 +1895,9 @@ void WebLocalFrameImpl::sendPings(const WebNode&
contextNode, const WebURL& dest
 WebURLRequest WebLocalFrameImpl::requestFromHistoryItem(const WebHistoryItem&
item,
     WebURLRequest::CachePolicy cachePolicy) const
 {
-    RawPtr<HistoryItem> historyItem = RawPtr<HistoryItem>(item);
+    HistoryItem* historyItem = item;
     ResourceRequest request = FrameLoader::resourceRequestFromHistoryItem(
-        historyItem.get(),
static_cast<ResourceRequestCachePolicy>(cachePolicy));
+        historyItem, static_cast<ResourceRequestCachePolicy>(cachePolicy));
     return WrappedResourceRequest(request);
 }
 
@@ -1930,9 +1926,9 @@ void WebLocalFrameImpl::load(const WebURLRequest& request,
WebFrameLoadType webF
     FrameLoadRequest frameRequest = FrameLoadRequest(nullptr, resourceRequest);
     if (isClientRedirect)
         frameRequest.setClientRedirect(ClientRedirectPolicy::ClientRedirect);
-    RawPtr<HistoryItem> historyItem = RawPtr<HistoryItem>(item);
+    HistoryItem* historyItem = item;
     frame()->loader().load(
-        frameRequest, static_cast<FrameLoadType>(webFrameLoadType),
historyItem.get(),
+        frameRequest, static_cast<FrameLoadType>(webFrameLoadType),
historyItem,
         static_cast<HistoryLoadType>(webHistoryLoadType));
 }
 
@@ -1961,9 +1957,9 @@ void WebLocalFrameImpl::loadD…
(message too large)

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

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

[keishi, 2016-04-08 11:46:21]

The CQ bit was checked by keishi@chromium.org

[keishi, 2016-04-08 11:46:23]

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

[commit-bot: I haz the power, 2016-04-08 11:46:41]

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

[commit-bot: I haz the power, 2016-04-08 15:04:03]

Committed patchset #7 (id:120001)

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

Description was changed from

==========
Remove RawPtr from Source/web/

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

to

==========
Remove RawPtr from Source/web/

BUG=585328

Committed: https://crrev.com/36eee5a33fff7b046946db6d0f8f80a60f3e2057
Cr-Commit-Position: refs/heads/master@{#386075}
==========

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

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