Skip to content

Commit

Permalink
[Cormornat] cleanup unused directory
Browse files Browse the repository at this point in the history
Bug: 142693
Change-Id: I67131db78efc6b8b619c85cc4340a070bc368852
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4363058
Commit-Queue: Jeffrey Cohen <jeffreycohen@chromium.org>
Reviewed-by: Wissem Gamra <wiss@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1120823}
  • Loading branch information
Jeffrey Cohen authored and Chromium LUCI CQ committed Mar 22, 2023
1 parent a14a64a commit 1f7edbd
Show file tree
Hide file tree
Showing 16 changed files with 0 additions and 288 deletions.
4 changes: 0 additions & 4 deletions chrome/browser/BUILD.gn
Expand Up @@ -2065,7 +2065,6 @@ static_library("browser") {
"//components/cookie_config",
"//components/country_codes",
"//components/crash/core/browser",
"//components/creator",
"//components/crx_file",
"//components/custom_handlers",
"//components/device_event_log",
Expand Down Expand Up @@ -2878,7 +2877,6 @@ static_library("browser") {
"content_creation/notes/internal/note_service_factory.h",
"content_settings/request_desktop_site_web_contents_observer_android.cc",
"content_settings/request_desktop_site_web_contents_observer_android.h",
"creator/android/creator_api_bridge.cc",
"device_reauth/android/device_authenticator_android.cc",
"device_reauth/android/device_authenticator_android.h",
"device_reauth/android/device_authenticator_bridge.h",
Expand Down Expand Up @@ -3264,7 +3262,6 @@ static_library("browser") {
"//chrome/browser/commerce/merchant_viewer/android:jni_headers",
"//chrome/browser/consent_auditor/android:jni_headers",
"//chrome/browser/content_creation/notes/internal/android:jni_headers",
"//chrome/browser/creator/android:jni_headers",
"//chrome/browser/device_reauth/android:jni_headers",
"//chrome/browser/download/internal/android",
"//chrome/browser/endpoint_fetcher:jni_headers",
Expand Down Expand Up @@ -3338,7 +3335,6 @@ static_library("browser") {
"//components/content_creation/notes/core/templates",
"//components/content_settings/android",
"//components/crash/android:crash_android",
"//components/creator",
"//components/device_reauth",
"//components/embedder_support/android:context_menu",
"//components/embedder_support/android:simple_factory_key",
Expand Down
6 changes: 0 additions & 6 deletions chrome/browser/creator/android/BUILD.gn
Expand Up @@ -6,7 +6,6 @@ import("//build/config/android/rules.gni")

android_library("java") {
sources = [
"java/src/org/chromium/chrome/browser/creator/CreatorApiBridge.java",
"java/src/org/chromium/chrome/browser/creator/CreatorCoordinator.java",
"java/src/org/chromium/chrome/browser/creator/CreatorMediator.java",
"java/src/org/chromium/chrome/browser/creator/CreatorProfileView.java",
Expand Down Expand Up @@ -85,11 +84,6 @@ android_resources("creator_java_resources") {
]
}

generate_jni("jni_headers") {
sources =
[ "java/src/org/chromium/chrome/browser/creator/CreatorApiBridge.java" ]
}

robolectric_library("junit") {
sources = [
"java/src/org/chromium/chrome/browser/creator/CreatorCoordinatorTest.java",
Expand Down
77 changes: 0 additions & 77 deletions chrome/browser/creator/android/creator_api_bridge.cc

This file was deleted.

This file was deleted.

Expand Up @@ -66,8 +66,6 @@ public class CreatorCoordinatorTest {
@Mock
private WebFeedBridge.Natives mWebFeedBridgeJniMock;
@Mock
private CreatorApiBridge.Natives mCreatorBridgeJniMock;
@Mock
private FeedServiceBridge.Natives mFeedServiceBridgeJniMock;
@Mock
private FeedReliabilityLoggingBridge.Natives mFeedReliabilityLoggingBridgeJniMock;
Expand Down Expand Up @@ -106,7 +104,6 @@ public class CreatorCoordinatorTest {
@Before
public void setUpTest() {
MockitoAnnotations.initMocks(this);
mJniMocker.mock(CreatorApiBridgeJni.TEST_HOOKS, mCreatorBridgeJniMock);
mJniMocker.mock(FeedServiceBridgeJni.TEST_HOOKS, mFeedServiceBridgeJniMock);
mJniMocker.mock(WebFeedBridge.getTestHooksForTesting(), mWebFeedBridgeJniMock);
mJniMocker.mock(FeedReliabilityLoggingBridge.getTestHooksForTesting(),
Expand Down
Expand Up @@ -8,20 +8,16 @@

import android.content.Context;

import org.chromium.chrome.browser.creator.CreatorApiBridge.Creator;
import org.chromium.chrome.browser.feed.FeedServiceBridge;
import org.chromium.chrome.browser.feed.webfeed.WebFeedBridge;
import org.chromium.ui.modelutil.PropertyModel;

import java.nio.charset.StandardCharsets;

/**
* Sets up the Mediator for Cormorant Creator surface. It is based on the doc at
* https://chromium.googlesource.com/chromium/src/+/HEAD/docs/ui/android/mvc_simple_list_tutorial.md
*/
public class CreatorMediator {
private Context mContext;
private Creator mCreator;
private PropertyModel mCreatorModel;
private final CreatorSnackbarController mCreatorSnackbarController;
private SignInInterstitialInitiator mSignInInterstitialInitiator;
Expand All @@ -33,9 +29,6 @@ public class CreatorMediator {
mCreatorModel = creatorModel;
mCreatorSnackbarController = creatorSnackbarController;
mSignInInterstitialInitiator = signInInterstitialInitiator;
if (mCreatorModel.get(CreatorProperties.WEB_FEED_ID_KEY) != null) {
getCreator();
}

// Set Follow OnClick Action
mCreatorModel.set(CreatorProperties.ON_FOLLOW_CLICK_KEY, this::followClickHandler);
Expand Down Expand Up @@ -67,15 +60,4 @@ private void followingClickHandler() {
result.requestStatus, mCreatorModel.get(CreatorProperties.TITLE_KEY));
});
}

private void getCreator() {
CreatorApiBridge.getCreator(new String(mCreatorModel.get(CreatorProperties.WEB_FEED_ID_KEY),
StandardCharsets.UTF_8),
this::onGetCreator);
}

private void onGetCreator(Creator creator) {
// TODO(crbug/1374058): Get Title and Url from CreatorAPI
mCreator = creator;
}
}
Expand Up @@ -60,8 +60,6 @@ public class CreatorMediatorTest {
@Mock
private WebFeedBridge.Natives mWebFeedBridgeJniMock;
@Mock
private CreatorApiBridge.Natives mCreatorBridgeJniMock;
@Mock
private FeedStream.Natives mFeedStreamJniMock;
@Mock
private FeedServiceBridge.Natives mFeedServiceBridgeJniMock;
Expand Down Expand Up @@ -107,7 +105,6 @@ public class CreatorMediatorTest {
@Before
public void setUpTest() {
MockitoAnnotations.initMocks(this);
mJniMocker.mock(CreatorApiBridgeJni.TEST_HOOKS, mCreatorBridgeJniMock);
mJniMocker.mock(FeedStreamJni.TEST_HOOKS, mFeedStreamJniMock);
mJniMocker.mock(FeedServiceBridgeJni.TEST_HOOKS, mFeedServiceBridgeJniMock);
mJniMocker.mock(WebFeedBridge.getTestHooksForTesting(), mWebFeedBridgeJniMock);
Expand Down
21 changes: 0 additions & 21 deletions components/creator/BUILD.gn

This file was deleted.

4 changes: 0 additions & 4 deletions components/creator/COMMON_METADATA

This file was deleted.

1 change: 0 additions & 1 deletion components/creator/DIR_METADATA

This file was deleted.

1 change: 0 additions & 1 deletion components/creator/OWNERS

This file was deleted.

7 changes: 0 additions & 7 deletions components/creator/README.md

This file was deleted.

25 changes: 0 additions & 25 deletions components/creator/creator_api_impl.cc

This file was deleted.

26 changes: 0 additions & 26 deletions components/creator/creator_api_impl.h

This file was deleted.

14 changes: 0 additions & 14 deletions components/creator/public/creator_api.cc

This file was deleted.

34 changes: 0 additions & 34 deletions components/creator/public/creator_api.h

This file was deleted.

0 comments on commit 1f7edbd

Please sign in to comment.