Use https for naver search engine

Use https for naver search engine

Naver search engine has been providing https support for some time now.
Current http url even automatically redirects to https,
So changing naver urls to https where appropriate.

BUG=

Committed: https://crrev.com/c33c68d1a0e8cc2374b3b23255048313fbd0fe4b
Cr-Commit-Position: refs/heads/master@{#411584}

[jihoon, 2016-08-12 06:27:33]

Description was changed from

==========
Use https for naver search engine

Naver search engine has been providing https support for some time now.
Current http url even automatically redirects to https,
So changing naver urls to https where appropriate.

BUG=
==========

to

==========
Use https for naver search engine

Naver search engine has been providing https support for some time now.
Current http url even automatically redirects to https,
So changing naver urls to https where appropriate.

BUG=
==========

[jihoon, 2016-08-12 06:27:33]

jihoon@gmail.com changed reviewers:
+ pkasting@chromium.org, stevet@chromium.org

[Peter Kasting, 2016-08-12 07:02:28]

LGTM with one fix

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

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

[Peter Kasting, 2016-08-12 07:04:28]

Also, I don't see your email in the AUTHORS list.  Have you contributed before? 
If not, then before this is landed you need to follow the "Legal stuff" bullets
in https://www.chromium.org/developers/contributing-code .  If you have
contributed, make sure you're signed in to the codereview site with the
credentials in AUTHORS.

[jihoon, 2016-08-12 07:47:00]

Done.

Signed the CLA && Added my emails.

[Peter Kasting, 2016-08-12 08:02:13]

The CQ bit was checked by pkasting@chromium.org

[Peter Kasting, 2016-08-12 08:02:13]

The patchset sent to the CQ was uploaded after l-g-t-m from
pkasting@chromium.org
Link to the patchset: https://codereview.chromium.org/2241723002/#ps20001
(title: "Use https for naver search engine")

[commit-bot: I haz the power, 2016-08-12 08:02:28]

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-08-12 08:36:41]

Description was changed from

==========
Use https for naver search engine

Naver search engine has been providing https support for some time now.
Current http url even automatically redirects to https,
So changing naver urls to https where appropriate.

BUG=
==========

to

==========
Use https for naver search engine

Naver search engine has been providing https support for some time now.
Current http url even automatically redirects to https,
So changing naver urls to https where appropriate.

BUG=
==========

[commit-bot: I haz the power, 2016-08-12 08:36:42]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-08-12 08:38:31]

Description was changed from

==========
Use https for naver search engine

Naver search engine has been providing https support for some time now.
Current http url even automatically redirects to https,
So changing naver urls to https where appropriate.

BUG=
==========

to

==========
Use https for naver search engine

Naver search engine has been providing https support for some time now.
Current http url even automatically redirects to https,
So changing naver urls to https where appropriate.

BUG=

Committed: https://crrev.com/c33c68d1a0e8cc2374b3b23255048313fbd0fe4b
Cr-Commit-Position: refs/heads/master@{#411584}
==========

[commit-bot: I haz the power, 2016-08-12 08:38:32]

Patchset 2 (id:??) landed as
https://crrev.com/c33c68d1a0e8cc2374b3b23255048313fbd0fe4b
Cr-Commit-Position: refs/heads/master@{#411584}