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

Side by Side Diff: content/public/common/context_menu_params.h

Issue 1105263004: "Load image" context menu item to reload a LoFi image. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 7 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 unified diff | Download patch
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef CONTENT_PUBLIC_COMMON_CONTEXT_MENU_PARAMS_H_ 5 #ifndef CONTENT_PUBLIC_COMMON_CONTEXT_MENU_PARAMS_H_
6 #define CONTENT_PUBLIC_COMMON_CONTEXT_MENU_PARAMS_H_ 6 #define CONTENT_PUBLIC_COMMON_CONTEXT_MENU_PARAMS_H_
7 7
8 #include <string> 8 #include <string>
9 #include <vector> 9 #include <vector>
10 10
(...skipping 65 matching lines...) Expand 10 before | Expand all | Expand 10 after
76 76
77 // This is the source URL for the element that the context menu was 77 // This is the source URL for the element that the context menu was
78 // invoked on. Example of elements with source URLs are img, audio, and 78 // invoked on. Example of elements with source URLs are img, audio, and
79 // video. 79 // video.
80 GURL src_url; 80 GURL src_url;
81 81
82 // This is true if the context menu was invoked on an image which has 82 // This is true if the context menu was invoked on an image which has
83 // non-empty contents. 83 // non-empty contents.
84 bool has_image_contents; 84 bool has_image_contents;
85 85
86 // If |media_type| is MediaTypeImage and |has_image_contents| is true, then
87 // this contains the if the image was a Lo-Fi image.
Charlie Reis 2015/05/11 22:32:58 nit: this contains whether the image
megjablon 2015/05/11 22:58:40 Done.
88 bool image_was_fetched_lo_fi;
Charlie Reis 2015/05/11 22:32:58 Same question: all the uses seem to be outside con
megjablon 2015/05/11 22:58:40 Acknowledged.
89
86 // This is the URL of the top level page that the context menu was invoked 90 // This is the URL of the top level page that the context menu was invoked
87 // on. 91 // on.
88 GURL page_url; 92 GURL page_url;
89 93
90 // This is the absolute keyword search URL including the %s search tag when 94 // This is the absolute keyword search URL including the %s search tag when
91 // the "Add as search engine..." option is clicked (left empty if not used). 95 // the "Add as search engine..." option is clicked (left empty if not used).
92 GURL keyword_url; 96 GURL keyword_url;
93 97
94 // This is the URL of the subframe that the context menu was invoked on. 98 // This is the URL of the subframe that the context menu was invoked on.
95 GURL frame_url; 99 GURL frame_url;
(...skipping 59 matching lines...) Expand 10 before | Expand all | Expand 10 after
155 // Points representing the coordinates in the document space of the start and 159 // Points representing the coordinates in the document space of the start and
156 // end of the selection, if there is one. 160 // end of the selection, if there is one.
157 gfx::Point selection_start; 161 gfx::Point selection_start;
158 gfx::Point selection_end; 162 gfx::Point selection_end;
159 #endif 163 #endif
160 }; 164 };
161 165
162 } // namespace content 166 } // namespace content
163 167
164 #endif // CONTENT_PUBLIC_COMMON_CONTEXT_MENU_PARAMS_H_ 168 #endif // CONTENT_PUBLIC_COMMON_CONTEXT_MENU_PARAMS_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698