Revert of Fix assert in Flash component install.

Revert of Fix assert in Flash component install. (patchset #1 id:1 of https://codereview.chromium.org/2109873002/ )

Reason for revert:
Compilation failure in Google Chrome Mac

https://build.chromium.org/p/chromium.chrome/builders/Google%20Chrome%20Mac/builds/11620

Detailed message:

In file included from ../../base/bind_internal.h:13:
../../base/bind_helpers.h:412:7: error: implicit instantiation of undefined template 'base::internal::DropTypeListItemImpl<1, base::internal::TypeList<> >'
    : DropTypeListItemImpl<n - 1, TypeList<List...>> {};
      ^
../../base/bind_helpers.h:426:1: note: in instantiation of template class 'base::internal::DropTypeListItemImpl<2, base::internal::TypeList<component_updater::ComponentUpdateService *> >' requested here
using DropTypeListItem = typename DropTypeListItemImpl<n, List>::Type;
^
../../base/bind_internal.h:395:23: note: in instantiation of template type alias 'DropTypeListItem' requested here
  using UnboundArgs = DropTypeListItem<sizeof...(BoundArgs), Args>;
                      ^
../../base/bind_internal.h:437:1: note: in instantiation of template class 'base::internal::MakeUnboundRunTypeImpl<void (*)(component_updater::ComponentUpdateService *), const base::FilePath &, const base::Version &>' requested here
using MakeUnboundRunType =
^
../../base/bind.h:50:23: note: in instantiation of template type alias 'MakeUnboundRunType' requested here
inline base::Callback<MakeUnboundRunType<Functor, Args...>>
                      ^
../../chrome/browser/component_updater/pepper_flash_component_installer.cc:113:7: note: while substituting deduced template arguments into function template 'Bind' [with Functor = void (*)(component_updater::ComponentUpdateService *), Args = <const base::FilePath &, const base::Version &>]
      base::Bind(&RegisterPepperFlashWithChrome, path, version));
      ^
../../base/bind_helpers.h:407:8: note: template is declared here
struct DropTypeListItemImpl;
       ^
../../base/bind_helpers.h:480:1: error: implicit instantiation of undefined template 'base::internal::MakeFunctionTypeImpl<void, int>'
using MakeFunctionType = typename MakeFunctionTypeImpl<R, ArgList>::Type;
^
../../base/bind_internal.h:396:16: note: in instantiation of template type alias 'MakeFunctionType' requested here
  using Type = MakeFunctionType<ReturnType, UnboundArgs>;
               ^
../../base/bind_internal.h:437:1: note: in instantiation of template class 'base::internal::MakeUnboundRunTypeImpl<void (*)(component_updater::ComponentUpdateService *), const base::FilePath &, const base::Version &>' requested here
using MakeUnboundRunType =
^
../../base/bind.h:50:23: note: in instantiation of template type alias 'MakeUnboundRunType' requested here
inline base::Callback<MakeUnboundRunType<Functor, Args...>>
                      ^
../../chrome/browser/component_updater/pepper_flash_component_installer.cc:113:7: note: while substituting deduced template arguments into function template 'Bind' [with Functor = void (*)(component_updater::ComponentUpdateService *), Args = <const base::FilePath &, const base::Version &>]
      base::Bind(&RegisterPepperFlashWithChrome, path, version));
      ^
../../base/bind_helpers.h:468:8: note: template is declared here
struct MakeFunctionTypeImpl;
       ^
../../chrome/browser/component_updater/pepper_flash_component_installer.cc:113:19: error: use of undeclared identifier 'RegisterPepperFlashWithChrome'
      base::Bind(&RegisterPepperFlashWithChrome, path, version));
                  ^
3 errors generated.

