mirror of
https://bitbucket.org/chromiumembedded/cef
synced 2025-06-05 21:39:12 +02:00
Update to Chromium revision 248478.
- Add new CefSettings.windowless_rendering_enabled value that must be enabled when using windowless (off-screen) rendering. - Improve naming and documentation for CefWindowInfo members. - CefBeginTracing now completes asynchronously. - Rename CefEndTracingAsync to CefEndTracing. - Rename CefCompletionHandler to CefCompletionCallback. git-svn-id: https://chromiumembedded.googlecode.com/svn/trunk@1592 5089003a-bbd8-11dd-ad1f-f1f9622dbc98
This commit is contained in:
@@ -1,6 +1,6 @@
|
||||
Index: common.gypi
|
||||
===================================================================
|
||||
--- common.gypi (revision 240657)
|
||||
--- common.gypi (revision 248478)
|
||||
+++ common.gypi (working copy)
|
||||
@@ -9,6 +9,9 @@
|
||||
# Variables expected to be overriden on the GYP command line (-D) or by
|
||||
@@ -12,18 +12,22 @@ Index: common.gypi
|
||||
# Putting a variables dict inside another variables dict looks kind of
|
||||
# weird. This is done so that 'host_arch', 'chromeos', etc are defined as
|
||||
# variables within the outer variables dict here. This is necessary
|
||||
@@ -201,7 +204,7 @@
|
||||
'enable_app_list%': 0,
|
||||
}],
|
||||
@@ -88,9 +91,9 @@
|
||||
#
|
||||
# TODO(erg): Merge this into the previous block once compiling with
|
||||
# aura safely implies including ash capabilities.
|
||||
- ['OS=="linux"', {
|
||||
- 'use_aura%': 1,
|
||||
- }],
|
||||
+ #['OS=="linux"', {
|
||||
+ # 'use_aura%': 1,
|
||||
+ #}],
|
||||
|
||||
- ['use_aura==1 or (OS!="win" and OS!="mac" and OS!="ios" and OS!="android")', {
|
||||
+ ['OS!="win" and OS!="mac" and OS!="ios" and OS!="android"', {
|
||||
'use_default_render_theme%': 1,
|
||||
}, {
|
||||
'use_default_render_theme%': 0,
|
||||
# Whether we're a traditional desktop unix.
|
||||
['(OS=="linux" or OS=="freebsd" or OS=="openbsd" or OS=="solaris") and chromeos==0', {
|
||||
Index: mac/strip_save_dsym
|
||||
===================================================================
|
||||
--- mac/strip_save_dsym (revision 240657)
|
||||
--- mac/strip_save_dsym (revision 248478)
|
||||
+++ mac/strip_save_dsym (working copy)
|
||||
@@ -48,7 +48,7 @@
|
||||
"bundle"]
|
||||
|
@@ -1,16 +1,16 @@
|
||||
Index: public/renderer/content_renderer_client.cc
|
||||
===================================================================
|
||||
--- public/renderer/content_renderer_client.cc (revision 242756)
|
||||
--- public/renderer/content_renderer_client.cc (revision 248478)
|
||||
+++ public/renderer/content_renderer_client.cc (working copy)
|
||||
@@ -91,7 +91,6 @@
|
||||
@@ -92,7 +92,6 @@
|
||||
return false;
|
||||
}
|
||||
|
||||
-#ifdef OS_ANDROID
|
||||
bool ContentRendererClient::HandleNavigation(
|
||||
RenderView* view,
|
||||
RenderFrame* render_frame,
|
||||
DocumentState* document_state,
|
||||
@@ -103,7 +102,6 @@
|
||||
@@ -104,7 +103,6 @@
|
||||
bool is_redirect) {
|
||||
return false;
|
||||
}
|
||||
@@ -20,9 +20,9 @@ Index: public/renderer/content_renderer_client.cc
|
||||
const GURL& url,
|
||||
Index: public/renderer/content_renderer_client.h
|
||||
===================================================================
|
||||
--- public/renderer/content_renderer_client.h (revision 242756)
|
||||
--- public/renderer/content_renderer_client.h (revision 248478)
|
||||
+++ public/renderer/content_renderer_client.h (working copy)
|
||||
@@ -172,7 +172,6 @@
|
||||
@@ -175,7 +175,6 @@
|
||||
// Returns true if a popup window should be allowed.
|
||||
virtual bool AllowPopup();
|
||||
|
||||
@@ -30,7 +30,7 @@ Index: public/renderer/content_renderer_client.h
|
||||
// TODO(sgurun) This callback is deprecated and will be removed as soon
|
||||
// as android webview completes implementation of a resource throttle based
|
||||
// shouldoverrideurl implementation. See crbug.com/325351
|
||||
@@ -187,7 +186,6 @@
|
||||
@@ -190,7 +189,6 @@
|
||||
blink::WebNavigationType type,
|
||||
blink::WebNavigationPolicy default_policy,
|
||||
bool is_redirect);
|
||||
@@ -40,17 +40,17 @@ Index: public/renderer/content_renderer_client.h
|
||||
// If |send_referrer| is set to false (which is the default), no referrer
|
||||
Index: renderer/render_view_impl.cc
|
||||
===================================================================
|
||||
--- renderer/render_view_impl.cc (revision 242756)
|
||||
--- renderer/render_view_impl.cc (revision 248478)
|
||||
+++ renderer/render_view_impl.cc (working copy)
|
||||
@@ -3124,7 +3124,6 @@
|
||||
WebFrame* frame, WebDataSource::ExtraData* extraData,
|
||||
const WebURLRequest& request, WebNavigationType type,
|
||||
WebNavigationPolicy default_policy, bool is_redirect) {
|
||||
@@ -3071,7 +3071,6 @@
|
||||
WebDataSource::ExtraData* extraData, const WebURLRequest& request,
|
||||
WebNavigationType type, WebNavigationPolicy default_policy,
|
||||
bool is_redirect) {
|
||||
-#ifdef OS_ANDROID
|
||||
// The handlenavigation API is deprecated and will be removed once
|
||||
// crbug.com/325351 is resolved.
|
||||
if (request.url() != GURL(kSwappedOutURL) &&
|
||||
@@ -3139,7 +3138,6 @@
|
||||
@@ -3086,7 +3085,6 @@
|
||||
is_redirect)) {
|
||||
return blink::WebNavigationPolicyIgnore;
|
||||
}
|
||||
|
@@ -1,8 +1,8 @@
|
||||
Index: message_loop.cc
|
||||
===================================================================
|
||||
--- message_loop.cc (revision 237081)
|
||||
--- message_loop.cc (revision 248478)
|
||||
+++ message_loop.cc (working copy)
|
||||
@@ -166,7 +166,7 @@
|
||||
@@ -142,7 +142,7 @@
|
||||
MessageLoop::~MessageLoop() {
|
||||
DCHECK_EQ(this, current());
|
||||
|
||||
|
@@ -1,34 +0,0 @@
|
||||
Index: native_theme_aura.cc
|
||||
===================================================================
|
||||
--- native_theme_aura.cc (revision 237081)
|
||||
+++ native_theme_aura.cc (working copy)
|
||||
@@ -17,10 +17,12 @@
|
||||
|
||||
namespace ui {
|
||||
|
||||
+#if !defined(OS_WIN)
|
||||
// static
|
||||
NativeTheme* NativeTheme::instance() {
|
||||
return NativeThemeAura::instance();
|
||||
}
|
||||
+#endif
|
||||
|
||||
// static
|
||||
NativeThemeAura* NativeThemeAura::instance() {
|
||||
Index: native_theme_win.cc
|
||||
===================================================================
|
||||
--- native_theme_win.cc (revision 237081)
|
||||
+++ native_theme_win.cc (working copy)
|
||||
@@ -211,12 +211,10 @@
|
||||
// TODO(sky): seems like we should default to NativeThemeWin, but that currently
|
||||
// breaks a couple of tests (FocusTraversalTest.NormalTraversal in
|
||||
// views_unittests).
|
||||
-#if !defined(USE_AURA)
|
||||
// static
|
||||
NativeTheme* NativeTheme::instance() {
|
||||
return NativeThemeWin::instance();
|
||||
}
|
||||
-#endif
|
||||
|
||||
// static
|
||||
NativeThemeWin* NativeThemeWin::instance() {
|
@@ -1,8 +1,21 @@
|
||||
Index: compositing_iosurface_layer_mac.mm
|
||||
===================================================================
|
||||
--- compositing_iosurface_layer_mac.mm (revision 248478)
|
||||
+++ compositing_iosurface_layer_mac.mm (working copy)
|
||||
@@ -119,7 +119,7 @@
|
||||
|
||||
if (!context_.get() || !renderWidgetHostView_ ||
|
||||
!renderWidgetHostView_->compositing_iosurface_) {
|
||||
- glClearColor(1, 1, 1, 1);
|
||||
+ glClearColor(0, 0, 0, 0);
|
||||
glClear(GL_COLOR_BUFFER_BIT);
|
||||
return;
|
||||
}
|
||||
Index: compositing_iosurface_mac.mm
|
||||
===================================================================
|
||||
--- compositing_iosurface_mac.mm (revision 242756)
|
||||
--- compositing_iosurface_mac.mm (revision 248478)
|
||||
+++ compositing_iosurface_mac.mm (working copy)
|
||||
@@ -433,7 +433,7 @@
|
||||
@@ -424,7 +424,7 @@
|
||||
glUseProgram(0); CHECK_AND_SAVE_GL_ERROR();
|
||||
} else {
|
||||
// Should match the clear color of RenderWidgetHostViewMac.
|
||||
@@ -11,10 +24,9 @@ Index: compositing_iosurface_mac.mm
|
||||
glClear(GL_COLOR_BUFFER_BIT);
|
||||
}
|
||||
|
||||
|
||||
Index: compositing_iosurface_shader_programs_mac.cc
|
||||
===================================================================
|
||||
--- compositing_iosurface_shader_programs_mac.cc (revision 242756)
|
||||
--- compositing_iosurface_shader_programs_mac.cc (revision 248478)
|
||||
+++ compositing_iosurface_shader_programs_mac.cc (working copy)
|
||||
@@ -11,6 +11,7 @@
|
||||
#include "base/debug/trace_event.h"
|
||||
@@ -49,7 +61,7 @@ Index: compositing_iosurface_shader_programs_mac.cc
|
||||
// Select and compile the shader program source code.
|
||||
if (shader_type == GL_VERTEX_SHADER) {
|
||||
const GLchar* source_snippets[] = {
|
||||
@@ -414,6 +426,14 @@
|
||||
@@ -412,6 +424,14 @@
|
||||
Reset();
|
||||
}
|
||||
|
||||
@@ -66,7 +78,7 @@ Index: compositing_iosurface_shader_programs_mac.cc
|
||||
shader_programs_[which] =
|
||||
Index: compositing_iosurface_shader_programs_mac.h
|
||||
===================================================================
|
||||
--- compositing_iosurface_shader_programs_mac.h (revision 242756)
|
||||
--- compositing_iosurface_shader_programs_mac.h (revision 248478)
|
||||
+++ compositing_iosurface_shader_programs_mac.h (working copy)
|
||||
@@ -48,6 +48,8 @@
|
||||
return rgb_to_yv12_output_format_;
|
||||
@@ -77,26 +89,11 @@ Index: compositing_iosurface_shader_programs_mac.h
|
||||
protected:
|
||||
FRIEND_TEST_ALL_PREFIXES(CompositingIOSurfaceTransformerTest,
|
||||
TransformsRGBToYV12);
|
||||
|
||||
Index: compositing_iosurface_layer_mac.mm
|
||||
===================================================================
|
||||
--- compositing_iosurface_layer_mac.mm (revision 242756)
|
||||
+++ compositing_iosurface_layer_mac.mm (working copy)
|
||||
@@ -89,7 +89,7 @@
|
||||
|
||||
if (!context_.get() || !renderWidgetHostView_ ||
|
||||
!renderWidgetHostView_->compositing_iosurface_) {
|
||||
- glClearColor(1, 1, 1, 1);
|
||||
+ glClearColor(0, 0, 0, 0);
|
||||
glClear(GL_COLOR_BUFFER_BIT);
|
||||
return;
|
||||
}
|
||||
|
||||
Index: render_widget_host_view_mac.mm
|
||||
===================================================================
|
||||
--- render_widget_host_view_mac.mm (revision 242756)
|
||||
--- render_widget_host_view_mac.mm (revision 248478)
|
||||
+++ render_widget_host_view_mac.mm (working copy)
|
||||
@@ -490,7 +490,7 @@
|
||||
@@ -494,7 +494,7 @@
|
||||
software_layer_.reset([[CALayer alloc] init]);
|
||||
if (!software_layer_)
|
||||
LOG(ERROR) << "Failed to create CALayer for software rendering";
|
||||
@@ -105,7 +102,7 @@ Index: render_widget_host_view_mac.mm
|
||||
[software_layer_ setDelegate:cocoa_view_];
|
||||
[software_layer_ setContentsGravity:kCAGravityTopLeft];
|
||||
[software_layer_ setFrame:NSRectToCGRect([cocoa_view_ bounds])];
|
||||
@@ -2737,7 +2737,7 @@
|
||||
@@ -2846,7 +2846,7 @@
|
||||
|
||||
NSRect r = [self flipRectToNSRect:gfx::Rect(x, y, width, height)];
|
||||
CGContextSetFillColorWithColor(context,
|
||||
@@ -114,7 +111,7 @@ Index: render_widget_host_view_mac.mm
|
||||
CGContextFillRect(context, NSRectToCGRect(r));
|
||||
}
|
||||
if (damagedRect.bottom() > rect.bottom()) {
|
||||
@@ -2759,7 +2759,7 @@
|
||||
@@ -2868,7 +2868,7 @@
|
||||
|
||||
NSRect r = [self flipRectToNSRect:gfx::Rect(x, y, width, height)];
|
||||
CGContextSetFillColorWithColor(context,
|
||||
@@ -123,7 +120,7 @@ Index: render_widget_host_view_mac.mm
|
||||
CGContextFillRect(context, NSRectToCGRect(r));
|
||||
}
|
||||
}
|
||||
@@ -2918,7 +2918,7 @@
|
||||
@@ -3029,7 +3029,7 @@
|
||||
}
|
||||
} else {
|
||||
CGContextSetFillColorWithColor(context,
|
||||
@@ -132,7 +129,7 @@ Index: render_widget_host_view_mac.mm
|
||||
CGContextFillRect(context, dirtyRect);
|
||||
if (renderWidgetHostView_->whiteout_start_time_.is_null())
|
||||
renderWidgetHostView_->whiteout_start_time_ = base::TimeTicks::Now();
|
||||
@@ -3925,7 +3925,7 @@
|
||||
@@ -4037,7 +4037,7 @@
|
||||
if (!renderWidgetHostView_->render_widget_host_ ||
|
||||
renderWidgetHostView_->render_widget_host_->is_hidden()) {
|
||||
CGContextSetFillColorWithColor(context,
|
||||
@@ -141,4 +138,3 @@ Index: render_widget_host_view_mac.mm
|
||||
CGContextFillRect(context, clipRect);
|
||||
return;
|
||||
}
|
||||
|
||||
|
@@ -1,8 +1,8 @@
|
||||
Index: core/frame/FrameView.cpp
|
||||
===================================================================
|
||||
--- core/frame/FrameView.cpp (revision 163979)
|
||||
--- core/frame/FrameView.cpp (revision 166298)
|
||||
+++ core/frame/FrameView.cpp (working copy)
|
||||
@@ -200,8 +200,10 @@
|
||||
@@ -168,8 +168,10 @@
|
||||
if (!isMainFrame())
|
||||
return;
|
||||
|
||||
@@ -13,13 +13,13 @@ Index: core/frame/FrameView.cpp
|
||||
}
|
||||
|
||||
PassRefPtr<FrameView> FrameView::create(Frame* frame)
|
||||
Index: platform/mac/NSScrollerImpDetails.mm
|
||||
Index: platform/scroll/ScrollbarThemeMacCommon.mm
|
||||
===================================================================
|
||||
--- platform/mac/NSScrollerImpDetails.mm (revision 163979)
|
||||
+++ platform/mac/NSScrollerImpDetails.mm (working copy)
|
||||
@@ -73,10 +73,14 @@
|
||||
|
||||
bool isScrollbarOverlayAPIAvailable()
|
||||
--- platform/scroll/ScrollbarThemeMacCommon.mm (revision 166298)
|
||||
+++ platform/scroll/ScrollbarThemeMacCommon.mm (working copy)
|
||||
@@ -369,10 +369,14 @@
|
||||
// static
|
||||
bool ScrollbarThemeMacCommon::isOverlayAPIAvailable()
|
||||
{
|
||||
+#if 0
|
||||
static bool apiAvailable =
|
||||
@@ -31,4 +31,4 @@ Index: platform/mac/NSScrollerImpDetails.mm
|
||||
+#endif
|
||||
}
|
||||
|
||||
NSScrollerStyle recommendedScrollerStyle()
|
||||
} // namespace WebCore
|
||||
|
@@ -1,332 +0,0 @@
|
||||
Index: trace_message_filter.cc
|
||||
===================================================================
|
||||
--- trace_message_filter.cc (revision 241258)
|
||||
+++ trace_message_filter.cc (working copy)
|
||||
@@ -121,7 +121,7 @@
|
||||
if (is_awaiting_end_ack_) {
|
||||
is_awaiting_end_ack_ = false;
|
||||
TracingControllerImpl::GetInstance()->OnDisableRecordingAcked(
|
||||
- known_categories);
|
||||
+ this, known_categories);
|
||||
} else {
|
||||
NOTREACHED();
|
||||
}
|
||||
@@ -132,7 +132,8 @@
|
||||
// but check in case the child process is compromised.
|
||||
if (is_awaiting_capture_monitoring_snapshot_ack_) {
|
||||
is_awaiting_capture_monitoring_snapshot_ack_ = false;
|
||||
- TracingControllerImpl::GetInstance()->OnCaptureMonitoringSnapshotAcked();
|
||||
+ TracingControllerImpl::GetInstance()->OnCaptureMonitoringSnapshotAcked(
|
||||
+ this);
|
||||
} else {
|
||||
NOTREACHED();
|
||||
}
|
||||
@@ -160,7 +161,7 @@
|
||||
if (is_awaiting_buffer_percent_full_ack_) {
|
||||
is_awaiting_buffer_percent_full_ack_ = false;
|
||||
TracingControllerImpl::GetInstance()->OnTraceBufferPercentFullReply(
|
||||
- percent_full);
|
||||
+ this, percent_full);
|
||||
} else {
|
||||
NOTREACHED();
|
||||
}
|
||||
Index: tracing_controller_impl.cc
|
||||
===================================================================
|
||||
--- tracing_controller_impl.cc (revision 241258)
|
||||
+++ tracing_controller_impl.cc (working copy)
|
||||
@@ -177,7 +177,7 @@
|
||||
is_recording_ = true;
|
||||
|
||||
// Notify all child processes.
|
||||
- for (TraceMessageFilterMap::iterator it = trace_message_filters_.begin();
|
||||
+ for (TraceMessageFilterSet::iterator it = trace_message_filters_.begin();
|
||||
it != trace_message_filters_.end(); ++it) {
|
||||
it->get()->SendBeginTracing(category_filter, trace_options);
|
||||
}
|
||||
@@ -212,6 +212,7 @@
|
||||
// Count myself (local trace) in pending_disable_recording_ack_count_,
|
||||
// acked below.
|
||||
pending_disable_recording_ack_count_ = trace_message_filters_.size() + 1;
|
||||
+ pending_disable_recording_filters_ = trace_message_filters_;
|
||||
|
||||
// Handle special case of zero child processes by immediately flushing the
|
||||
// trace log. Once the flush has completed the caller will be notified that
|
||||
@@ -224,7 +225,7 @@
|
||||
}
|
||||
|
||||
// Notify all child processes.
|
||||
- for (TraceMessageFilterMap::iterator it = trace_message_filters_.begin();
|
||||
+ for (TraceMessageFilterSet::iterator it = trace_message_filters_.begin();
|
||||
it != trace_message_filters_.end(); ++it) {
|
||||
it->get()->SendEndTracing();
|
||||
}
|
||||
@@ -254,7 +255,7 @@
|
||||
static_cast<TraceLog::Options>(monitoring_tracing_options));
|
||||
|
||||
// Notify all child processes.
|
||||
- for (TraceMessageFilterMap::iterator it = trace_message_filters_.begin();
|
||||
+ for (TraceMessageFilterSet::iterator it = trace_message_filters_.begin();
|
||||
it != trace_message_filters_.end(); ++it) {
|
||||
it->get()->SendEnableMonitoring(category_filter,
|
||||
static_cast<TraceLog::Options>(monitoring_tracing_options));
|
||||
@@ -276,7 +277,7 @@
|
||||
TraceLog::GetInstance()->SetDisabled();
|
||||
|
||||
// Notify all child processes.
|
||||
- for (TraceMessageFilterMap::iterator it = trace_message_filters_.begin();
|
||||
+ for (TraceMessageFilterSet::iterator it = trace_message_filters_.begin();
|
||||
it != trace_message_filters_.end(); ++it) {
|
||||
it->get()->SendDisableMonitoring();
|
||||
}
|
||||
@@ -311,6 +312,7 @@
|
||||
// acked below.
|
||||
pending_capture_monitoring_snapshot_ack_count_ =
|
||||
trace_message_filters_.size() + 1;
|
||||
+ pending_capture_monitoring_filters_ = trace_message_filters_;
|
||||
|
||||
// Handle special case of zero child processes by immediately flushing the
|
||||
// trace log. Once the flush has completed the caller will be notified that
|
||||
@@ -323,7 +325,7 @@
|
||||
}
|
||||
|
||||
// Notify all child processes.
|
||||
- for (TraceMessageFilterMap::iterator it = trace_message_filters_.begin();
|
||||
+ for (TraceMessageFilterSet::iterator it = trace_message_filters_.begin();
|
||||
it != trace_message_filters_.end(); ++it) {
|
||||
it->get()->SendCaptureMonitoringSnapshot();
|
||||
}
|
||||
@@ -347,6 +349,7 @@
|
||||
// Count myself in pending_trace_buffer_percent_full_ack_count_, acked below.
|
||||
pending_trace_buffer_percent_full_ack_count_ =
|
||||
trace_message_filters_.size() + 1;
|
||||
+ pending_trace_buffer_percent_full_filters_ = trace_message_filters_;
|
||||
maximum_trace_buffer_percent_full_ = 0;
|
||||
|
||||
// Handle special case of zero child processes.
|
||||
@@ -354,11 +357,12 @@
|
||||
BrowserThread::PostTask(BrowserThread::UI, FROM_HERE,
|
||||
base::Bind(&TracingControllerImpl::OnTraceBufferPercentFullReply,
|
||||
base::Unretained(this),
|
||||
+ scoped_refptr<TraceMessageFilter>(),
|
||||
TraceLog::GetInstance()->GetBufferPercentFull()));
|
||||
}
|
||||
|
||||
// Notify all child processes.
|
||||
- for (TraceMessageFilterMap::iterator it = trace_message_filters_.begin();
|
||||
+ for (TraceMessageFilterSet::iterator it = trace_message_filters_.begin();
|
||||
it != trace_message_filters_.end(); ++it) {
|
||||
it->get()->SendGetTraceBufferPercentFull();
|
||||
}
|
||||
@@ -383,7 +387,7 @@
|
||||
base::Bind(&TracingControllerImpl::OnWatchEventMatched,
|
||||
base::Unretained(this)));
|
||||
|
||||
- for (TraceMessageFilterMap::iterator it = trace_message_filters_.begin();
|
||||
+ for (TraceMessageFilterSet::iterator it = trace_message_filters_.begin();
|
||||
it != trace_message_filters_.end(); ++it) {
|
||||
it->get()->SendSetWatchEvent(category_name, event_name);
|
||||
}
|
||||
@@ -396,7 +400,7 @@
|
||||
if (!can_cancel_watch_event())
|
||||
return false;
|
||||
|
||||
- for (TraceMessageFilterMap::iterator it = trace_message_filters_.begin();
|
||||
+ for (TraceMessageFilterSet::iterator it = trace_message_filters_.begin();
|
||||
it != trace_message_filters_.end(); ++it) {
|
||||
it->get()->SendCancelWatchEvent();
|
||||
}
|
||||
@@ -437,15 +441,54 @@
|
||||
return;
|
||||
}
|
||||
|
||||
+ // If a filter is removed while a response from that filter is pending then
|
||||
+ // simulate the response. Otherwise the response count will be wrong and the
|
||||
+ // completion callback will never be executed.
|
||||
+ if (pending_disable_recording_ack_count_ > 0) {
|
||||
+ TraceMessageFilterSet::const_iterator it =
|
||||
+ pending_disable_recording_filters_.find(trace_message_filter);
|
||||
+ if (it != pending_disable_recording_filters_.end()) {
|
||||
+ BrowserThread::PostTask(BrowserThread::UI, FROM_HERE,
|
||||
+ base::Bind(&TracingControllerImpl::OnDisableRecordingAcked,
|
||||
+ base::Unretained(this),
|
||||
+ make_scoped_refptr(trace_message_filter),
|
||||
+ std::vector<std::string>()));
|
||||
+ }
|
||||
+ }
|
||||
+ if (pending_capture_monitoring_snapshot_ack_count_ > 0) {
|
||||
+ TraceMessageFilterSet::const_iterator it =
|
||||
+ pending_capture_monitoring_filters_.find(trace_message_filter);
|
||||
+ if (it != pending_capture_monitoring_filters_.end()) {
|
||||
+ BrowserThread::PostTask(BrowserThread::UI, FROM_HERE,
|
||||
+ base::Bind(&TracingControllerImpl::OnCaptureMonitoringSnapshotAcked,
|
||||
+ base::Unretained(this),
|
||||
+ make_scoped_refptr(trace_message_filter)));
|
||||
+ }
|
||||
+ }
|
||||
+ if (pending_trace_buffer_percent_full_ack_count_ > 0) {
|
||||
+ TraceMessageFilterSet::const_iterator it =
|
||||
+ pending_trace_buffer_percent_full_filters_.find(trace_message_filter);
|
||||
+ if (it != pending_trace_buffer_percent_full_filters_.end()) {
|
||||
+ BrowserThread::PostTask(BrowserThread::UI, FROM_HERE,
|
||||
+ base::Bind(&TracingControllerImpl::OnTraceBufferPercentFullReply,
|
||||
+ base::Unretained(this),
|
||||
+ make_scoped_refptr(trace_message_filter),
|
||||
+ 0));
|
||||
+ }
|
||||
+ }
|
||||
+
|
||||
trace_message_filters_.erase(trace_message_filter);
|
||||
}
|
||||
|
||||
void TracingControllerImpl::OnDisableRecordingAcked(
|
||||
+ TraceMessageFilter* trace_message_filter,
|
||||
const std::vector<std::string>& known_category_groups) {
|
||||
if (!BrowserThread::CurrentlyOn(BrowserThread::UI)) {
|
||||
BrowserThread::PostTask(BrowserThread::UI, FROM_HERE,
|
||||
base::Bind(&TracingControllerImpl::OnDisableRecordingAcked,
|
||||
- base::Unretained(this), known_category_groups));
|
||||
+ base::Unretained(this),
|
||||
+ make_scoped_refptr(trace_message_filter),
|
||||
+ known_category_groups));
|
||||
return;
|
||||
}
|
||||
|
||||
@@ -456,6 +499,12 @@
|
||||
if (pending_disable_recording_ack_count_ == 0)
|
||||
return;
|
||||
|
||||
+ if (trace_message_filter &&
|
||||
+ !pending_disable_recording_filters_.erase(trace_message_filter)) {
|
||||
+ // The response from the specified message filter has already been received.
|
||||
+ return;
|
||||
+ }
|
||||
+
|
||||
if (--pending_disable_recording_ack_count_ == 1) {
|
||||
// All acks from subprocesses have been received. Now flush the local trace.
|
||||
// During or after this call, our OnLocalTraceDataCollected will be
|
||||
@@ -497,17 +546,25 @@
|
||||
result_file_.reset();
|
||||
}
|
||||
|
||||
-void TracingControllerImpl::OnCaptureMonitoringSnapshotAcked() {
|
||||
+void TracingControllerImpl::OnCaptureMonitoringSnapshotAcked(
|
||||
+ TraceMessageFilter* trace_message_filter) {
|
||||
if (!BrowserThread::CurrentlyOn(BrowserThread::UI)) {
|
||||
BrowserThread::PostTask(BrowserThread::UI, FROM_HERE,
|
||||
base::Bind(&TracingControllerImpl::OnCaptureMonitoringSnapshotAcked,
|
||||
- base::Unretained(this)));
|
||||
+ base::Unretained(this),
|
||||
+ make_scoped_refptr(trace_message_filter)));
|
||||
return;
|
||||
}
|
||||
|
||||
if (pending_capture_monitoring_snapshot_ack_count_ == 0)
|
||||
return;
|
||||
|
||||
+ if (trace_message_filter &&
|
||||
+ !pending_capture_monitoring_filters_.erase(trace_message_filter)) {
|
||||
+ // The response from the specified message filter has already been received.
|
||||
+ return;
|
||||
+ }
|
||||
+
|
||||
if (--pending_capture_monitoring_snapshot_ack_count_ == 1) {
|
||||
// All acks from subprocesses have been received. Now flush the local trace.
|
||||
// During or after this call, our OnLocalMonitoringTraceDataCollected
|
||||
@@ -582,7 +639,7 @@
|
||||
// Simulate an DisableRecordingAcked for the local trace.
|
||||
std::vector<std::string> category_groups;
|
||||
TraceLog::GetInstance()->GetKnownCategoryGroups(&category_groups);
|
||||
- OnDisableRecordingAcked(category_groups);
|
||||
+ OnDisableRecordingAcked(NULL, category_groups);
|
||||
}
|
||||
|
||||
void TracingControllerImpl::OnLocalMonitoringTraceDataCollected(
|
||||
@@ -595,20 +652,30 @@
|
||||
return;
|
||||
|
||||
// Simulate an CaptureMonitoringSnapshotAcked for the local trace.
|
||||
- OnCaptureMonitoringSnapshotAcked();
|
||||
+ OnCaptureMonitoringSnapshotAcked(NULL);
|
||||
}
|
||||
|
||||
-void TracingControllerImpl::OnTraceBufferPercentFullReply(float percent_full) {
|
||||
+void TracingControllerImpl::OnTraceBufferPercentFullReply(
|
||||
+ TraceMessageFilter* trace_message_filter,
|
||||
+ float percent_full) {
|
||||
if (!BrowserThread::CurrentlyOn(BrowserThread::UI)) {
|
||||
BrowserThread::PostTask(BrowserThread::UI, FROM_HERE,
|
||||
base::Bind(&TracingControllerImpl::OnTraceBufferPercentFullReply,
|
||||
- base::Unretained(this), percent_full));
|
||||
+ base::Unretained(this),
|
||||
+ make_scoped_refptr(trace_message_filter),
|
||||
+ percent_full));
|
||||
return;
|
||||
}
|
||||
|
||||
if (pending_trace_buffer_percent_full_ack_count_ == 0)
|
||||
return;
|
||||
|
||||
+ if (trace_message_filter &&
|
||||
+ !pending_trace_buffer_percent_full_filters_.erase(trace_message_filter)) {
|
||||
+ // The response from the specified message filter has already been received.
|
||||
+ return;
|
||||
+ }
|
||||
+
|
||||
maximum_trace_buffer_percent_full_ =
|
||||
std::max(maximum_trace_buffer_percent_full_, percent_full);
|
||||
|
||||
@@ -625,6 +692,7 @@
|
||||
BrowserThread::PostTask(BrowserThread::UI, FROM_HERE,
|
||||
base::Bind(&TracingControllerImpl::OnTraceBufferPercentFullReply,
|
||||
base::Unretained(this),
|
||||
+ make_scoped_refptr(trace_message_filter),
|
||||
TraceLog::GetInstance()->GetBufferPercentFull()));
|
||||
}
|
||||
}
|
||||
Index: tracing_controller_impl.h
|
||||
===================================================================
|
||||
--- tracing_controller_impl.h (revision 241258)
|
||||
+++ tracing_controller_impl.h (working copy)
|
||||
@@ -55,7 +55,7 @@
|
||||
virtual bool CancelWatchEvent() OVERRIDE;
|
||||
|
||||
private:
|
||||
- typedef std::set<scoped_refptr<TraceMessageFilter> > TraceMessageFilterMap;
|
||||
+ typedef std::set<scoped_refptr<TraceMessageFilter> > TraceMessageFilterSet;
|
||||
class ResultFile;
|
||||
|
||||
friend struct base::DefaultLazyInstanceTraits<TracingControllerImpl>;
|
||||
@@ -107,23 +107,30 @@
|
||||
bool has_more_events);
|
||||
|
||||
void OnDisableRecordingAcked(
|
||||
+ TraceMessageFilter* trace_message_filter,
|
||||
const std::vector<std::string>& known_category_groups);
|
||||
void OnResultFileClosed();
|
||||
|
||||
- void OnCaptureMonitoringSnapshotAcked();
|
||||
+ void OnCaptureMonitoringSnapshotAcked(
|
||||
+ TraceMessageFilter* trace_message_filter);
|
||||
void OnMonitoringSnapshotFileClosed();
|
||||
|
||||
- void OnTraceBufferPercentFullReply(float percent_full);
|
||||
+ void OnTraceBufferPercentFullReply(
|
||||
+ TraceMessageFilter* trace_message_filter,
|
||||
+ float percent_full);
|
||||
|
||||
void OnWatchEventMatched();
|
||||
|
||||
- TraceMessageFilterMap trace_message_filters_;
|
||||
+ TraceMessageFilterSet trace_message_filters_;
|
||||
// Pending acks for DisableRecording.
|
||||
int pending_disable_recording_ack_count_;
|
||||
+ TraceMessageFilterSet pending_disable_recording_filters_;
|
||||
// Pending acks for CaptureMonitoringSnapshot.
|
||||
int pending_capture_monitoring_snapshot_ack_count_;
|
||||
+ TraceMessageFilterSet pending_capture_monitoring_filters_;
|
||||
// Pending acks for GetTraceBufferPercentFull.
|
||||
int pending_trace_buffer_percent_full_ack_count_;
|
||||
+ TraceMessageFilterSet pending_trace_buffer_percent_full_filters_;
|
||||
float maximum_trace_buffer_percent_full_;
|
||||
|
||||
bool is_recording_;
|
@@ -1,12 +1,12 @@
|
||||
Index: underlay_opengl_hosting_window.h
|
||||
===================================================================
|
||||
--- underlay_opengl_hosting_window.h (revision 214871)
|
||||
--- underlay_opengl_hosting_window.h (revision 248478)
|
||||
+++ underlay_opengl_hosting_window.h (working copy)
|
||||
@@ -12,7 +12,7 @@
|
||||
// Common base class for windows that host a OpenGL surface that renders under
|
||||
// the window. Contains methods relating to hole punching so that the OpenGL
|
||||
// surface is visible through the window.
|
||||
-UI_EXPORT
|
||||
-UI_BASE_EXPORT
|
||||
+__attribute__((visibility("default")))
|
||||
@interface UnderlayOpenGLHostingWindow : NSWindow
|
||||
@end
|
||||
|
@@ -1,8 +1,8 @@
|
||||
Index: desktop_aura/desktop_root_window_host_win.cc
|
||||
===================================================================
|
||||
--- desktop_aura/desktop_root_window_host_win.cc (revision 242756)
|
||||
--- desktop_aura/desktop_root_window_host_win.cc (revision 248478)
|
||||
+++ desktop_aura/desktop_root_window_host_win.cc (working copy)
|
||||
@@ -131,7 +131,9 @@
|
||||
@@ -132,7 +132,9 @@
|
||||
native_widget_delegate_);
|
||||
|
||||
HWND parent_hwnd = NULL;
|
||||
@@ -13,9 +13,9 @@ Index: desktop_aura/desktop_root_window_host_win.cc
|
||||
parent_hwnd =
|
||||
params.parent->GetDispatcher()->host()->GetAcceleratedWidget();
|
||||
}
|
||||
@@ -752,7 +754,7 @@
|
||||
@@ -771,7 +773,7 @@
|
||||
|
||||
void DesktopRootWindowHostWin::HandleCreate() {
|
||||
void DesktopWindowTreeHostWin::HandleCreate() {
|
||||
// TODO(beng): moar
|
||||
- NOTIMPLEMENTED();
|
||||
+ // NOTIMPLEMENTED();
|
||||
@@ -24,7 +24,7 @@ Index: desktop_aura/desktop_root_window_host_win.cc
|
||||
|
||||
Index: desktop_aura/desktop_screen_win.cc
|
||||
===================================================================
|
||||
--- desktop_aura/desktop_screen_win.cc (revision 242756)
|
||||
--- desktop_aura/desktop_screen_win.cc (revision 248478)
|
||||
+++ desktop_aura/desktop_screen_win.cc (working copy)
|
||||
@@ -54,6 +54,8 @@
|
||||
}
|
||||
@@ -37,7 +37,7 @@ Index: desktop_aura/desktop_screen_win.cc
|
||||
}
|
||||
Index: widget.h
|
||||
===================================================================
|
||||
--- widget.h (revision 242756)
|
||||
--- widget.h (revision 248478)
|
||||
+++ widget.h (working copy)
|
||||
@@ -197,6 +197,7 @@
|
||||
// Should the widget be double buffered? Default is false.
|
||||
|
92
patch/patches/web_contents_304341.patch
Normal file
92
patch/patches/web_contents_304341.patch
Normal file
@@ -0,0 +1,92 @@
|
||||
Index: browser/browser_plugin/browser_plugin_guest.cc
|
||||
===================================================================
|
||||
--- browser/browser_plugin/browser_plugin_guest.cc (revision 248478)
|
||||
+++ browser/browser_plugin/browser_plugin_guest.cc (working copy)
|
||||
@@ -785,7 +785,8 @@
|
||||
return this;
|
||||
}
|
||||
|
||||
-bool BrowserPluginGuest::HandleContextMenu(const ContextMenuParams& params) {
|
||||
+bool BrowserPluginGuest::HandleContextMenu(RenderFrameHost* render_frame_host,
|
||||
+ const ContextMenuParams& params) {
|
||||
// TODO(fsamuel): We show the regular page context menu handler for now until
|
||||
// we implement the Apps Context Menu API for Browser Plugin (see
|
||||
// http://crbug.com/140315).
|
||||
Index: browser/browser_plugin/browser_plugin_guest.h
|
||||
===================================================================
|
||||
--- browser/browser_plugin/browser_plugin_guest.h (revision 248478)
|
||||
+++ browser/browser_plugin/browser_plugin_guest.h (working copy)
|
||||
@@ -200,7 +200,8 @@
|
||||
double progress) OVERRIDE;
|
||||
virtual void CloseContents(WebContents* source) OVERRIDE;
|
||||
virtual JavaScriptDialogManager* GetJavaScriptDialogManager() OVERRIDE;
|
||||
- virtual bool HandleContextMenu(const ContextMenuParams& params) OVERRIDE;
|
||||
+ virtual bool HandleContextMenu(RenderFrameHost* render_frame_host,
|
||||
+ const ContextMenuParams& params) OVERRIDE;
|
||||
virtual void HandleKeyboardEvent(
|
||||
WebContents* source,
|
||||
const NativeWebKeyboardEvent& event) OVERRIDE;
|
||||
Index: browser/web_contents/web_contents_impl.cc
|
||||
===================================================================
|
||||
--- browser/web_contents/web_contents_impl.cc (revision 248478)
|
||||
+++ browser/web_contents/web_contents_impl.cc (working copy)
|
||||
@@ -2687,7 +2687,7 @@
|
||||
void WebContentsImpl::ShowContextMenu(RenderFrameHost* render_frame_host,
|
||||
const ContextMenuParams& params) {
|
||||
// Allow WebContentsDelegates to handle the context menu operation first.
|
||||
- if (delegate_ && delegate_->HandleContextMenu(params))
|
||||
+ if (delegate_ && delegate_->HandleContextMenu(render_frame_host, params))
|
||||
return;
|
||||
|
||||
render_view_host_delegate_view_->ShowContextMenu(render_frame_host, params);
|
||||
Index: browser/web_contents/web_contents_view_mac.mm
|
||||
===================================================================
|
||||
--- browser/web_contents/web_contents_view_mac.mm (revision 248478)
|
||||
+++ browser/web_contents/web_contents_view_mac.mm (working copy)
|
||||
@@ -233,12 +233,6 @@
|
||||
void WebContentsViewMac::ShowContextMenu(
|
||||
content::RenderFrameHost* render_frame_host,
|
||||
const ContextMenuParams& params) {
|
||||
- // Allow delegates to handle the context menu operation first.
|
||||
- if (web_contents_->GetDelegate() &&
|
||||
- web_contents_->GetDelegate()->HandleContextMenu(params)) {
|
||||
- return;
|
||||
- }
|
||||
-
|
||||
if (delegate())
|
||||
delegate()->ShowContextMenu(render_frame_host, params);
|
||||
else
|
||||
Index: public/browser/web_contents_delegate.cc
|
||||
===================================================================
|
||||
--- public/browser/web_contents_delegate.cc (revision 248478)
|
||||
+++ public/browser/web_contents_delegate.cc (working copy)
|
||||
@@ -76,6 +76,7 @@
|
||||
}
|
||||
|
||||
bool WebContentsDelegate::HandleContextMenu(
|
||||
+ RenderFrameHost* render_frame_host,
|
||||
const content::ContextMenuParams& params) {
|
||||
return false;
|
||||
}
|
||||
Index: public/browser/web_contents_delegate.h
|
||||
===================================================================
|
||||
--- public/browser/web_contents_delegate.h (revision 248478)
|
||||
+++ public/browser/web_contents_delegate.h (working copy)
|
||||
@@ -35,6 +35,7 @@
|
||||
class DownloadItem;
|
||||
class JavaScriptDialogManager;
|
||||
class PageState;
|
||||
+class RenderFrameHost;
|
||||
class RenderViewHost;
|
||||
class SessionStorageNamespace;
|
||||
class WebContents;
|
||||
@@ -228,7 +229,8 @@
|
||||
virtual int GetExtraRenderViewHeight() const;
|
||||
|
||||
// Returns true if the context menu operation was handled by the delegate.
|
||||
- virtual bool HandleContextMenu(const content::ContextMenuParams& params);
|
||||
+ virtual bool HandleContextMenu(RenderFrameHost* render_frame_host,
|
||||
+ const content::ContextMenuParams& params);
|
||||
|
||||
// Opens source view for given WebContents that is navigated to the given
|
||||
// page url.
|
@@ -1,8 +1,8 @@
|
||||
Index: WebNode.cpp
|
||||
===================================================================
|
||||
--- WebNode.cpp (revision 160715)
|
||||
--- WebNode.cpp (revision 166298)
|
||||
+++ WebNode.cpp (working copy)
|
||||
@@ -169,7 +169,7 @@
|
||||
@@ -171,7 +171,7 @@
|
||||
void WebNode::addEventListener(const WebString& eventType, WebDOMEventListener* listener, bool useCapture)
|
||||
{
|
||||
// Please do not add more eventTypes to this list without an API review.
|
||||
|
@@ -1,6 +1,6 @@
|
||||
Index: public/web/WebView.h
|
||||
===================================================================
|
||||
--- public/web/WebView.h (revision 164381)
|
||||
--- public/web/WebView.h (revision 166298)
|
||||
+++ public/web/WebView.h (working copy)
|
||||
@@ -417,6 +417,7 @@
|
||||
|
||||
@@ -12,9 +12,9 @@ Index: public/web/WebView.h
|
||||
virtual void hidePopups() = 0;
|
||||
Index: Source/web/ChromeClientImpl.cpp
|
||||
===================================================================
|
||||
--- Source/web/ChromeClientImpl.cpp (revision 164381)
|
||||
--- Source/web/ChromeClientImpl.cpp (revision 166298)
|
||||
+++ Source/web/ChromeClientImpl.cpp (working copy)
|
||||
@@ -851,7 +851,7 @@
|
||||
@@ -828,7 +828,7 @@
|
||||
|
||||
PassRefPtr<PopupMenu> ChromeClientImpl::createPopupMenu(Frame& frame, PopupMenuClient* client) const
|
||||
{
|
||||
@@ -25,9 +25,9 @@ Index: Source/web/ChromeClientImpl.cpp
|
||||
return adoptRef(new PopupMenuChromium(frame, client));
|
||||
Index: Source/web/WebViewImpl.cpp
|
||||
===================================================================
|
||||
--- Source/web/WebViewImpl.cpp (revision 164381)
|
||||
--- Source/web/WebViewImpl.cpp (revision 166298)
|
||||
+++ Source/web/WebViewImpl.cpp (working copy)
|
||||
@@ -348,6 +348,7 @@
|
||||
@@ -340,6 +340,7 @@
|
||||
, m_fakePageScaleAnimationPageScaleFactor(0)
|
||||
, m_fakePageScaleAnimationUseAnchor(false)
|
||||
, m_contextMenuAllowed(false)
|
||||
@@ -35,7 +35,7 @@ Index: Source/web/WebViewImpl.cpp
|
||||
, m_doingDragAndDrop(false)
|
||||
, m_ignoreInputEvents(false)
|
||||
, m_compositorDeviceScaleFactorOverride(0)
|
||||
@@ -3552,9 +3553,14 @@
|
||||
@@ -3517,9 +3518,14 @@
|
||||
updateLayerTreeViewport();
|
||||
}
|
||||
|
||||
@@ -53,9 +53,9 @@ Index: Source/web/WebViewImpl.cpp
|
||||
void WebViewImpl::startDragging(Frame* frame,
|
||||
Index: Source/web/WebViewImpl.h
|
||||
===================================================================
|
||||
--- Source/web/WebViewImpl.h (revision 164381)
|
||||
--- Source/web/WebViewImpl.h (revision 166298)
|
||||
+++ Source/web/WebViewImpl.h (working copy)
|
||||
@@ -398,7 +398,8 @@
|
||||
@@ -397,7 +397,8 @@
|
||||
|
||||
// Returns true if popup menus should be rendered by the browser, false if
|
||||
// they should be rendered by WebKit (which is the default).
|
||||
@@ -65,7 +65,7 @@ Index: Source/web/WebViewImpl.h
|
||||
|
||||
bool contextMenuAllowed() const
|
||||
{
|
||||
@@ -680,6 +681,8 @@
|
||||
@@ -683,6 +684,8 @@
|
||||
|
||||
bool m_contextMenuAllowed;
|
||||
|
||||
|
Reference in New Issue
Block a user