Switch RenderViewContextMenu to use RequestOpenURL

content/test/web_contents_observer_sanity_checker.cc

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "content/test/web_contents_observer_sanity_checker.h"
 
 #include "base/strings/stringprintf.h"
 #include "build/build_config.h"
 #include "content/browser/frame_host/render_frame_host_impl.h"
 #include "content/common/frame_messages.h"
@@ skipping 261 matching lines @@
     bool was_ignored_by_handler) {
   AssertRenderFrameExists(render_frame_host);
 }
 
 void WebContentsObserverSanityChecker::DidOpenRequestedURL(
     WebContents* new_contents,
     RenderFrameHost* source_render_frame_host,
     const GURL& url,
     const Referrer& referrer,
     WindowOpenDisposition disposition,
-    ui::PageTransition transition) {
-  AssertRenderFrameExists(source_render_frame_host);
+    ui::PageTransition transition,
+    bool started_from_context_menu) {
+  // AssertRenderFrameExists(source_render_frame_host);

[nasko, 2017/02/14 00:10:23]

This shouldn't be commented out.

[Patrick Noland, 2017/02/14 01:16:16]

Done.

 }
 
 void WebContentsObserverSanityChecker::MediaStartedPlaying(
     const MediaPlayerInfo& media_info,
     const MediaPlayerId& id) {
   CHECK(!web_contents_destroyed_);
   CHECK(std::find(active_media_players_.begin(), active_media_players_.end(),
                   id) == active_media_players_.end());
   active_media_players_.push_back(id);
 }
@@ skipping 112 matching lines @@
       if (live_routes_.count(entry.first))
         return true;
       if (current_hosts_.count(entry.first))
         return true;
     }
   }
   return false;
 }
 
 }  // namespace content