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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: content/browser/frame_host/frame_tree.cc

Issue 2426503002: Make printing work better with OOPIF. (Closed)
Patch Set: Fix build, fix some tests Created 4 years, 2 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
« components/printing/common/print_messages.h ('K') | « components/printing/test/print_web_view_helper_browsertest.cc ('k') | content/browser/frame_host/render_frame_host_impl.h » ('j') | content/public/renderer/render_frame_observer.h » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/browser/frame_host/frame_tree.cc
diff --git a/content/browser/frame_host/frame_tree.cc b/content/browser/frame_host/frame_tree.cc
index 511454f4d1069275dedc0d193778593d0d395985..6ef7f729d5881e84f22c3042e7705951e3bbde53 100644
--- a/content/browser/frame_host/frame_tree.cc
+++ b/content/browser/frame_host/frame_tree.cc
@@ -109,7 +109,7 @@ FrameTree::FrameTree(Navigator* navigator,
std::string(),
std::string(),
FrameOwnerProperties())),
- focused_frame_tree_node_id_(-1),
+ focused_frame_tree_node_id_(FrameTreeNode::kFrameTreeNodeInvalidId),
load_progress_(0.0) {}
FrameTree::~FrameTree() {
@@ -381,7 +381,7 @@ void FrameTree::ReleaseRenderViewHostRef(RenderViewHostImpl* render_view_host) {
void FrameTree::FrameRemoved(FrameTreeNode* frame) {
if (frame->frame_tree_node_id() == focused_frame_tree_node_id_)
- focused_frame_tree_node_id_ = -1;
+ focused_frame_tree_node_id_ = FrameTreeNode::kFrameTreeNodeInvalidId;
// No notification for the root frame.
if (!frame->parent()) {
« components/printing/common/print_messages.h ('K') | « components/printing/test/print_web_view_helper_browsertest.cc ('k') | content/browser/frame_host/render_frame_host_impl.h » ('j') | content/public/renderer/render_frame_observer.h » ('J')

Powered by Google App Engine
This is Rietveld 408576698