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

Issue 666804 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Nov 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug

Blocking:
issue 666801



Sign in to add a comment

wifi display session mojo interfaces should be in a nested namespace

Project Member Reported by dcheng@chromium.org, Nov 18 2016

Issue description

By convention, interfaces generally go in a distinct namespace to avoid name collisions.
 

Comment 1 by dcheng@chromium.org, Nov 18 2016

Status: Available (was: Untriaged)
Owner: eero.hak...@intel.com
Status: Started (was: Available)
Project Member

Comment 4 by bugdroid1@chromium.org, Nov 25 2016

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

commit 2d1b42ce5407b8678270730317e67a52d78be15d
Author: eero.hakkinen <eero.hakkinen@intel.com>
Date: Fri Nov 25 06:15:34 2016

[chrome.displaySource] Move mojo interfaces to a nested namespace.

BUG=242107, 666804 

Review-Url: https://codereview.chromium.org/2526173003
Cr-Commit-Position: refs/heads/master@{#434440}

[modify] https://crrev.com/2d1b42ce5407b8678270730317e67a52d78be15d/extensions/browser/api/display_source/wifi_display/wifi_display_media_service_impl.cc
[modify] https://crrev.com/2d1b42ce5407b8678270730317e67a52d78be15d/extensions/browser/api/display_source/wifi_display/wifi_display_media_service_impl.h
[modify] https://crrev.com/2d1b42ce5407b8678270730317e67a52d78be15d/extensions/browser/api/display_source/wifi_display/wifi_display_session_service_impl.cc
[modify] https://crrev.com/2d1b42ce5407b8678270730317e67a52d78be15d/extensions/browser/api/display_source/wifi_display/wifi_display_session_service_impl.h
[modify] https://crrev.com/2d1b42ce5407b8678270730317e67a52d78be15d/extensions/common/mojo/wifi_display_session_service.mojom
[modify] https://crrev.com/2d1b42ce5407b8678270730317e67a52d78be15d/extensions/renderer/api/display_source/wifi_display/wifi_display_media_manager.cc
[modify] https://crrev.com/2d1b42ce5407b8678270730317e67a52d78be15d/extensions/renderer/api/display_source/wifi_display/wifi_display_media_manager.h
[modify] https://crrev.com/2d1b42ce5407b8678270730317e67a52d78be15d/extensions/renderer/api/display_source/wifi_display/wifi_display_media_pipeline.cc
[modify] https://crrev.com/2d1b42ce5407b8678270730317e67a52d78be15d/extensions/renderer/api/display_source/wifi_display/wifi_display_media_pipeline.h
[modify] https://crrev.com/2d1b42ce5407b8678270730317e67a52d78be15d/extensions/renderer/api/display_source/wifi_display/wifi_display_session.h

Status: Fixed (was: Started)

Sign in to add a comment