Original issue's description:
> Fix assert in Flash component install.
>
> BUG=
>
> Committed: https://crrev.com/df973871410891db6f94d224327d8c7cb4b8ae1e
> Cr-Commit-Position: refs/heads/master@{#403218}

TBR=sorin@chromium.org,waffles@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=

Committed: https://crrev.com/37261e5591e404d6bdcfedd78518812eab94f29d
Cr-Commit-Position: refs/heads/master@{#403233}

[kelvinp, 2016-06-30 18:42:57]

The CQ bit was checked by kelvinp@chromium.org

[kelvinp, 2016-06-30 18:42:57]

Created Revert of Fix assert in Flash component install.

[commit-bot: I haz the power, 2016-06-30 18:43:20]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-06-30 18:44:03]

Description was changed from

==========
Revert of Fix assert in Flash component install. (patchset #1 id:1 of
https://codereview.chromium.org/2109873002/ )

Reason for revert:
Compilation failure in Google Chrome Mac

https://build.chromium.org/p/chromium.chrome/builders/Google%20Chrome%20Mac/b...


Detailed message:

In file included from ../../base/bind_internal.h:13:
../../base/bind_helpers.h:412:7: error: implicit instantiation of undefined
template 'base::internal::DropTypeListItemImpl<1, base::internal::TypeList<> >'
    : DropTypeListItemImpl<n - 1, TypeList<List...>> {};
      ^
../../base/bind_helpers.h:426:1: note: in instantiation of template class
'base::internal::DropTypeListItemImpl<2,
base::internal::TypeList<component_updater::ComponentUpdateService *> >'
requested here
using DropTypeListItem = typename DropTypeListItemImpl<n, List>::Type;
^
../../base/bind_internal.h:395:23: note: in instantiation of template type alias
'DropTypeListItem' requested here
  using UnboundArgs = DropTypeListItem<sizeof...(BoundArgs), Args>;
                      ^
../../base/bind_internal.h:437:1: note: in instantiation of template class
'base::internal::MakeUnboundRunTypeImpl<void
(*)(component_updater::ComponentUpdateService *), const base::FilePath &, const
base::Version &>' requested here
using MakeUnboundRunType =
^
../../base/bind.h:50:23: note: in instantiation of template type alias
'MakeUnboundRunType' requested here
inline base::Callback<MakeUnboundRunType<Functor, Args...>>
                      ^
../../chrome/browser/component_updater/pepper_flash_component_installer.cc:113:7:
note: while substituting deduced template arguments into function template
'Bind' [with Functor = void (*)(component_updater::ComponentUpdateService *),
Args = <const base::FilePath &, const base::Version &>]
      base::Bind(&RegisterPepperFlashWithChrome, path, version));
      ^
../../base/bind_helpers.h:407:8: note: template is declared here
struct DropTypeListItemImpl;
       ^
../../base/bind_helpers.h:480:1: error: implicit instantiation of undefined
template 'base::internal::MakeFunctionTypeImpl<void, int>'
using MakeFunctionType = typename MakeFunctionTypeImpl<R, ArgList>::Type;
^
../../base/bind_internal.h:396:16: note: in instantiation of template type alias
'MakeFunctionType' requested here
  using Type = MakeFunctionType<ReturnType, UnboundArgs>;
               ^
../../base/bind_internal.h:437:1: note: in instantiation of template class
'base::internal::MakeUnboundRunTypeImpl<void
(*)(component_updater::ComponentUpdateService *), const base::FilePath &, const
base::Version &>' requested here
using MakeUnboundRunType =
^
../../base/bind.h:50:23: note: in instantiation of template type alias
'MakeUnboundRunType' requested here
inline base::Callback<MakeUnboundRunType<Functor, Args...>>
                      ^
../../chrome/browser/component_updater/pepper_flash_component_installer.cc:113:7:
note: while substituting deduced template arguments into function template
'Bind' [with Functor = void (*)(component_updater::ComponentUpdateService *),
Args = <const base::FilePath &, const base::Version &>]
      base::Bind(&RegisterPepperFlashWithChrome, path, version));
      ^
../../base/bind_helpers.h:468:8: note: template is declared here
struct MakeFunctionTypeImpl;
       ^
../../chrome/browser/component_updater/pepper_flash_component_installer.cc:113:19:
error: use of undeclared identifier 'RegisterPepperFlashWithChrome'
      base::Bind(&RegisterPepperFlashWithChrome, path, version));
                  ^
3 errors generated.



