Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(8204)

Unified Diff: chrome/renderer/net/net_error_helper.cc

Issue 137463002: RenderFrame: flesh out Observer (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@lkgr
Patch Set: Smash changesets together Created 6 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/renderer/net/net_error_helper.cc
diff --git a/chrome/renderer/net/net_error_helper.cc b/chrome/renderer/net/net_error_helper.cc
index 2c16add28ea6f6038a3fc0885b2254b8f07cb647..9f3e3d39642b99e120e63954a3ac3ffd364b8f17 100644
--- a/chrome/renderer/net/net_error_helper.cc
+++ b/chrome/renderer/net/net_error_helper.cc
@@ -16,6 +16,7 @@
#include "content/public/common/content_client.h"
#include "content/public/common/url_constants.h"
#include "content/public/renderer/content_renderer_client.h"
+#include "content/public/renderer/render_frame.h"
#include "content/public/renderer/render_thread.h"
#include "content/public/renderer/render_view.h"
#include "content/public/renderer/resource_fetcher.h"
@@ -36,9 +37,9 @@
using base::JSONWriter;
using chrome_common_net::DnsProbeStatus;
using chrome_common_net::DnsProbeStatusToString;
+using content::RenderFrame;
+using content::RenderFrameObserver;
using content::RenderThread;
-using content::RenderView;
-using content::RenderViewObserver;
using content::kUnreachableWebDataURL;
namespace {
@@ -62,9 +63,9 @@ NetErrorHelperCore::FrameType GetFrameType(const blink::WebFrame* frame) {
} // namespace
-NetErrorHelper::NetErrorHelper(RenderView* render_view)
- : RenderViewObserver(render_view),
- content::RenderViewObserverTracker<NetErrorHelper>(render_view),
+NetErrorHelper::NetErrorHelper(RenderFrame* render_view)
+ : RenderFrameObserver(render_view),
+ content::RenderFrameObserverTracker<NetErrorHelper>(render_view),
core_(this) {
}
@@ -123,7 +124,8 @@ void NetErrorHelper::GenerateLocalizedErrorPage(const blink::WebURLError& error,
LocalizedError::GetStrings(error.reason, error.domain.utf8(),
error.unreachableURL, is_failed_post,
RenderThread::Get()->GetLocale(),
- render_view()->GetAcceptLanguages(),
+ render_frame()->GetRenderView()->
+ GetAcceptLanguages(),
&error_strings);
// "t" is the id of the template's root node.
*error_html = webui::GetTemplatesHtml(template_html, &error_strings, "t");
@@ -132,7 +134,7 @@ void NetErrorHelper::GenerateLocalizedErrorPage(const blink::WebURLError& error,
void NetErrorHelper::LoadErrorPageInMainFrame(const std::string& html,
const GURL& failed_url) {
- blink::WebView* web_view = render_view()->GetWebView();
+ blink::WebView* web_view = render_frame()->GetRenderView()->GetWebView();
if (!web_view)
return;
blink::WebFrame* frame = web_view->mainFrame();
@@ -147,7 +149,8 @@ void NetErrorHelper::UpdateErrorPage(const blink::WebURLError& error,
error.unreachableURL,
is_failed_post,
RenderThread::Get()->GetLocale(),
- render_view()->GetAcceptLanguages(),
+ render_frame()->GetRenderView()->
+ GetAcceptLanguages(),
&error_strings);
std::string json;
@@ -162,13 +165,13 @@ void NetErrorHelper::UpdateErrorPage(const blink::WebURLError& error,
}
base::string16 frame_xpath;
- render_view()->EvaluateScript(frame_xpath, js16, 0, false);
+ render_frame()->GetRenderView()->EvaluateScript(frame_xpath, js16, 0, false);
}
void NetErrorHelper::FetchErrorPage(const GURL& url) {
DCHECK(!alt_error_page_fetcher_.get());
- blink::WebView* web_view = render_view()->GetWebView();
+ blink::WebView* web_view = render_frame()->GetRenderView()->GetWebView();
if (!web_view)
return;
blink::WebFrame* frame = web_view->mainFrame();

Powered by Google App Engine
This is Rietveld 408576698