diff options
| author | Pierre Rossi <pierre.rossi@digia.com> | 2014-04-03 12:18:16 +0200 |
|---|---|---|
| committer | Andras Becsi <andras.becsi@theqtcompany.com> | 2014-12-09 10:54:15 +0100 |
| commit | b5d2ed5745e421d46ee010dba75e245b33f106ff (patch) | |
| tree | e7133f2571e685a5114dd41aea5aeba83bc80f39 | |
| parent | 1db557b28df4a668aaa57135698b0d4e8bdefc4f (diff) | |
| download | qtwebengine-chromium-b5d2ed5745e421d46ee010dba75e245b33f106ff.tar.gz | |
<chromium> fixup select elements.
We use RenderWidgetHostViewQt, not RWHVMac, even on Mac.
Since some platform decisions seem to have transpired all the way to
the content layer, we should simply use the web ui for select elements
on mac for the time being.
Change-Id: Ib430f8686862cf0958ccdbfefb8bfa36af590e97
Reviewed-by: Andras Becsi <andras.becsi@theqtcompany.com>
| -rw-r--r-- | chromium/content/browser/web_contents/web_contents_impl.cc | 4 | ||||
| -rw-r--r-- | chromium/content/renderer/render_thread_impl.cc | 2 |
2 files changed, 3 insertions, 3 deletions
diff --git a/chromium/content/browser/web_contents/web_contents_impl.cc b/chromium/content/browser/web_contents/web_contents_impl.cc index d3e8b13b9d3..06bfa9caa9d 100644 --- a/chromium/content/browser/web_contents/web_contents_impl.cc +++ b/chromium/content/browser/web_contents/web_contents_impl.cc @@ -1636,7 +1636,7 @@ void WebContentsImpl::CreateNewWidget(int render_process_id, // Save the created widget associated with the route so we can show it later. pending_widget_views_[route_id] = widget_view; -#if defined(OS_MACOSX) +#if defined(OS_MACOSX) && !defined(TOOLKIT_QT) // A RenderWidgetHostViewMac has lifetime scoped to the view. We'll retain it // to allow it to survive the trip without being hosted. base::mac::NSObjectRetain(widget_view->GetNativeView()); @@ -1708,7 +1708,7 @@ void WebContentsImpl::ShowCreatedWidget(int route_id, // used to implement Pepper Flash fullscreen. render_widget_host_impl->set_allow_privileged_mouse_lock(is_fullscreen); -#if defined(OS_MACOSX) +#if defined(OS_MACOSX) && !defined(TOOLKIT_QT) // A RenderWidgetHostViewMac has lifetime scoped to the view. Now that it's // properly embedded (or purposefully ignored) we can release the retain we // took in CreateNewWidget(). diff --git a/chromium/content/renderer/render_thread_impl.cc b/chromium/content/renderer/render_thread_impl.cc index 6984b137743..0c665858ed4 100644 --- a/chromium/content/renderer/render_thread_impl.cc +++ b/chromium/content/renderer/render_thread_impl.cc @@ -415,7 +415,7 @@ void RenderThreadImpl::Init() { base::PlatformThread::CurrentId(), kTraceEventRendererMainThreadSortIndex); -#if defined(OS_MACOSX) || defined(OS_ANDROID) +#if (defined(OS_MACOSX) || defined(OS_ANDROID)) && !defined(TOOLKIT_QT) // On Mac and Android, the select popups are rendered by the browser. blink::WebView::setUseExternalPopupMenus(true); #endif |