Original issue's description:
> Fix assert in Flash component install.
> 
> BUG=
> 
> Committed: https://crrev.com/df973871410891db6f94d224327d8c7cb4b8ae1e
> Cr-Commit-Position: refs/heads/master@{#403218}

TBR=sorin@chromium.org,waffles@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=
==========

to

==========
Revert of Fix assert in Flash component install. (patchset #1 id:1 of
https://codereview.chromium.org/2109873002/ )

Reason for revert:
Compilation failure in Google Chrome Mac

https://build.chromium.org/p/chromium.chrome/builders/Google%20Chrome%20Mac/b...


Detailed message:

In file included from ../../base/bind_internal.h:13:
../../base/bind_helpers.h:412:7: error: implicit instantiation of undefined
template 'base::internal::DropTypeListItemImpl<1, base::internal::TypeList<> >'
    : DropTypeListItemImpl<n - 1, TypeList<List...>> {};
      ^
../../base/bind_helpers.h:426:1: note: in instantiation of template class
'base::internal::DropTypeListItemImpl<2,
base::internal::TypeList<component_updater::ComponentUpdateService *> >'
requested here
using DropTypeListItem = typename DropTypeListItemImpl<n, List>::Type;
^
../../base/bind_internal.h:395:23: note: in instantiation of template type alias
'DropTypeListItem' requested here
  using UnboundArgs = DropTypeListItem<sizeof...(BoundArgs), Args>;
                      ^
../../base/bind_internal.h:437:1: note: in instantiation of template class
'base::internal::MakeUnboundRunTypeImpl<void
(*)(component_updater::ComponentUpdateService *), const base::FilePath &, const
base::Version &>' requested here
using MakeUnboundRunType =
^
../../base/bind.h:50:23: note: in instantiation of template type alias
'MakeUnboundRunType' requested here
inline base::Callback<MakeUnboundRunType<Functor, Args...>>
                      ^
../../chrome/browser/component_updater/pepper_flash_component_installer.cc:113:7:
note: while substituting deduced template arguments into function template
'Bind' [with Functor = void (*)(component_updater::ComponentUpdateService *),
Args = <const base::FilePath &, const base::Version &>]
      base::Bind(&RegisterPepperFlashWithChrome, path, version));
      ^
../../base/bind_helpers.h:407:8: note: template is declared here
struct DropTypeListItemImpl;
       ^
../../base/bind_helpers.h:480:1: error: implicit instantiation of undefined
template 'base::internal::MakeFunctionTypeImpl<void, int>'
using MakeFunctionType = typename MakeFunctionTypeImpl<R, ArgList>::Type;
^
../../base/bind_internal.h:396:16: note: in instantiation of template type alias
'MakeFunctionType' requested here
  using Type = MakeFunctionType<ReturnType, UnboundArgs>;
               ^
../../base/bind_internal.h:437:1: note: in instantiation of template class
'base::internal::MakeUnboundRunTypeImpl<void
(*)(component_updater::ComponentUpdateService *), const base::FilePath &, const
base::Version &>' requested here
using MakeUnboundRunType =
^
../../base/bind.h:50:23: note: in instantiation of template type alias
'MakeUnboundRunType' requested here
inline base::Callback<MakeUnboundRunType<Functor, Args...>>
                      ^
../../chrome/browser/component_updater/pepper_flash_component_installer.cc:113:7:
note: while substituting deduced template arguments into function template
'Bind' [with Functor = void (*)(component_updater::ComponentUpdateService *),
Args = <const base::FilePath &, const base::Version &>]
      base::Bind(&RegisterPepperFlashWithChrome, path, version));
      ^
../../base/bind_helpers.h:468:8: note: template is declared here
struct MakeFunctionTypeImpl;
       ^
../../chrome/browser/component_updater/pepper_flash_component_installer.cc:113:19:
error: use of undeclared identifier 'RegisterPepperFlashWithChrome'
      base::Bind(&RegisterPepperFlashWithChrome, path, version));
                  ^
3 errors generated.



Original issue's description:
> Fix assert in Flash component install.
> 
> BUG=
> 
> Committed: https://crrev.com/df973871410891db6f94d224327d8c7cb4b8ae1e
> Cr-Commit-Position: refs/heads/master@{#403218}

TBR=sorin@chromium.org,waffles@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=
==========

[commit-bot: I haz the power, 2016-06-30 18:44:04]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-06-30 18:45:48]

Description was changed from

