New issue
Advanced search Search tips
Note: Color blocks (like █ or █) mean that a user may not be available. Tooltip shows the reason.

Issue 789826 link

Starred by 1 user
Status: Fixed
Owner:
█ cm.san...@samsung.com
Last visit > 30 days ago
Closed: Dec 2017
Cc:
dominickn@chromium.org
Components:
EstimatedDays: ----
NextAction: 2017-12-20
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

Convert GetWebApplicationInfo to mojom.

Project Member Reported by cm.san...@samsung.com, Nov 30 2017 
This is a tracking bug to eliminate legacy 
ChromeFrameMsg_GetWebApplicationInfo
ChromeFrameHostMsg_DidGetWebApplicationInfo,
                    WebApplicationInfo
Project Member

Comment 1 by bugdroid1@chromium.org, Dec 18 2017 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/2b601c3e7cfc0867e54b3c42fca863cbd636da4c

commit 2b601c3e7cfc0867e54b3c42fca863cbd636da4c
Author: cm.sanchi <cm.sanchi@samsung.com>
Date: Mon Dec 18 10:49:28 2017

Convert webapplicationinfo messages to use mojo.

This patch mojofies the ChromeFrameMsg_GetWebApplicationInfo
and ChromeFrameHostMsg_DidGetWebApplicationInfo message

Bug:  789826 
Change-Id: Iff07e66d41f56f85e9298fc8e2ba38ab00f3e734
Reviewed-on: https://chromium-review.googlesource.com/800130
Commit-Queue: Chandramouli Sanchi <cm.sanchi@samsung.com>
Reviewed-by: Daniel Cheng <dcheng@chromium.org>
Reviewed-by: Dominick Ng <dominickn@chromium.org>
Reviewed-by: Reilly Grant <reillyg@chromium.org>
Reviewed-by: Jochen Eisinger <jochen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#524674}
[modify] https://crrev.com/2b601c3e7cfc0867e54b3c42fca863cbd636da4c/chrome/browser/android/webapps/add_to_homescreen_data_fetcher.cc
[modify] https://crrev.com/2b601c3e7cfc0867e54b3c42fca863cbd636da4c/chrome/browser/android/webapps/add_to_homescreen_data_fetcher.h
[modify] https://crrev.com/2b601c3e7cfc0867e54b3c42fca863cbd636da4c/chrome/browser/android/webapps/add_to_homescreen_data_fetcher_unittest.cc
[modify] https://crrev.com/2b601c3e7cfc0867e54b3c42fca863cbd636da4c/chrome/browser/extensions/bookmark_app_helper_browsertest.cc
[modify] https://crrev.com/2b601c3e7cfc0867e54b3c42fca863cbd636da4c/chrome/browser/extensions/tab_helper.cc
[modify] https://crrev.com/2b601c3e7cfc0867e54b3c42fca863cbd636da4c/chrome/browser/extensions/tab_helper.h
[modify] https://crrev.com/2b601c3e7cfc0867e54b3c42fca863cbd636da4c/chrome/browser/renderer_context_menu/render_view_context_menu_browsertest.cc
[modify] https://crrev.com/2b601c3e7cfc0867e54b3c42fca863cbd636da4c/chrome/common/BUILD.gn
[modify] https://crrev.com/2b601c3e7cfc0867e54b3c42fca863cbd636da4c/chrome/common/chrome_render_frame.mojom
[modify] https://crrev.com/2b601c3e7cfc0867e54b3c42fca863cbd636da4c/chrome/common/render_messages.h
[add] https://crrev.com/2b601c3e7cfc0867e54b3c42fca863cbd636da4c/chrome/common/web_application_info_provider.typemap
[add] https://crrev.com/2b601c3e7cfc0867e54b3c42fca863cbd636da4c/chrome/common/web_application_info_provider_param_traits.h
[modify] https://crrev.com/2b601c3e7cfc0867e54b3c42fca863cbd636da4c/chrome/renderer/chrome_render_frame_observer.cc
[modify] https://crrev.com/2b601c3e7cfc0867e54b3c42fca863cbd636da4c/chrome/renderer/chrome_render_frame_observer.h
[modify] https://crrev.com/2b601c3e7cfc0867e54b3c42fca863cbd636da4c/chrome/typemaps.gni

Comment 2 by cm.san...@samsung.com, Dec 20 2017

Status: Started (was: Assigned)

Comment 3 by cm.san...@samsung.com, Dec 20 2017

NextAction: 2017-12-20
Status: Fixed (was: Started)

Comment 4 by monor...@bugs.chromium.org, Dec 20 2017 

The NextAction date has arrived: 2017-12-20
►

Sign in to add a comment