Remove Frame::host() method

third_party/WebKit/Source/core/frame/RemoteFrame.cpp

Index: third_party/WebKit/Source/core/frame/RemoteFrame.cpp
diff --git a/third_party/WebKit/Source/core/frame/RemoteFrame.cpp b/third_party/WebKit/Source/core/frame/RemoteFrame.cpp
index e864435226d862991a9816cad4617f57d8fc1615..7173101a816f7630f88ce5fe5ee20398d462f853 100644
--- a/third_party/WebKit/Source/core/frame/RemoteFrame.cpp
+++ b/third_party/WebKit/Source/core/frame/RemoteFrame.cpp
@@ -108,7 +108,7 @@ void RemoteFrame::detach(FrameDetachType type) {
 
 bool RemoteFrame::prepareForCommit() {
   detachChildren();
-  return !!host();
+  return !!page();

[sashab, 2017/03/24 00:33:05]

Replacing this with page() != nullptr gives the error:

third_party/WebKit/Source/core/frame/RemoteFrame.cpp:111:  Tests for true/false
and null/non-null should be done without equality comparisons. 
[readability/comparison_to_boolean] [5]

I can't find this in the style guide :/ is this !! pattern the "normal" way to
convert to bool?

[haraken, 2017/03/24 00:36:49]

Yes, it's a common pattern.

 }
 
 RemoteSecurityContext* RemoteFrame::securityContext() const {