|
 |
9071ee2 |
From cf4c534f04c223f8a9d65407852e2a531a6d7fb6 Mon Sep 17 00:00:00 2001
|
|
|
9071ee2 |
From: Raphael Kubo da Costa <raphael.kubo.da.costa@intel.com>
|
|
|
9071ee2 |
Date: Thu, 14 Feb 2019 22:22:21 +0000
|
|
|
9071ee2 |
Subject: [PATCH] chrome/browser: Replace some forward declarations with actual
|
|
|
9071ee2 |
includes
|
|
|
9071ee2 |
MIME-Version: 1.0
|
|
|
9071ee2 |
Content-Type: text/plain; charset=UTF-8
|
|
|
9071ee2 |
Content-Transfer-Encoding: 8bit
|
|
|
9071ee2 |
|
|
|
9071ee2 |
This fixes the GCC build which was failing like this:
|
|
|
9071ee2 |
|
|
|
9071ee2 |
../../base/scoped_observer.h: In instantiation of ‘void ScopedObserver<Source, Observer>::RemoveAll() [with Source = TabStripModel; Observer = TabStripModelObserver]’:
|
|
|
9071ee2 |
../../base/scoped_observer.h:26:5: required from ‘ScopedObserver<Source, Observer>::~ScopedObserver() [with Source = TabStripModel; Observer = TabStripModelObserver]’
|
|
|
9071ee2 |
../../chrome/browser/ui/views/extensions/extension_popup.h:115:70: required from here
|
|
|
9071ee2 |
../../base/scoped_observer.h:45:20: error: invalid use of incomplete type ‘class TabStripModel’
|
|
|
9071ee2 |
sources_[i]->RemoveObserver(observer_);
|
|
|
9071ee2 |
~~~~~~~~~~~~~^~~~~~~~~~~~~~
|
|
|
9071ee2 |
|
|
|
9071ee2 |
This is caused by https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89311 ("Brace
|
|
|
9071ee2 |
initialization needlessly invokes destructor"), i.e. having something like
|
|
|
9071ee2 |
|
|
|
9071ee2 |
ScopedObserver<T, U> observer_{this};
|
|
|
9071ee2 |
|
|
|
9071ee2 |
in a header declaration requires T and U to be fully declared because
|
|
|
9071ee2 |
ScopedObserver's destructor references them. In a few cases, T was only
|
|
|
9071ee2 |
forward-declared.
|
|
|
9071ee2 |
|
|
|
9071ee2 |
Bug: 819294
|
|
|
9071ee2 |
Change-Id: Ie5b9dc2745e27d4532c5539e3845a8c9147a0595
|
|
|
9071ee2 |
Reviewed-on: https://chromium-review.googlesource.com/c/1472576
|
|
|
9071ee2 |
Auto-Submit: Raphael Kubo da Costa <raphael.kubo.da.costa@intel.com>
|
|
|
9071ee2 |
Commit-Queue: Alan Cutter <alancutter@chromium.org>
|
|
|
9071ee2 |
Reviewed-by: Finnur Thorarinsson <finnur@chromium.org>
|
|
|
9071ee2 |
Reviewed-by: Alan Cutter <alancutter@chromium.org>
|
|
|
9071ee2 |
Cr-Commit-Position: refs/heads/master@{#632385}
|
|
|
9071ee2 |
---
|
|
|
9071ee2 |
chrome/browser/ui/views/extensions/extension_popup.cc | 1 -
|
|
|
9071ee2 |
chrome/browser/ui/views/extensions/extension_popup.h | 1 +
|
|
|
9071ee2 |
chrome/browser/ui/views/toolbar/extension_toolbar_menu_view.cc | 1 -
|
|
|
9071ee2 |
chrome/browser/ui/views/toolbar/extension_toolbar_menu_view.h | 2 +-
|
|
|
9071ee2 |
.../web_applications/extensions/bookmark_app_tab_helper.cc | 1 -
|
|
|
9071ee2 |
.../web_applications/extensions/bookmark_app_tab_helper.h | 2 +-
|
|
|
9071ee2 |
6 files changed, 3 insertions(+), 5 deletions(-)
|
|
|
9071ee2 |
|
|
|
9071ee2 |
diff --git a/chrome/browser/ui/views/extensions/extension_popup.cc b/chrome/browser/ui/views/extensions/extension_popup.cc
|
|
|
9071ee2 |
index 3f0985d383dd..57d16aba9e9f 100644
|
|
|
9071ee2 |
--- a/chrome/browser/ui/views/extensions/extension_popup.cc
|
|
|
9071ee2 |
+++ b/chrome/browser/ui/views/extensions/extension_popup.cc
|
|
|
9071ee2 |
@@ -8,7 +8,6 @@
|
|
|
9071ee2 |
#include "chrome/browser/devtools/devtools_window.h"
|
|
|
9071ee2 |
#include "chrome/browser/extensions/extension_view_host.h"
|
|
|
9071ee2 |
#include "chrome/browser/ui/browser.h"
|
|
|
9071ee2 |
-#include "chrome/browser/ui/tabs/tab_strip_model.h"
|
|
|
9071ee2 |
#include "content/public/browser/devtools_agent_host.h"
|
|
|
9071ee2 |
#include "content/public/browser/notification_details.h"
|
|
|
9071ee2 |
#include "content/public/browser/notification_source.h"
|
|
|
9071ee2 |
diff --git a/chrome/browser/ui/views/extensions/extension_popup.h b/chrome/browser/ui/views/extensions/extension_popup.h
|
|
|
9071ee2 |
index 3661b5bda950..9018efa0fea5 100644
|
|
|
9071ee2 |
--- a/chrome/browser/ui/views/extensions/extension_popup.h
|
|
|
9071ee2 |
+++ b/chrome/browser/ui/views/extensions/extension_popup.h
|
|
|
9071ee2 |
@@ -9,6 +9,7 @@
|
|
|
9071ee2 |
#include "base/compiler_specific.h"
|
|
|
9071ee2 |
#include "base/macros.h"
|
|
|
9071ee2 |
#include "base/scoped_observer.h"
|
|
|
9071ee2 |
+#include "chrome/browser/ui/tabs/tab_strip_model.h"
|
|
|
9071ee2 |
#include "chrome/browser/ui/tabs/tab_strip_model_observer.h"
|
|
|
9071ee2 |
#include "chrome/browser/ui/views/extensions/extension_view_views.h"
|
|
|
9071ee2 |
#include "content/public/browser/devtools_agent_host_observer.h"
|
|
|
9071ee2 |
diff --git a/chrome/browser/ui/views/toolbar/extension_toolbar_menu_view.h b/chrome/browser/ui/views/toolbar/extension_toolbar_menu_view.h
|
|
|
9071ee2 |
index 2a3ea8e4f91f..ac44a4f05dec 100644
|
|
|
9071ee2 |
--- a/chrome/browser/ui/views/toolbar/extension_toolbar_menu_view.h
|
|
|
9071ee2 |
+++ b/chrome/browser/ui/views/toolbar/extension_toolbar_menu_view.h
|
|
|
9071ee2 |
@@ -8,6 +8,7 @@
|
|
|
9071ee2 |
#include "base/macros.h"
|
|
|
9071ee2 |
#include "base/memory/weak_ptr.h"
|
|
|
9071ee2 |
#include "base/scoped_observer.h"
|
|
|
9071ee2 |
+#include "chrome/browser/ui/toolbar/toolbar_actions_bar.h"
|
|
|
9071ee2 |
#include "chrome/browser/ui/toolbar/toolbar_actions_bar_observer.h"
|
|
|
9071ee2 |
#include "chrome/browser/ui/views/toolbar/app_menu_observer.h"
|
|
|
9071ee2 |
#include "ui/views/controls/scroll_view.h"
|
|
|
9071ee2 |
@@ -15,7 +16,6 @@ class AppMenu;
|
|
|
9071ee2 |
class AppMenu;
|
|
|
9071ee2 |
class Browser;
|
|
|
9071ee2 |
class BrowserActionsContainer;
|
|
|
9071ee2 |
-class ToolbarActionsBar;
|
|
|
9071ee2 |
|
|
|
9071ee2 |
namespace views {
|
|
|
9071ee2 |
class MenuItemView;
|