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

Side by Side Diff: content/browser/frame_host/render_frame_host_impl.cc

Issue 2519273002: Fail when saving page as MHTML provides information about the cause. (Closed)
Patch Set: Fixed typo. Created 4 years 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 unified diff | Download patch
« no previous file with comments | « content/browser/frame_host/render_frame_host_impl.h ('k') | content/common/BUILD.gn » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "content/browser/frame_host/render_frame_host_impl.h" 5 #include "content/browser/frame_host/render_frame_host_impl.h"
6 6
7 #include <utility> 7 #include <utility>
8 8
9 #include "base/bind.h" 9 #include "base/bind.h"
10 #include "base/command_line.h" 10 #include "base/command_line.h"
(...skipping 2145 matching lines...) Expand 10 before | Expand all | Expand 10 after
2156 if (is_active()) 2156 if (is_active())
2157 frame_tree_node_->DidStopLoading(); 2157 frame_tree_node_->DidStopLoading();
2158 } 2158 }
2159 2159
2160 void RenderFrameHostImpl::OnDidChangeLoadProgress(double load_progress) { 2160 void RenderFrameHostImpl::OnDidChangeLoadProgress(double load_progress) {
2161 frame_tree_node_->DidChangeLoadProgress(load_progress); 2161 frame_tree_node_->DidChangeLoadProgress(load_progress);
2162 } 2162 }
2163 2163
2164 void RenderFrameHostImpl::OnSerializeAsMHTMLResponse( 2164 void RenderFrameHostImpl::OnSerializeAsMHTMLResponse(
2165 int job_id, 2165 int job_id,
2166 bool success, 2166 MhtmlSaveStatus save_status,
2167 const std::set<std::string>& digests_of_uris_of_serialized_resources, 2167 const std::set<std::string>& digests_of_uris_of_serialized_resources,
2168 base::TimeDelta renderer_main_thread_time) { 2168 base::TimeDelta renderer_main_thread_time) {
2169 MHTMLGenerationManager::GetInstance()->OnSerializeAsMHTMLResponse( 2169 MHTMLGenerationManager::GetInstance()->OnSerializeAsMHTMLResponse(
2170 this, job_id, success, digests_of_uris_of_serialized_resources, 2170 this, job_id, save_status, digests_of_uris_of_serialized_resources,
2171 renderer_main_thread_time); 2171 renderer_main_thread_time);
2172 } 2172 }
2173 2173
2174 void RenderFrameHostImpl::OnSelectionChanged(const base::string16& text, 2174 void RenderFrameHostImpl::OnSelectionChanged(const base::string16& text,
2175 uint32_t offset, 2175 uint32_t offset,
2176 const gfx::Range& range) { 2176 const gfx::Range& range) {
2177 has_selection_ = !text.empty(); 2177 has_selection_ = !text.empty();
2178 GetRenderWidgetHost()->SelectionChanged(text, offset, range); 2178 GetRenderWidgetHost()->SelectionChanged(text, offset, range);
2179 } 2179 }
2180 2180
(...skipping 1150 matching lines...) Expand 10 before | Expand all | Expand 10 after
3331 // There is no pending NavigationEntry in these cases, so pass 0 as the 3331 // There is no pending NavigationEntry in these cases, so pass 0 as the
3332 // pending_nav_entry_id. If the previous handle was a prematurely aborted 3332 // pending_nav_entry_id. If the previous handle was a prematurely aborted
3333 // navigation loaded via LoadDataWithBaseURL, propagate the entry id. 3333 // navigation loaded via LoadDataWithBaseURL, propagate the entry id.
3334 return NavigationHandleImpl::Create( 3334 return NavigationHandleImpl::Create(
3335 params.url, frame_tree_node_, is_renderer_initiated, 3335 params.url, frame_tree_node_, is_renderer_initiated,
3336 params.was_within_same_page, params.is_srcdoc, base::TimeTicks::Now(), 3336 params.was_within_same_page, params.is_srcdoc, base::TimeTicks::Now(),
3337 entry_id_for_data_nav, false); // started_from_context_menu 3337 entry_id_for_data_nav, false); // started_from_context_menu
3338 } 3338 }
3339 3339
3340 } // namespace content 3340 } // namespace content
OLDNEW
« no previous file with comments | « content/browser/frame_host/render_frame_host_impl.h ('k') | content/common/BUILD.gn » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698