==========
Revert of Fix assert in Flash component install. (patchset #1 id:1 of
https://codereview.chromium.org/2109873002/ )

Reason for revert:
Compilation failure in Google Chrome Mac

https://build.chromium.org/p/chromium.chrome/builders/Google%20Chrome%20Mac/b...


Detailed message:

In file included from ../../base/bind_internal.h:13:
../../base/bind_helpers.h:412:7: error: implicit instantiation of undefined
template 'base::internal::DropTypeListItemImpl<1, base::internal::TypeList<> >'
    : DropTypeListItemImpl<n - 1, TypeList<List...>> {};
      ^
../../base/bind_helpers.h:426:1: note: in instantiation of template class
'base::internal::DropTypeListItemImpl<2,
base::internal::TypeList<component_updater::ComponentUpdateService *> >'
requested here
using DropTypeListItem = typename DropTypeListItemImpl<n, List>::Type;
^
../../base/bind_internal.h:395:23: note: in instantiation of template type alias
'DropTypeListItem' requested here
  using UnboundArgs = DropTypeListItem<sizeof...(BoundArgs), Args>;
                      ^
../../base/bind_internal.h:437:1: note: in instantiation of template class
'base::internal::MakeUnboundRunTypeImpl<void
(*)(component_updater::ComponentUpdateService *), const base::FilePath &, const
base::Version &>' requested here
using MakeUnboundRunType =
^
../../base/bind.h:50:23: note: in instantiation of template type alias
'MakeUnboundRunType' requested here
inline base::Callback<MakeUnboundRunType<Functor, Args...>>
                      ^
../../chrome/browser/component_updater/pepper_flash_component_installer.cc:113:7:
note: while substituting deduced template arguments into function template
'Bind' [with Functor = void (*)(component_updater::ComponentUpdateService *),
Args = <const base::FilePath &, const base::Version &>]
      base::Bind(&RegisterPepperFlashWithChrome, path, version));
      ^
../../base/bind_helpers.h:407:8: note: template is declared here
struct DropTypeListItemImpl;
       ^
../../base/bind_helpers.h:480:1: error: implicit instantiation of undefined
template 'base::internal::MakeFunctionTypeImpl<void, int>'
using MakeFunctionType = typename MakeFunctionTypeImpl<R, ArgList>::Type;
^
../../base/bind_internal.h:396:16: note: in instantiation of template type alias
'MakeFunctionType' requested here
  using Type = MakeFunctionType<ReturnType, UnboundArgs>;
               ^
../../base/bind_internal.h:437:1: note: in instantiation of template class
'base::internal::MakeUnboundRunTypeImpl<void
(*)(component_updater::ComponentUpdateService *), const base::FilePath &, const
base::Version &>' requested here
using MakeUnboundRunType =
^
../../base/bind.h:50:23: note: in instantiation of template type alias
'MakeUnboundRunType' requested here
inline base::Callback<MakeUnboundRunType<Functor, Args...>>
                      ^
../../chrome/browser/component_updater/pepper_flash_component_installer.cc:113:7:
note: while substituting deduced template arguments into function template
'Bind' [with Functor = void (*)(component_updater::ComponentUpdateService *),
Args = <const base::FilePath &, const base::Version &>]
      base::Bind(&RegisterPepperFlashWithChrome, path, version));
      ^
../../base/bind_helpers.h:468:8: note: template is declared here
struct MakeFunctionTypeImpl;
       ^
../../chrome/browser/component_updater/pepper_flash_component_installer.cc:113:19:
error: use of undeclared identifier 'RegisterPepperFlashWithChrome'
      base::Bind(&RegisterPepperFlashWithChrome, path, version));
                  ^
3 errors generated.



