Enabling SSL for search engines that now support it

Enabling SSL for search engines that now support it

BUG=624794
R=pkasting@chromium.org

[Peter Kasting, 2016-08-20 02:56:42]

Sorry, I only just now noticed this in my queue.  To make sure reviewers see
patches you post, make sure to "Publish+Mail Comments" after setting a reviewer
or when you make changes.

LGTM with one change, although first you'll want to update your source tree so
you bump the version number to the right value.

https://codereview.chromium.org/2116243002/diff/1/components/search_engines/p...
File components/search_engines/prepopulated_engines.json (right):

https://codereview.chromium.org/2116243002/diff/1/components/search_engines/p...
components/search_engines/prepopulated_engines.json:33: "kCurrentDataVersion":
91
You need to bump this number.

[Peter Kasting, 2017-02-11 02:02:27]

This CL is old, but still relevant.  Any chance of updating it and landing?

[yakovsh, 2017-03-17 16:44:23]

Description was changed from

==========
Enabling SSL for search engines that now support it

BUG=624794
R=pkasting@chromium.org
==========

to

==========
Enabling SSL for search engines that now support it

BUG=624794
R=pkasting@chromium.org
==========

[yakovsh, 2017-03-17 16:44:45]

On 2017/02/11 02:02:27, Peter Kasting wrote:
> This CL is old, but still relevant.  Any chance of updating it and landing?

Added this as a new CL:
https://codereview.chromium.org/2753293002

[yakovsh, 2017-03-17 16:48:14]

Hi,

I added a new CL here:
https://codereview.chromium.org/2753293002

On Fri, Feb 10, 2017 at 9:02 PM,  <pkasting@chromium.org> wrote:
> This CL is old, but still relevant. Any chance of updating it and landing?
>
> https://codereview.chromium.org/2116243002/

-- 
You received this message because you are subscribed to the Google Groups
"Chromium-reviews" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.