Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(942)

Unified Diff: chrome/browser/ui/views/apps/app_info_dialog/app_info_tab_permissions_views.cc

Issue 180723010: Added tabs to the App Info dialog. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Linking moved files Created 6 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/views/apps/app_info_dialog/app_info_tab_permissions_views.cc
diff --git a/chrome/browser/ui/views/apps/app_info_dialog/app_info_tab_permissions_views.cc b/chrome/browser/ui/views/apps/app_info_dialog/app_info_tab_permissions_views.cc
new file mode 100644
index 0000000000000000000000000000000000000000..fe63cce19a9eebd701f97275fbf8c175bf6cda95
--- /dev/null
+++ b/chrome/browser/ui/views/apps/app_info_dialog/app_info_tab_permissions_views.cc
@@ -0,0 +1,110 @@
+// Copyright 2014 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "chrome/browser/ui/views/apps/app_info_dialog/app_info_tab_permissions_views.h"
+
+#include "extensions/common/extension.h"
+#include "extensions/common/permissions/permission_message_provider.h"
+#include "extensions/common/permissions/permission_set.h"
+#include "grit/generated_resources.h"
+#include "ui/base/l10n/l10n_util.h"
+#include "ui/views/controls/label.h"
+#include "ui/views/layout/grid_layout.h"
+#include "ui/views/layout/layout_constants.h"
+
+PermissionsScrollView::PermissionsScrollView(int min_height,
+ int max_height,
+ const extensions::Extension* app)
+ : message_center::BoundedScrollView(min_height, max_height) {
+ views::View* inner_scrollable_view = new views::View();
+ this->SetContents(inner_scrollable_view);
+
+ // Get the permission messages for the app
Matt Giuca 2014/03/04 01:36:57 Nit: Comments are complete sentences so they shoul
sashab 2014/03/04 04:10:58 Done to all comments like this.
+ std::vector<base::string16> permission_messages =
+ extensions::PermissionMessageProvider::Get()->GetWarningMessages(
+ app->GetActivePermissions(), app->GetType());
+
+ // Create the layout
+ views::GridLayout* layout =
+ views::GridLayout::CreatePanel(inner_scrollable_view);
+ inner_scrollable_view->SetLayoutManager(layout);
+
+ // Create 2 columns: one for the bullet, one for the bullet text
+ static const int kPermissionBulletsColumnSetId = 1;
+ views::ColumnSet* permission_bullets_column_set =
+ layout->AddColumnSet(kPermissionBulletsColumnSetId);
+ permission_bullets_column_set->AddPaddingColumn(0, 10);
+ permission_bullets_column_set->AddColumn(views::GridLayout::LEADING,
+ views::GridLayout::LEADING,
+ 0,
+ views::GridLayout::USE_PREF,
+ 0, // no fixed width
+ 0);
+ permission_bullets_column_set->AddPaddingColumn(0, 5);
+ permission_bullets_column_set->AddColumn(views::GridLayout::LEADING,
+ views::GridLayout::LEADING,
+ 0,
+ views::GridLayout::USE_PREF,
+ 0, // no fixed width
+ 0);
+
+ // Add permissions to scrollable view
+ for (std::vector<base::string16>::const_iterator it =
+ permission_messages.begin();
+ it != permission_messages.end();
+ ++it) {
+ views::Label* permission_label = new views::Label(*it);
+
+ permission_label->SetMultiLine(true);
+ permission_label->SetHorizontalAlignment(gfx::ALIGN_LEFT);
+ permission_label->SizeToFit(250);
+
+ layout->StartRow(0, kPermissionBulletsColumnSetId);
+ // Extract only the bullet from the IDS_EXTENSION_PERMISSION_LINE text.
+ layout->AddView(new views::Label(l10n_util::GetStringFUTF16(
+ IDS_EXTENSION_PERMISSION_LINE, base::string16())));
+ // Place the text second, so multi-lined permissions line up below the
+ // bullet.
+ layout->AddView(permission_label);
+
+ layout->AddPaddingRow(0, views::kRelatedControlVerticalSpacing);
+ }
+}
+
+PermissionsScrollView::~PermissionsScrollView() {}
+
+AppInfoPermissionsTabView::AppInfoPermissionsTabView(
+ gfx::NativeWindow parent_window,
+ Profile* profile,
+ const extensions::Extension* app,
+ const base::Closure& close_callback)
+ : AppInfoTabView(parent_window, profile, app, close_callback) {
+
+ // Create the layout
+ views::GridLayout* layout = views::GridLayout::CreatePanel(this);
+ SetLayoutManager(layout);
+
+ static const int kPermissionsColumnSetId = 0;
+ views::ColumnSet* permissions_column_set =
+ layout->AddColumnSet(kPermissionsColumnSetId);
+ permissions_column_set->AddColumn(views::GridLayout::LEADING,
+ views::GridLayout::LEADING,
+ 0,
+ views::GridLayout::USE_PREF,
+ 0, // no fixed width
+ 0);
+
+ views::Label* required_permissions_heading = new views::Label(
+ l10n_util::GetStringUTF16(IDS_EXTENSION_PROMPT_CAN_ACCESS));
+ required_permissions_heading->SetHorizontalAlignment(gfx::ALIGN_LEFT);
+
+ layout->StartRow(0, kPermissionsColumnSetId);
+ layout->AddView(required_permissions_heading);
+ layout->AddPaddingRow(0, views::kRelatedControlVerticalSpacing);
+ layout->StartRow(0, kPermissionsColumnSetId);
+ layout->AddView(new PermissionsScrollView(0, 100, app));
+ layout->AddPaddingRow(0, views::kUnrelatedControlHorizontalSpacing);
+}
+
+AppInfoPermissionsTabView::~AppInfoPermissionsTabView() {}

Powered by Google App Engine
This is Rietveld 408576698