Original issue's description:
> Fix assert in Flash component install.
> 
> BUG=
> 
> Committed: https://crrev.com/df973871410891db6f94d224327d8c7cb4b8ae1e
> Cr-Commit-Position: refs/heads/master@{#403218}

TBR=sorin@chromium.org,waffles@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=
==========

to

==========
Revert of Fix assert in Flash component install. (patchset #1 id:1 of
https://codereview.chromium.org/2109873002/ )

Reason for revert:
Compilation failure in Google Chrome Mac

https://build.chromium.org/p/chromium.chrome/builders/Google%20Chrome%20Mac/b...

Detailed message:

In file included from ../../base/bind_internal.h:13:
../../base/bind_helpers.h:412:7: error: implicit instantiation of undefined
template 'base::internal::DropTypeListItemImpl<1, base::internal::TypeList<> >'
    : DropTypeListItemImpl<n - 1, TypeList<List...>> {};
      ^
../../base/bind_helpers.h:426:1: note: in instantiation of template class
'base::internal::DropTypeListItemImpl<2,
base::internal::TypeList<component_updater::ComponentUpdateService *> >'
requested here
using DropTypeListItem = typename DropTypeListItemImpl<n, List>::Type;
^
../../base/bind_internal.h:395:23: note: in instantiation of template type alias
'DropTypeListItem' requested here
  using UnboundArgs = DropTypeListItem<sizeof...(BoundArgs), Args>;
                      ^
../../base/bind_internal.h:437:1: note: in instantiation of template class
'base::internal::MakeUnboundRunTypeImpl<void
(*)(component_updater::ComponentUpdateService *), const base::FilePath &, const
base::Version &>' requested here
using MakeUnboundRunType =
^
../../base/bind.h:50:23: note: in instantiation of template type alias
'MakeUnboundRunType' requested here
inline base::Callback<MakeUnboundRunType<Functor, Args...>>
                      ^
../../chrome/browser/component_updater/pepper_flash_component_installer.cc:113:7:
note: while substituting deduced template arguments into function template
'Bind' [with Functor = void (*)(component_updater::ComponentUpdateService *),
Args = <const base::FilePath &, const base::Version &>]
      base::Bind(&RegisterPepperFlashWithChrome, path, version));
      ^
../../base/bind_helpers.h:407:8: note: template is declared here
struct DropTypeListItemImpl;
       ^
../../base/bind_helpers.h:480:1: error: implicit instantiation of undefined
template 'base::internal::MakeFunctionTypeImpl<void, int>'
using MakeFunctionType = typename MakeFunctionTypeImpl<R, ArgList>::Type;
^
../../base/bind_internal.h:396:16: note: in instantiation of template type alias
'MakeFunctionType' requested here
  using Type = MakeFunctionType<ReturnType, UnboundArgs>;
               ^
../../base/bind_internal.h:437:1: note: in instantiation of template class
'base::internal::MakeUnboundRunTypeImpl<void
(*)(component_updater::ComponentUpdateService *), const base::FilePath &, const
base::Version &>' requested here
using MakeUnboundRunType =
^
../../base/bind.h:50:23: note: in instantiation of template type alias
'MakeUnboundRunType' requested here
inline base::Callback<MakeUnboundRunType<Functor, Args...>>
                      ^
../../chrome/browser/component_updater/pepper_flash_component_installer.cc:113:7:
note: while substituting deduced template arguments into function template
'Bind' [with Functor = void (*)(component_updater::ComponentUpdateService *),
Args = <const base::FilePath &, const base::Version &>]
      base::Bind(&RegisterPepperFlashWithChrome, path, version));
      ^
../../base/bind_helpers.h:468:8: note: template is declared here
struct MakeFunctionTypeImpl;
       ^
../../chrome/browser/component_updater/pepper_flash_component_installer.cc:113:19:
error: use of undeclared identifier 'RegisterPepperFlashWithChrome'
      base::Bind(&RegisterPepperFlashWithChrome, path, version));
                  ^
3 errors generated.

Original issue's description:
> Fix assert in Flash component install.
>
> BUG=
>
> Committed: https://crrev.com/df973871410891db6f94d224327d8c7cb4b8ae1e
> Cr-Commit-Position: refs/heads/master@{#403218}

TBR=sorin@chromium.org,waffles@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=

Committed: https://crrev.com/37261e5591e404d6bdcfedd78518812eab94f29d
Cr-Commit-Position: refs/heads/master@{#403233}
==========

[commit-bot: I haz the power, 2016-06-30 18:45:49]

Patchset 1 (id:??) landed as
https://crrev.com/37261e5591e404d6bdcfedd78518812eab94f29d
Cr-Commit-Position: refs/heads/master@{#403233}